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

Update CI #1106

Merged
merged 1 commit into from
Jan 31, 2022
Merged

Update CI #1106

merged 1 commit into from
Jan 31, 2022

Conversation

fao89
Copy link
Contributor

@fao89 fao89 commented Jan 26, 2022

I noticed CI update workflow is broken: https://github.com/ansible/galaxy_ng/actions/workflows/update_ci.yml
So I manually update it from plugin-template

@app-sre-bot
Copy link
Collaborator

Can one of the admins verify this patch?

@@ -85,11 +86,13 @@ tasking_allow_async_unsafe: true
test_azure: true
test_bindings: false
test_cli: false
test_deprecations: true
Copy link
Contributor Author

Choose a reason for hiding this comment

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

PR fails because we are testing pulpcore deprecations

Copy link
Member

Choose a reason for hiding this comment

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

@fao89 should we keep testing it? or change to false in this PR?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I don't know, it is up to galaxy
I personally think this deprecation test could be moved to your nightly latest

@@ -1,4 +1,4 @@
FROM {{ ci_base | default("pulp/pulp-ci-centos:" + pulp_container_tag) }}
FROM {{ ci_base | default("ghcr.io/pulp/pulp-ci-centos:" + pulp_container_tag) }}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this was my main motivator for this PR, use github container registry instead of docker

No-Issue
@rochacbruno
Copy link
Member

/retest

@fao89
Copy link
Contributor Author

fao89 commented Jan 31, 2022

could you please merge?

@chouseknecht chouseknecht merged commit df39958 into ansible:master Jan 31, 2022
bmclaughlin added a commit to bmclaughlin/galaxy_ng that referenced this pull request Feb 1, 2022
Issue: AAH-1243
Issue: AAH-1202
newswangerd pushed a commit that referenced this pull request Feb 8, 2022
* Update pulpcore and pulp_ansible dependencies
Issue: AAH-1243
Issue: AAH-1202

* update CI with new pulpcore, pulp_ansible dependencies
Issue: AAH-1243
Issue: AAH-1202

* Turn off deprecations check on 4.4 per #1106
Issue: AAH-1243
Issue: AAH-1202
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants