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 version references in README.md #1642

Closed
wants to merge 1 commit into from

Conversation

wbclark
Copy link
Contributor

@wbclark wbclark commented Jan 23, 2023

Older Ansible versions are now EOL, while 2.12 is still receiving security updates until May 2023 at least. Also, the devel and nightly setups haven't used centos7 for a while now.

@@ -26,7 +26,7 @@ Forklift provides tools to create Foreman/Katello environments for development,
### Requirements

* Vagrant - 2.2+ - Both the VirtualBox and Libvirt providers are tested
* Ansible - 2.9+
* Ansible - 2.12+
Copy link
Member

Choose a reason for hiding this comment

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

In #1641 it's suggested recent 2.9 should still work.

Copy link
Contributor Author

@wbclark wbclark Jan 23, 2023

Choose a reason for hiding this comment

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

Thanks, I missed the GH issue and got here via https://community.theforeman.org/t/problems-using-forklift/32003

Even if later 2.9+ still works, perhaps it is not ideal to advertise or recommend this fact considering there are multiple EOL versions between 2.9+ and the current maintenance branch.

EDIT: I initially wrote some other thoughts here that were prompted by this, but to avoid getting bogged down, I've opened another pull request.

Older Ansible versions are now EOL, while 2.12 is still receiving security
updates until May 2023 at least. Also, the devel and nightly setups haven't
used centos7 for a while now.
@wbclark wbclark changed the title Update README.md Update version references in README.md Jan 23, 2023
@wbclark
Copy link
Contributor Author

wbclark commented Feb 16, 2023

Closing in favor of #1643 , thanks for the feedback on both

@wbclark wbclark closed this Feb 16, 2023
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.

2 participants