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

redhat_subscription: deprecate "pool" #6650

Conversation

ptoscano
Copy link
Contributor

@ptoscano ptoscano commented Jun 9, 2023

SUMMARY

The pool option is slower to use, and the regexp may expand to broader results than wanted. Because of that, deprecate it in favour of the pool_ids options (which is much better), slating it for removal in community.general 10.0.0.

ISSUE TYPE
  • Docs Pull Request
COMPONENT NAME

redhat_subscription

The "pool" option is slower to use, and the regexp may expand to broader
results than wanted. Because of that, deprecate it in favour of the
"pool_ids" options (which is much better), slating it for removal in
community.general 10.0.0.
@ansibullbot
Copy link
Collaborator

@ptoscano ptoscano force-pushed the redhat_subscription-deprecate-pool branch from ea76a86 to 648cb2d Compare June 9, 2023 06:42
@felixfontein felixfontein added check-before-release PR will be looked at again shortly before release and merged if possible. backport-7 labels Jun 9, 2023
@felixfontein felixfontein removed the check-before-release PR will be looked at again shortly before release and merged if possible. label Jun 9, 2023
@felixfontein felixfontein merged commit 61b8897 into ansible-collections:main Jun 9, 2023
@patchback
Copy link

patchback bot commented Jun 9, 2023

Backport to stable-7: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-7/61b889749e6cb955c75a773ce708eabc8765e1fe/pr-6650

Backported as #6657

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

@felixfontein
Copy link
Collaborator

@ptoscano thanks!

patchback bot pushed a commit that referenced this pull request Jun 9, 2023
The "pool" option is slower to use, and the regexp may expand to broader
results than wanted. Because of that, deprecate it in favour of the
"pool_ids" options (which is much better), slating it for removal in
community.general 10.0.0.

(cherry picked from commit 61b8897)
@ptoscano ptoscano deleted the redhat_subscription-deprecate-pool branch June 9, 2023 12:05
felixfontein pushed a commit that referenced this pull request Jun 9, 2023
… "pool" (#6657)

redhat_subscription: deprecate "pool" (#6650)

The "pool" option is slower to use, and the regexp may expand to broader
results than wanted. Because of that, deprecate it in favour of the
"pool_ids" options (which is much better), slating it for removal in
community.general 10.0.0.

(cherry picked from commit 61b8897)

Co-authored-by: Pino Toscano <ptoscano@redhat.com>
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.

3 participants