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

parted option align: undefined to maintain parted 1.8.3 compatibility #405

Merged

Conversation

rosowiecki
Copy link
Contributor

SUMMARY

Added option align: undefined to support parted versions < 2.1

ISSUE TYPE
  • Bugfix Pull Request
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:

  • omit -a option for 'optimal' (parted's default setting is 'optimal' too, works in my patched ansible 2.7)
  • update required version of parted to 2.1 instead

@ansibullbot ansibullbot added affects_2.10 bug This issue/PR relates to a bug module module labels May 23, 2020
@rosowiecki
Copy link
Contributor Author

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?)

@rosowiecki
Copy link
Contributor Author

Regarding Zuul tests failure: #402

@rosowiecki rosowiecki changed the title option align: undefined to maintain parted 1.8.3 compatibility parted option align: undefined to maintain parted 1.8.3 compatibility May 24, 2020
@ColOfAbRiX
Copy link

ColOfAbRiX commented May 29, 2020

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
When Python 2.6 was out probably the majority of systems were still using parted 1.8.3. So I would say we still have to support parted 1.8.3.

I think the way you've done in this PR is the correct way

@pabelanger
Copy link

recheck

@felixfontein
Copy link
Collaborator

Please add a changelog fragment. Then I think this can be merged (as @ColOfAbRiX seems to approve).

@ansibullbot ansibullbot added the stale_ci CI is older than 7 days, rerun before merging label Jun 15, 2020
@ansibullbot ansibullbot removed the stale_ci CI is older than 7 days, rerun before merging label Jun 16, 2020
Copy link
Collaborator

@felixfontein felixfontein left a comment

Choose a reason for hiding this comment

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

LGTM

@felixfontein felixfontein merged commit 9e28f3c into ansible-collections:master Jun 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue/PR relates to a bug community_review module module
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants