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
Bug 2109697: Port to ansible-core #12397
Bug 2109697: Port to ansible-core #12397
Conversation
Skipping CI for Draft Pull Request. |
475da2a
to
7829b85
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
7829b85
to
b9b70a2
Compare
/test all |
b9b70a2
to
6ebdfa4
Compare
/test all |
6ebdfa4
to
32506ce
Compare
/test all |
32506ce
to
0987eb3
Compare
/test e2e-aws-workers-rhel8 |
0987eb3
to
9806df8
Compare
/test e2e-aws-workers-rhel8 |
1 similar comment
/test e2e-aws-workers-rhel8 |
9806df8
to
a090290
Compare
/test e2e-aws-workers-rhel8 |
a090290
to
8326fb4
Compare
0f715a5
to
bffaac3
Compare
bffaac3
to
7909896
Compare
/retest |
41d3692
to
d1a4c1e
Compare
d1a4c1e
to
64643d3
Compare
/retest |
@yselkowitz: all tests passed! Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
This is a big one, so if need be let's discuss further. |
Would you mind elaborating a little bit more on these options? I think we should try to get together for a meeting soon (I will send an invite)--I will try to educate myself on this in the meantime. Are the .py files you added for each of these dependencies a temporary workaround or are these expected to be merged? |
The patch in its current state represents the "bundled" option, in which case the .py files are the dependencies, and barring a much more complicated path to extract these from their collection tarballs in the spec file, would be added here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: barbacbd, yselkowitz The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/cherry-pick release-4.11 |
@yselkowitz: #12397 failed to apply on top of branch "release-4.11":
In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
cherry-pick depends on CI backport #12412 |
@yselkowitz: All pull requests linked via external trackers have merged: Bugzilla bug 2109697 has been moved to the MODIFIED state. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/cherry-pick release-4.11 |
@yselkowitz: new pull request created: #12416 In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Ansible Engine 2.9 is going out of support, and being replaced by ansible-core in RHEL AppStream. Therefore, openshift-ansible needs to migrate. Since this requires modules which are no longer builtin the core but provided through collections --
seboolean
andsysctl
from ansible.posix, andini_file
from community.general -- those need to be either bundled or their collections packaged as RPMs./cc @patrickdillon @barbacbd