Fix CI and do cleanup #162
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Skip these for now because they fail due to processes entering zombie status
This change breaks CI. This reverts commit 867f7ae.
This gets the CI working with Ubuntu 22.04 (Jammy) inside a docker container that runs on ubuntu-latest.
Closed
|
It seems that the CI fails after merging. Any ideas? 
|
|
I think that's because github actions needs write permissions for packages to upload the CI docker image, should be somewhere in the settings |
|
@jd can you go to Settings -> Actions in sidebar menu -> under Workflow permissions section and verify "Read and write permissions" is checked |
|
|
not sure what permission is needed, but the push from the CI job to packages is failing, it does the same things as gnocchi does for the container image |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This gets the CI working again on Ubuntu 22.04 (Jammy) with Python 3.10, we are not testing with Python 3.9 right now because we need the Ceph jobs to be skipped for that because the python3-rados package is required for that.
This reverts commit 867f7ae that was wrong but not caught due to no CI.
This moves away from PBR to just using setuptools and adds a job to verify building, also removes the support for Python 3.6 and 3.7 in setup.cfg which means the minimum supported version will be Python 3.8 even though that is not tested as of now.