-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
parted option align: undefined to maintain parted 1.8.3 compatibility #405
parted option align: undefined to maintain parted 1.8.3 compatibility #405
Conversation
I couldn'd find what's special about 1.8.3 version anyway? Maybe it should be just 1.8... (@ColOfAbRiX, what do you think?) |
Regarding Zuul tests failure: #402 |
I'm trying to understand if we're still supposed to support parted 1.8, I'm going to write down my notes as I look up for things. At the moment the earlier maintained version of Ansible is 2.7 (https://docs.ansible.com/ansible/2.9/reference_appendices/release_and_maintenance.html#release-status) Ansible 2.7 must support Python 2.6 on the managed nodes (https://docs.ansible.com/ansible/2.7/installation_guide/intro_installation.html#managed-node-requirements) Python 2.6 was released in 2008 (https://www.python.org/downloads/release/python-260/) Parted 1.8.3 was released in 2007 (http://freshmeat.sourceforge.net/projects/gnuparted/releases), Parted 2.0 in 2009 (https://git.savannah.gnu.org/cgit/parted.git/tag/?h=v2.0) Result I think the way you've done in this PR is the correct way |
recheck |
Please add a changelog fragment. Then I think this can be merged (as @ColOfAbRiX seems to approve). |
Co-authored-by: Felix Fontein <[email protected]>
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
SUMMARY
Added option align: undefined to support parted versions < 2.1
ISSUE TYPE
COMPONENT NAME
parted
ADDITIONAL INFORMATION
I've actually tested parted 1.8.3 on some ancient system, but it failed. The reason was that -a option was added in parted 2.1.
Alternative solutions: