Hi Everyone,
We’re in the process of relaxing our ...
# questionsm
Mate Scharnitzky
05/10/2023, 3:33 PMHi Everyone,
We’re in the process of relaxing our kedro dependencies in our repository. The current Kedro version is
==0.18.3~=0.18.30.18.8 Copy code
KeyError: 'logging'0.18.*j
Juan Luis
05/10/2023, 3:35 PM@Mate Scharnitzky all the 0.18.x releases should be backward compatible yes. are you able to share a full traceback to know where does that come from?
m
Mate Scharnitzky
05/10/2023, 3:37 PMThis is what the log looks like on CI.
Mate Scharnitzky
05/10/2023, 3:40 PMOk, actually I’m getting confusing info about what kedro version CI is using. Let me clarify that.
👍🏼 1
j
Juan Luis
05/10/2023, 3:41 PMfor the record, the
logging_config = self._get_config_loader()["logging"]Juan Luis
05/10/2023, 3:41 PMcould be something else
m
Mate Scharnitzky
05/10/2023, 3:57 PMthanks! I’ll look into it
a
Antony Milne
05/10/2023, 4:09 PMAre you using a custom configloader?
m
Mate Scharnitzky
05/10/2023, 4:15 PMWe use Jinja and we should deprecate that soon
Mate Scharnitzky
05/10/2023, 4:17 PMI don’t think that’s the issue as one of the apps don’t fail and it also uses Jinja
Mate Scharnitzky
05/10/2023, 4:20 PMLet me get back to you tomorrow with more thoughts from the team.
Mate Scharnitzky
05/10/2023, 4:21 PMIf there is nothing obvious from your side, then my hunch is that there might some issues how the dependencies are installed or some package config error.
a
Antony Milne
05/10/2023, 4:38 PMI’m pretty sure we haven’t seen this problem before so indeed I think it’s something peculiar to your setup. I would double check what version of kedro is being run here.
Antony Milne
05/10/2023, 4:40 PMIt’s plausible there’s been something overlooked on our side with the changes to the configloader, but if that was the case then I’d expect you to get the same problem on other apps with the same configloader and for many other people to have similar problems.
Antony Milne
05/10/2023, 4:41 PMLet us know how you get on though, because this is not a very revealing error message so I’ll be curious to see if it’s something we should improve anyway!
m
Mate Scharnitzky
05/10/2023, 7:14 PMThanks, Antony! I’ll keep you posted.
👍🏼 1
Mate Scharnitzky
05/15/2023, 9:50 AMHi Antony, Juan,
Mate Scharnitzky
05/15/2023, 9:50 AMI just wanted to get back on the above topic.
Mate Scharnitzky
05/15/2023, 9:51 AMWe haven’t found the exact root cause of the problem, but we’re quite sure it’s in connection with our custom config loader + multi-runner.
Mate Scharnitzky
05/15/2023, 9:52 AMWe still use jinja templated config loader but it’s in our backlog to investigate how config should be done with omegaconf + multi-runner, so we will upgrade our kedro version as part of that effort.
3 Views