Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: Drop rhel-8* from supported OSes, fix for ruff 0.3.1 (fixes failing "tox" test) #20149

Merged
merged 2 commits into from
Mar 8, 2024

Conversation

martinpitt
Copy link
Member

That doesn't work any more, RHEL 8 testing now happens on the rhel-8 branch.


This is primarily an excuse to work out our shiny new job-runner CI.

That doesn't work any more, RHEL 8 testing now happens on the `rhel-8`
branch.
@martinpitt
Copy link
Member Author

Ah, the tox failure is due to new ruff 0.3.1, similar to what I just fixed in cockpit-project/bots#6035 . I'll fix it here.

@martinpitt martinpitt changed the title test: Drop rhel-8* from supported OSes test: Drop rhel-8* from supported OSes, fix for ruff 0.3.1 Mar 7, 2024
@martinpitt martinpitt requested review from allisonkarlitskaya and jelly and removed request for jelly and allisonkarlitskaya March 7, 2024 16:39
@martinpitt martinpitt marked this pull request as draft March 7, 2024 17:07
ruff 0.3.1, which now runs in our "tox" test, and which we will get with
the next tasks container refresh), starts complaining about some
.format() usages.
@martinpitt martinpitt marked this pull request as ready for review March 8, 2024 04:36
@martinpitt
Copy link
Member Author

The /devel failure is a flake, but only in the 2nd retry. But primarily it is a failure with job-runner to upload coverage attachment, see cockpit-project/bots#6042 . This doesn't need to block landing this PR.

@martinpitt martinpitt changed the title test: Drop rhel-8* from supported OSes, fix for ruff 0.3.1 test: Drop rhel-8* from supported OSes, fix for ruff 0.3.1 (fixes failing "tox" test) Mar 8, 2024
@martinpitt
Copy link
Member Author

Re-running, as the "directory attachments" fix landed.

Copy link
Member

@allisonkarlitskaya allisonkarlitskaya left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just out of curiosity: was that a case of --fix or did you do it?

@martinpitt
Copy link
Member Author

@allisonkarlitskaya I did it manually. I tried --fix on the bots PR, and it didn't want to do anything.

@martinpitt martinpitt merged commit 6b3fbba into cockpit-project:main Mar 8, 2024
74 checks passed
@martinpitt martinpitt deleted the readme-c8 branch March 8, 2024 07:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants