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

GHA: Exclude Python 3.9 runs for milestone branch #1913

Merged
merged 1 commit into from
Aug 22, 2023

Conversation

felixfontein
Copy link
Contributor

SUMMARY

Fixes GHA CI.

Ref: ansible-collections/news-for-maintainers#56

ISSUE TYPE
  • Test Pull Request
COMPONENT NAME

GHA sanity and unit tests

@felixfontein
Copy link
Contributor Author

Another reason why simply running ansible-test sanity --docker and ansible-test units --docker would be better (ansible-network/github_actions#120) - that automatically uses the supported Python versions for every ansible-core version.

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.
https://ansible.softwarefactory-project.io/zuul/buildset/ac9f84543ebe4aa6a6fd0b429c331d16

ansible-galaxy-importer FAILURE in 5m 39s (non-voting)
✔️ build-ansible-collection SUCCESS in 14m 44s

@felixfontein
Copy link
Contributor Author

The corresponding change was already made for ansible-network/github_actions in ansible-network/github_actions@ad77b41.

@felixfontein felixfontein added the mergeit Merge the PR (SoftwareFactory) label Aug 20, 2023
@tremble tremble added mergeit Merge the PR (SoftwareFactory) and removed mergeit Merge the PR (SoftwareFactory) labels Aug 21, 2023
@felixfontein felixfontein added mergeit Merge the PR (SoftwareFactory) and removed mergeit Merge the PR (SoftwareFactory) labels Aug 22, 2023
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).
https://ansible.softwarefactory-project.io/zuul/buildset/7348f1ac6e3641a488ab05d735cac49e

ansible-galaxy-importer FAILURE in 5m 08s (non-voting)
✔️ build-ansible-collection SUCCESS in 12m 29s

@softwarefactory-project-zuul
Copy link
Contributor

Pull request merge failed: Resource not accessible by integration, You may need to manually rebase your PR and retry.

@felixfontein felixfontein merged commit 3d35782 into ansible-collections:main Aug 22, 2023
83 of 153 checks passed
@felixfontein
Copy link
Contributor Author

@alinabuzachis @tremble thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community_review mergeit Merge the PR (SoftwareFactory) needs_triage python3 small_patch Hopefully easy to review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants