-
Notifications
You must be signed in to change notification settings - Fork 168
profile_* callbacks: avoid deprecated/deleted functions #650
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
profile_* callbacks: avoid deprecated/deleted functions #650
Conversation
Build failed.
|
Zuul is currently broken, so the failing tests there are not surprising (https://forum.ansible.com/t/43604). |
recheck |
Build succeeded. ✔️ ansible-galaxy-importer SUCCESS in 3m 14s |
Ping @saito-hideki this affects ansible-core 2.19 compatibility. |
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.
@felixfontein Thank you for your great PR. LGTM!
Build succeeded (gate pipeline). ✔️ ansible-galaxy-importer SUCCESS in 3m 14s |
c273ac2
into
ansible-collections:main
@felixfontein @Akasurde I will backport this pull request to stable-1 and stable-2 as well. |
@Akasurde @saito-hideki thanks for reviewing and merging! |
profile_* callbacks: avoid deprecated/deleted functions SUMMARY The profile_roles and profile_tasks callbacks define methods playbook_on_setup and playbook_on_stats which have been deleted/deprecated: playbook_on_stats has been deprecated, v2_playbook_on_stats should be used instead (that one has already been there for many years: ansible/ansible@ba0e532 was added in 2015). playbook_on_setup has been deleted (ansible/ansible@eec57ec), and its v2 variant was already deleted in 2017: ansible/ansible@59d5481 Ref: ansible-collections#635 ISSUE TYPE Bugfix Pull Request Feature Pull Request COMPONENT NAME profile_roles profile_tasks Reviewed-by: Abhijeet Kasurde Reviewed-by: Hideki Saito <saito@fgrep.org>
SUMMARY
The profile_roles and profile_tasks callbacks define methods
playbook_on_setup
andplaybook_on_stats
which have been deleted/deprecated:playbook_on_stats
has been deprecated,v2_playbook_on_stats
should be used instead (that one has already been there for many years: ansible/ansible@ba0e532 was added in 2015).playbook_on_setup
has been deleted (ansible/ansible@eec57ec), and its v2 variant was already deleted in 2017: ansible/ansible@59d5481Ref: #635
ISSUE TYPE
COMPONENT NAME
profile_roles
profile_tasks