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: 2.9.12 -> 2.10.0; ansible_2_7: drop #95683
Conversation
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.
actions LGTM
However, it appears that ansible_2_10
no longer installs the "ansible" module, and instead just installs "ansible_base". Not sure if this is an upstream bug, but it breaks all of the other ansible packages
https://github.com/NixOS/nixpkgs/pull/95683
7 packages failed to build:
ansible-lint python27Packages.pytest-ansible python37Packages.ansible-kernel python37Packages.ansible-lint python37Packages.pytest-ansible python38Packages.ansible-kernel python38Packages.pytest-ansible
7 packages built:
ansible ansible_2_9 kargo python27Packages.ansible python37Packages.ansible python37Packages.ansible-runner python38Packages.ansible-runner
ansible 2.10 site-packages:
[nix-shell:/home/jon/.cache/nixpkgs-review/pr-95683]$ ls ./results/ansible/lib/python3.8/site-packages/ansible
ansible/ ansible_base-2.10.0.dist-info/ ansible_test/
This appears to have been intentional: ansible/ansible@2268a37#diff-2eeaed663bd0d25b7e608891384b7298 I would say, keep the 2.9 version in |
Updated. |
owner = "ansible"; | ||
repo = "ansible"; | ||
rev = "v${version}"; | ||
sha256 = "0c794k0cyl54807sh9in0l942ah6g6wlz5kf3qvy5lhd581zlgyb"; | ||
sha256 = "0k9rs5ajx0chaq0xr1cj4x7fr5n8kd4y856miss6k01iv2m7yx42"; | ||
}; |
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.
the hash needs to be reverted, still pointing to 2_10
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.
Whoops, fixed.
This version went EOL on 2020/06/18.
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.
diff LGTM
https://github.com/NixOS/nixpkgs/pull/95683
1 package built:
ansible
Motivation for this change
https://github.com/ansible/ansible/blob/v2.10.0/changelogs/CHANGELOG-v2.10.rst#v2-10-0
Dropping 2.7 as per https://access.redhat.com/support/policy/updates/ansible-engine. I thought we had time until v2.10.0 was released, as the EOL previously was given as:
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)