Skip to content

Bump ansible to v2.16.18#1972

Merged
k8s-ci-robot merged 1 commit intokubernetes-sigs:mainfrom
mboersma:bump-ansible
Apr 7, 2026
Merged

Bump ansible to v2.16.18#1972
k8s-ci-robot merged 1 commit intokubernetes-sigs:mainfrom
mboersma:bump-ansible

Conversation

@mboersma
Copy link
Copy Markdown
Contributor

@mboersma mboersma commented Apr 1, 2026

Change description

Updates ansible-core to v2.16.18, supporting Python 3.10-3.12.

Related issues

See #1925 for prior art.

Additional context

See also #1894. I think we can get there, but maybe in baby steps...

@k8s-ci-robot k8s-ci-robot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Apr 1, 2026
@mboersma mboersma requested review from drew-viles and removed request for AverageMarcus April 1, 2026 14:18
@mboersma
Copy link
Copy Markdown
Contributor Author

mboersma commented Apr 1, 2026

/hold

I realize ansible-core 2.17 also supports Python 3.10-3.12. I'll open another PR and see if we can bump up to that.

@k8s-ci-robot k8s-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Apr 1, 2026
@mboersma
Copy link
Copy Markdown
Contributor Author

mboersma commented Apr 1, 2026

/test pull-azure-sigs
/test pull-ova-all
/test pull-gcp-all

@mboersma
Copy link
Copy Markdown
Contributor Author

mboersma commented Apr 1, 2026

/test pull-lint

@mboersma
Copy link
Copy Markdown
Contributor Author

mboersma commented Apr 1, 2026

Note that ansible-core's support matrix claims broader support for Python versions, but the packages available at pypi are much more constrained, and that's what determines whether they can actually be installed with a given version of Python.

@mboersma
Copy link
Copy Markdown
Contributor Author

mboersma commented Apr 1, 2026

/hold cancel

This is a safe change. Let's revisit #1973 and #1974 after this merges.

@k8s-ci-robot k8s-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Apr 1, 2026
@mboersma
Copy link
Copy Markdown
Contributor Author

mboersma commented Apr 1, 2026

/retest

@mboersma
Copy link
Copy Markdown
Contributor Author

mboersma commented Apr 2, 2026

/retest

Transient error.

Copy link
Copy Markdown
Contributor

@drew-viles drew-viles left a comment

Choose a reason for hiding this comment

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

/lgtm
/approve

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Apr 7, 2026
@k8s-ci-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: drew-viles

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Apr 7, 2026
@k8s-ci-robot k8s-ci-robot merged commit f65b489 into kubernetes-sigs:main Apr 7, 2026
10 checks passed
@mboersma mboersma deleted the bump-ansible branch April 7, 2026 13:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants