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.

Sign up for GitHub

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

Jump to bottom

Module defaults groups stable or not? #1759

Open
mariolenz opened this issue Aug 14, 2024 · 1 comment · May be fixed by #1893
Open

Module defaults groups stable or not? #1759

mariolenz opened this issue Aug 14, 2024 · 1 comment · May be fixed by #1893

Comments

@mariolenz
Copy link
Contributor

Module defaults states that:

Ansible 2.7 adds a preview-status feature to group together modules that share common sets of parameters.

It's not clear to me if it is still a preview-status feature or not. If it's not a preview feature anymore but stable meanwhile, I think we should simply drop this sentence. Who cares what has been a preview in 2.7 and is stable for quite some time?

I'm not sure what to do if it's not officially stable yet. Ping the core team?
@ansible-documentation-bot ansible-documentation-bot bot added the needs_triage Needs a first human triage before being processed. label Aug 14, 2024
@s-hertel
Copy link
Contributor

I think deleting the bit about preview status is fine. I left it there in 2.12 when adding the feature for collections because it needed feedback, which has trickled in slowly for a smattering of issues over a number of releases. I'd only call it "stable" very recently (2.16.8/2.17.1+) since there have been bugs.

The section on the legacy group names also needs an update since some collections do not define the legacy group. For example, group/os is a no-op because ansible-core defers to openstack.cloud.os, but the collection only defines a group openstack. New groups in collections have to use the fully qualified group name, so the correct usage is group/openstack.cloud.openstack.

@samccann samccann removed the needs_triage Needs a first human triage before being processed. label Aug 20, 2024
@mariolenz mariolenz linked a pull request Sep 15, 2024 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
Ansible Documentation
Status: 🆕 Triage
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update module default groups documentation mariolenz/ansible-documentation
3 participants
@mariolenz @s-hertel @samccann