-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Add a scaleway group to be able to use module_defaults #10647
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
base: main
Are you sure you want to change the base?
Conversation
15e7bc4
to
7db1217
Compare
f1e8146
to
08dc0a9
Compare
08dc0a9
to
f293e59
Compare
cc @DenBeke @Lunik @Spredzy @abarbare @guillaume_ro_fr @pastral @remyleone |
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.
Thanks for your contribution!
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.
Thanks for your contribution!
@@ -22,6 +22,7 @@ | |||
extends_documentation_fragment: | |||
- community.general.scaleway | |||
- community.general.attributes | |||
- community.general.scaleway.actiongroup_scaleway | |||
|
|||
attributes: | |||
check_mode: |
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.
Can you add action_group:
/ version_added: 11.2.0
(resp. 11.3.0) in all these modules?
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.
I think that's done. But after trying to find a smart sed invocation, i did it manually, so better double check.
LGTMbut for the comments by @felixfontein |
f293e59
to
1bec7f5
Compare
1bec7f5
to
5470815
Compare
5470815
to
05b3578
Compare
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.
LGTM
SUMMARY
Add a scaleway group to be able to use module_defaults
Since all modules requires the use of a api_key and various others repeated arguments, being able to use module_defaults allow to write cleaner roles.
ISSUE TYPE
COMPONENT NAME
all scaleway