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

Lock Podman major version #613

Merged

Conversation

matt-elastic
Copy link
Contributor

Summary

To prevent automatic Podman major version updates, configure the Podman version to be locked while still allowing minor and patch updates.

Changes

1 2 3

@matt-elastic matt-elastic marked this pull request as ready for review February 27, 2025 13:10
Copy link
Contributor

@kilfoyle kilfoyle left a comment

Choose a reason for hiding this comment

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

LGTM! 🚢
@florent-leborgne It'd be nice to have a second set of eyes since I haven't viewed many PRs in the new system yet.

Copy link
Contributor

@florent-leborgne florent-leborgne left a comment

Choose a reason for hiding this comment

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

LGTM as well, thanks for making those changes @matt-elastic!
Just to be sure, this is just for ECE 4.0 and later? Or should we also port this back to the "old docs" for ECE 3.8 or previous versions?

@florent-leborgne
Copy link
Contributor

Merging regardless of my previous question as it would be a different PR anyways in /cloud ^^

@florent-leborgne florent-leborgne enabled auto-merge (squash) February 27, 2025 14:36
@matt-elastic
Copy link
Contributor Author

Thanks a lot for your reviews @kilfoyle @florent-leborgne
I think these changes need to be back ported to previous versions as well.

@florent-leborgne florent-leborgne merged commit feea1d1 into elastic:main Feb 27, 2025
4 checks passed
@matt-elastic matt-elastic deleted the feature/lock-podman-major-version branch February 27, 2025 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants