Hello, we observe the following issue when running pep8 job in Component CI for OSP 17 and 17.1 ``` cliff/tests/test_app.py:297:23: E275 missing whitespace after keyword ``` Indeed there is an issue in the mentioned line. The problem was recently closed in upstream master: https://review.opendev.org/c/openstack/cliff/+/852177 Can we have it cherry-picked to OSP 17 and OSP 17.1 code as well? Yours, Szymon
For now we proceed with a in-job workaround to make the job functional and voting: https://github.com/RedHatCRE/znoyder/pull/101 Tested here: https://code.engineering.redhat.com/gerrit/c/python-cliff/+/420536 When the proper change is cherry-picked to downstream, we will remove the workaround.
So one other thing to consider is if we have something out of sync with upstream since upstream pep8 job for stable/wallaby is either broken as well and we should fix there first or it is fine and we are doing something different from upstream stable/wallaby (17.1 and 17.0)
> Jon Schlueter 2022-12-15 16:17:16 UTC > Assignee: jschluet → sdatko From Glossary: – Assignee ––– The person in charge of resolving the bug. I only introduced a workaround, but I feel I have no power here to anything more. What else is expected?
So, at this point can we close fixed upstream? and if the change comes in we are good? or do we need to dig into why the linter job that we are using is complaining but the upstream linter job is not complaining about this existing code defect?
As per the first comment, there is nothing to dig: > The problem was recently closed in upstream master: > https://review.opendev.org/c/openstack/cliff/+/852177 I can also see there was a cherry-pick to wallaby done in the meantime. So, if the repos are synced, we are good.