8000 profile_* callbacks: avoid deprecated/deleted functions by felixfontein · Pull Request #650 · ansible-collections/ansible.posix · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

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

Merged

Conversation

felixfontein
Copy link
Contributor
SUMMARY

The profile_roles and profile_tasks callbacks define methods playbook_on_setup and playbook_on_stats which have been deleted/deprecated:

Ref: #635

ISSUE TYPE
  • Bugfix Pull Request
  • Feature Pull Request
COMPONENT NAME

profile_roles
profile_tasks

Copy link
Contributor

Build failed.
https://ansible.softwarefactory-project.io/zuul/buildset/b1a492a961e843bf8491d73259446e44

⚠️ ansible-galaxy-importer SKIPPED Skipped due to failed job build-ansible-collection
build-ansible-collection POST_FAILURE in 5m 49s

@felixfontein
Copy link
Contributor Author

Zuul is currently broken, so the failing tests there are not surprising (https://forum.ansible.com/t/43604).

@felixfontein
Copy link
Contributor Author

recheck

Copy link
Contributor

@felixfontein
Copy link
Contributor Author

Ping @saito-hideki this affects ansible-core 2.19 compatibility.

Copy link
Collaborator
@saito-hideki saito-hideki left a 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!

@saito-hideki saito-hideki added the mergeit Gate PR in Zuul CI label Jul 8, 2025
@saito-hideki saito-hideki self-assigned this Jul 8, 2025
@saito-hideki saito-hideki added bug This issue/PR relates to a bug. feature This issue/PR relates to a feature request. labels Jul 8, 2025
Copy link
Contributor

Build succeeded (gate pipeline).
https://ansible.softwarefactory-project.io/zuul/buildset/96be5e3936434122a5b895927932192e

✔️ ansible-galaxy-importer SUCCESS in 3m 14s
✔️ build-ansible-collection SUCCESS in 5m 52s

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit c273ac2 into ansible-collections:main Jul 8, 2025
45 checks passed
@saito-hideki
Copy link
Collaborator

@felixfontein @Akasurde I will backport this pull request to stable-1 and stable-2 as well.
Thanks again!

@felixfontein felixfontein deleted the callback branch July 8, 2025 15:04
@felixfontein
Copy link
Contributor Author

@Akasurde @saito-hideki thanks for reviewing and merging!

saito-hideki pushed a commit to saito-hideki/ansible.posix that referenced this pull request Jul 9, 2025
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>
softwarefactory-project-zuul bot added a commit that referenced this pull request Jul 10, 2025
[Backport][stable-2] #650 #654

SUMMARY
Backporting #650 to stable-2

Ref: #635

Backporting #654 to stable-2
ISSUE TYPE

Bugfix Pull Request
Feature Pull Request
CI Pull Request

COMPONENT NAME

profile_roles
profile_tasks
CI
@saito-hideki saito-hideki mentioned this pull request Jul 10, 2025
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. feature This issue/PR relates to a feature request. mergeit Gate PR in Zuul CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0