Bug 2145125 - pep8 job fails due to bad pre-commit configuration
Summary: pep8 job fails due to bad pre-commit configuration
Keywords:
Status: NEW
Alias: None
Product: Red Hat OpenStack
Classification: Red Hat
Component: python-automaton
Version: 17.0 (Wallaby)
Hardware: Unspecified
OS: Unspecified
unspecified
low
Target Milestone: ---
: ---
Assignee: OSP Team
QA Contact: Nobody
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2022-11-23 09:59 UTC by Szymon Datko
Modified: 2023-06-26 06:56 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed:
Target Upstream Version:
Embargoed:

Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker OSP-20378 0 None None None 2022-11-23 10:30:58 UTC

Description Szymon Datko 2022-11-23 09:59:44 UTC 
Hello,

there is an issue with the pep8 we run as part of Component CI for OSP 17.0 and OSP 17.1

The problem is, the project uses git pre-commit hooks as part of pep8 tox environment and specifies the old, not existing anymore repository of flake8, which was recently moved from GitLab to GitHub.

Some details: https://www.reddit.com/r/Python/comments/yvfww8/flake8_took_down_the_gitlab_repository_in_favor/

Yours, Szymon
Comment 1 Szymon Datko 2022-11-23 10:03:17 UTC 
In-job workaround with sed was implemented for now: https://github.com/RedHatCRE/znoyder/pull/112

But in case of developers running the tests locally, the fix in downstream source code shall be implemented.

Note: the issue is also present in upstream wallaby https://opendev.org/openstack/automaton/src/branch/stable/wallaby/.pre-commit-config.yaml but was recently fixed in master https://review.opendev.org/c/openstack/automaton/+/782440
Note You need to log in before you can comment on or make changes to this bug.