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

ansible: Update to latest supported releases #48082

Merged
merged 4 commits into from Nov 3, 2018
Merged

Conversation

sengaya
Copy link
Contributor

@sengaya sengaya commented Oct 8, 2018

Motivation for this change

Update to latest supported ansible releases, see https://docs.ansible.com/ansible/latest/reference_appendices/release_and_maintenance.html#release-status

Things done
  • removed ansible 2.4

  • updated to latest minor release for 2.5 + 2.6

  • added ansible 2.7

  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)

  • Built on platform(s)

    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)

  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"

  • Tested execution of all binary files (usually in ./result/bin/)

  • Determined the impact on package closure size (by running nix path-info -S before and after)

  • Fits CONTRIBUTING.md.


I had to add the preBuild step for ansible-lint because some tests failed during nix-shell -p nox --run "nox-review wip". I don't really understand why this was no issue before this change.

cc @jgeerds @joamaki

@dasJ
Copy link
Member

dasJ commented Oct 31, 2018

Looks good! Any chance this can get merged?

@sengaya
Copy link
Contributor Author

sengaya commented Nov 1, 2018

@GrahamcOfBorg build ansible_2_5 ansible_2_6 ansible_2_7

@@ -19,6 +19,10 @@ pythonPackages.buildPythonPackage rec {
patchShebangs bin/ansible-lint
'';

preBuild = ''
export HOME="$TMP"
Copy link
Contributor

Choose a reason for hiding this comment

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

Why is this needed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

not sure, see original PR message

@c0bw3b
Copy link
Contributor

c0bw3b commented Nov 1, 2018

(only known users can invoke the bot)

Could you please split each updates in its own commit?
ansible_2_5: 2.5.2 -> 2.5.11
ansible_2_6: 2.6.2 -> 2.6.7
ansible_2_7: init at 2.7.1 in which you also do ansible2 = ansible_2_7
ansible_2_4: remove

Also for the removal of v2.4 : it needs to be documented in the release notes for 19.03
If you want the updates to be merged faster then maybe take v2.4 removal to a separate PR entirely

@c0bw3b
Copy link
Contributor

c0bw3b commented Nov 1, 2018

@GrahamcOfBorg build ansible_2_7

@c0bw3b c0bw3b self-assigned this Nov 1, 2018
@GrahamcOfBorg
Copy link

Success on x86_64-linux (full log)

Attempted: ansible_2_7

Partial log (click to expand)

wrapping `/nix/store/3ff65yi1l0hbr27bz0av0fb1mkwn3kjk-python2.7-ansible-2.7.1/bin/.ansible-connection-wrapped'...
wrapping `/nix/store/3ff65yi1l0hbr27bz0av0fb1mkwn3kjk-python2.7-ansible-2.7.1/bin/.ansible-galaxy-wrapped'...
wrapping `/nix/store/3ff65yi1l0hbr27bz0av0fb1mkwn3kjk-python2.7-ansible-2.7.1/bin/.ansible-pull-wrapped'...
wrapping `/nix/store/3ff65yi1l0hbr27bz0av0fb1mkwn3kjk-python2.7-ansible-2.7.1/bin/.ansible-vault-wrapped'...
wrapping `/nix/store/3ff65yi1l0hbr27bz0av0fb1mkwn3kjk-python2.7-ansible-2.7.1/bin/.ansible-wrapped'...
wrapping `/nix/store/3ff65yi1l0hbr27bz0av0fb1mkwn3kjk-python2.7-ansible-2.7.1/bin/.ansible-playbook-wrapped'...
wrapping `/nix/store/3ff65yi1l0hbr27bz0av0fb1mkwn3kjk-python2.7-ansible-2.7.1/bin/.ansible-doc-wrapped'...
wrapping `/nix/store/3ff65yi1l0hbr27bz0av0fb1mkwn3kjk-python2.7-ansible-2.7.1/bin/.ansible-console-wrapped'...
wrapping `/nix/store/3ff65yi1l0hbr27bz0av0fb1mkwn3kjk-python2.7-ansible-2.7.1/bin/.ansible-inventory-wrapped'...
/nix/store/3ff65yi1l0hbr27bz0av0fb1mkwn3kjk-python2.7-ansible-2.7.1

@GrahamcOfBorg
Copy link

Success on aarch64-linux (full log)

Attempted: ansible_2_7

Partial log (click to expand)

wrapping `/nix/store/dvjvnd39fd84wiakd2hcv9hhlc69jjcg-python2.7-ansible-2.7.1/bin/.ansible-pull-wrapped'...
wrapping `/nix/store/dvjvnd39fd84wiakd2hcv9hhlc69jjcg-python2.7-ansible-2.7.1/bin/.ansible-playbook-wrapped'...
wrapping `/nix/store/dvjvnd39fd84wiakd2hcv9hhlc69jjcg-python2.7-ansible-2.7.1/bin/.ansible-inventory-wrapped'...
wrapping `/nix/store/dvjvnd39fd84wiakd2hcv9hhlc69jjcg-python2.7-ansible-2.7.1/bin/.ansible-galaxy-wrapped'...
wrapping `/nix/store/dvjvnd39fd84wiakd2hcv9hhlc69jjcg-python2.7-ansible-2.7.1/bin/.ansible-doc-wrapped'...
wrapping `/nix/store/dvjvnd39fd84wiakd2hcv9hhlc69jjcg-python2.7-ansible-2.7.1/bin/.ansible-console-wrapped'...
wrapping `/nix/store/dvjvnd39fd84wiakd2hcv9hhlc69jjcg-python2.7-ansible-2.7.1/bin/.ansible-connection-wrapped'...
wrapping `/nix/store/dvjvnd39fd84wiakd2hcv9hhlc69jjcg-python2.7-ansible-2.7.1/bin/.ansible-config-wrapped'...
wrapping `/nix/store/dvjvnd39fd84wiakd2hcv9hhlc69jjcg-python2.7-ansible-2.7.1/bin/.ansible-wrapped'...
/nix/store/dvjvnd39fd84wiakd2hcv9hhlc69jjcg-python2.7-ansible-2.7.1

@c0bw3b
Copy link
Contributor

c0bw3b commented Nov 1, 2018

@GrahamcOfBorg build ansible_2_6

@GrahamcOfBorg
Copy link

Success on x86_64-linux (full log)

Attempted: ansible_2_6

Partial log (click to expand)

wrapping `/nix/store/rfa6gwvl8dppcbf7vry0dvxm9q0xcvrj-python2.7-ansible-2.6.7/bin/.ansible-config-wrapped'...
wrapping `/nix/store/rfa6gwvl8dppcbf7vry0dvxm9q0xcvrj-python2.7-ansible-2.6.7/bin/.ansible-inventory-wrapped'...
wrapping `/nix/store/rfa6gwvl8dppcbf7vry0dvxm9q0xcvrj-python2.7-ansible-2.6.7/bin/.ansible-vault-wrapped'...
wrapping `/nix/store/rfa6gwvl8dppcbf7vry0dvxm9q0xcvrj-python2.7-ansible-2.6.7/bin/.ansible-playbook-wrapped'...
wrapping `/nix/store/rfa6gwvl8dppcbf7vry0dvxm9q0xcvrj-python2.7-ansible-2.6.7/bin/.ansible-connection-wrapped'...
wrapping `/nix/store/rfa6gwvl8dppcbf7vry0dvxm9q0xcvrj-python2.7-ansible-2.6.7/bin/.ansible-wrapped'...
wrapping `/nix/store/rfa6gwvl8dppcbf7vry0dvxm9q0xcvrj-python2.7-ansible-2.6.7/bin/.ansible-console-wrapped'...
wrapping `/nix/store/rfa6gwvl8dppcbf7vry0dvxm9q0xcvrj-python2.7-ansible-2.6.7/bin/.ansible-doc-wrapped'...
wrapping `/nix/store/rfa6gwvl8dppcbf7vry0dvxm9q0xcvrj-python2.7-ansible-2.6.7/bin/.ansible-pull-wrapped'...
/nix/store/rfa6gwvl8dppcbf7vry0dvxm9q0xcvrj-python2.7-ansible-2.6.7

@GrahamcOfBorg
Copy link

Success on aarch64-linux (full log)

Attempted: ansible_2_6

Partial log (click to expand)

wrapping `/nix/store/dkcy3y5vb4y3v1918y3iv6fv9ycbdn98-python2.7-ansible-2.6.7/bin/.ansible-pull-wrapped'...
wrapping `/nix/store/dkcy3y5vb4y3v1918y3iv6fv9ycbdn98-python2.7-ansible-2.6.7/bin/.ansible-playbook-wrapped'...
wrapping `/nix/store/dkcy3y5vb4y3v1918y3iv6fv9ycbdn98-python2.7-ansible-2.6.7/bin/.ansible-inventory-wrapped'...
wrapping `/nix/store/dkcy3y5vb4y3v1918y3iv6fv9ycbdn98-python2.7-ansible-2.6.7/bin/.ansible-galaxy-wrapped'...
wrapping `/nix/store/dkcy3y5vb4y3v1918y3iv6fv9ycbdn98-python2.7-ansible-2.6.7/bin/.ansible-doc-wrapped'...
wrapping `/nix/store/dkcy3y5vb4y3v1918y3iv6fv9ycbdn98-python2.7-ansible-2.6.7/bin/.ansible-console-wrapped'...
wrapping `/nix/store/dkcy3y5vb4y3v1918y3iv6fv9ycbdn98-python2.7-ansible-2.6.7/bin/.ansible-connection-wrapped'...
wrapping `/nix/store/dkcy3y5vb4y3v1918y3iv6fv9ycbdn98-python2.7-ansible-2.6.7/bin/.ansible-config-wrapped'...
wrapping `/nix/store/dkcy3y5vb4y3v1918y3iv6fv9ycbdn98-python2.7-ansible-2.6.7/bin/.ansible-wrapped'...
/nix/store/dkcy3y5vb4y3v1918y3iv6fv9ycbdn98-python2.7-ansible-2.6.7

@c0bw3b
Copy link
Contributor

c0bw3b commented Nov 1, 2018

@GrahamcOfBorg build ansible_2_5

@GrahamcOfBorg
Copy link

Success on aarch64-linux (full log)

Attempted: ansible_2_5

Partial log (click to expand)

wrapping `/nix/store/w0754jaq2zmnrmh22b3kwf7hvssk7l6d-python2.7-ansible-2.5.11/bin/.ansible-pull-wrapped'...
wrapping `/nix/store/w0754jaq2zmnrmh22b3kwf7hvssk7l6d-python2.7-ansible-2.5.11/bin/.ansible-playbook-wrapped'...
wrapping `/nix/store/w0754jaq2zmnrmh22b3kwf7hvssk7l6d-python2.7-ansible-2.5.11/bin/.ansible-inventory-wrapped'...
wrapping `/nix/store/w0754jaq2zmnrmh22b3kwf7hvssk7l6d-python2.7-ansible-2.5.11/bin/.ansible-galaxy-wrapped'...
wrapping `/nix/store/w0754jaq2zmnrmh22b3kwf7hvssk7l6d-python2.7-ansible-2.5.11/bin/.ansible-doc-wrapped'...
wrapping `/nix/store/w0754jaq2zmnrmh22b3kwf7hvssk7l6d-python2.7-ansible-2.5.11/bin/.ansible-console-wrapped'...
wrapping `/nix/store/w0754jaq2zmnrmh22b3kwf7hvssk7l6d-python2.7-ansible-2.5.11/bin/.ansible-connection-wrapped'...
wrapping `/nix/store/w0754jaq2zmnrmh22b3kwf7hvssk7l6d-python2.7-ansible-2.5.11/bin/.ansible-config-wrapped'...
wrapping `/nix/store/w0754jaq2zmnrmh22b3kwf7hvssk7l6d-python2.7-ansible-2.5.11/bin/.ansible-wrapped'...
/nix/store/w0754jaq2zmnrmh22b3kwf7hvssk7l6d-python2.7-ansible-2.5.11

@GrahamcOfBorg
Copy link

Success on x86_64-linux (full log)

Attempted: ansible_2_5

Partial log (click to expand)

wrapping `/nix/store/qkni6lggsp694w5wnngci12smr176sjg-python2.7-ansible-2.5.11/bin/.ansible-connection-wrapped'...
wrapping `/nix/store/qkni6lggsp694w5wnngci12smr176sjg-python2.7-ansible-2.5.11/bin/.ansible-galaxy-wrapped'...
wrapping `/nix/store/qkni6lggsp694w5wnngci12smr176sjg-python2.7-ansible-2.5.11/bin/.ansible-pull-wrapped'...
wrapping `/nix/store/qkni6lggsp694w5wnngci12smr176sjg-python2.7-ansible-2.5.11/bin/.ansible-vault-wrapped'...
wrapping `/nix/store/qkni6lggsp694w5wnngci12smr176sjg-python2.7-ansible-2.5.11/bin/.ansible-wrapped'...
wrapping `/nix/store/qkni6lggsp694w5wnngci12smr176sjg-python2.7-ansible-2.5.11/bin/.ansible-playbook-wrapped'...
wrapping `/nix/store/qkni6lggsp694w5wnngci12smr176sjg-python2.7-ansible-2.5.11/bin/.ansible-doc-wrapped'...
wrapping `/nix/store/qkni6lggsp694w5wnngci12smr176sjg-python2.7-ansible-2.5.11/bin/.ansible-console-wrapped'...
wrapping `/nix/store/qkni6lggsp694w5wnngci12smr176sjg-python2.7-ansible-2.5.11/bin/.ansible-inventory-wrapped'...
/nix/store/qkni6lggsp694w5wnngci12smr176sjg-python2.7-ansible-2.5.11

@sengaya
Copy link
Contributor Author

sengaya commented Nov 2, 2018

@c0bw3b thanks for the recommendations, I'll update the PR.

@c0bw3b
Copy link
Contributor

c0bw3b commented Nov 3, 2018

@GrahamcOfBorg build ansible_2_5 ansible_2_6 ansible_2_7

@GrahamcOfBorg
Copy link

Success on x86_64-linux (full log)

Attempted: ansible_2_5, ansible_2_6, ansible_2_7

Partial log (click to expand)

wrapping `/nix/store/3ff65yi1l0hbr27bz0av0fb1mkwn3kjk-python2.7-ansible-2.7.1/bin/.ansible-vault-wrapped'...
wrapping `/nix/store/3ff65yi1l0hbr27bz0av0fb1mkwn3kjk-python2.7-ansible-2.7.1/bin/.ansible-playbook-wrapped'...
wrapping `/nix/store/3ff65yi1l0hbr27bz0av0fb1mkwn3kjk-python2.7-ansible-2.7.1/bin/.ansible-connection-wrapped'...
wrapping `/nix/store/3ff65yi1l0hbr27bz0av0fb1mkwn3kjk-python2.7-ansible-2.7.1/bin/.ansible-wrapped'...
wrapping `/nix/store/3ff65yi1l0hbr27bz0av0fb1mkwn3kjk-python2.7-ansible-2.7.1/bin/.ansible-console-wrapped'...
wrapping `/nix/store/3ff65yi1l0hbr27bz0av0fb1mkwn3kjk-python2.7-ansible-2.7.1/bin/.ansible-doc-wrapped'...
wrapping `/nix/store/3ff65yi1l0hbr27bz0av0fb1mkwn3kjk-python2.7-ansible-2.7.1/bin/.ansible-pull-wrapped'...
/nix/store/qkni6lggsp694w5wnngci12smr176sjg-python2.7-ansible-2.5.11
/nix/store/rfa6gwvl8dppcbf7vry0dvxm9q0xcvrj-python2.7-ansible-2.6.7
/nix/store/3ff65yi1l0hbr27bz0av0fb1mkwn3kjk-python2.7-ansible-2.7.1

@c0bw3b
Copy link
Contributor

c0bw3b commented Nov 3, 2018

Security label because of CVE-2018-16837

@GrahamcOfBorg
Copy link

Success on aarch64-linux (full log)

Attempted: ansible_2_5, ansible_2_6, ansible_2_7

Partial log (click to expand)

wrapping `/nix/store/dvjvnd39fd84wiakd2hcv9hhlc69jjcg-python2.7-ansible-2.7.1/bin/.ansible-inventory-wrapped'...
wrapping `/nix/store/dvjvnd39fd84wiakd2hcv9hhlc69jjcg-python2.7-ansible-2.7.1/bin/.ansible-galaxy-wrapped'...
wrapping `/nix/store/dvjvnd39fd84wiakd2hcv9hhlc69jjcg-python2.7-ansible-2.7.1/bin/.ansible-doc-wrapped'...
wrapping `/nix/store/dvjvnd39fd84wiakd2hcv9hhlc69jjcg-python2.7-ansible-2.7.1/bin/.ansible-console-wrapped'...
wrapping `/nix/store/dvjvnd39fd84wiakd2hcv9hhlc69jjcg-python2.7-ansible-2.7.1/bin/.ansible-connection-wrapped'...
wrapping `/nix/store/dvjvnd39fd84wiakd2hcv9hhlc69jjcg-python2.7-ansible-2.7.1/bin/.ansible-config-wrapped'...
wrapping `/nix/store/dvjvnd39fd84wiakd2hcv9hhlc69jjcg-python2.7-ansible-2.7.1/bin/.ansible-wrapped'...
/nix/store/w0754jaq2zmnrmh22b3kwf7hvssk7l6d-python2.7-ansible-2.5.11
/nix/store/dkcy3y5vb4y3v1918y3iv6fv9ycbdn98-python2.7-ansible-2.6.7
/nix/store/dvjvnd39fd84wiakd2hcv9hhlc69jjcg-python2.7-ansible-2.7.1

@c0bw3b c0bw3b merged commit 78a6ee6 into NixOS:master Nov 3, 2018
@c0bw3b
Copy link
Contributor

c0bw3b commented Nov 3, 2018

Thank you @sengaya

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants