-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
feat: add security_group to scaleway compute resource #45699
Conversation
The test
|
@abarbare a rebase is required here |
a4287ad
to
2622c4b
Compare
The test
|
2622c4b
to
1a2066e
Compare
Cc @sieben |
@@ -354,7 +376,7 @@ def present_strategy(compute_api, wished_server): | |||
if compute_api.module.check_mode: | |||
return changed, {"status": "Server %s attributes would be changed." % target_server["id"]} | |||
|
|||
server_change_attributes(compute_api=compute_api, target_server=target_server, wished_server=wished_server) | |||
target_server = server_change_attributes(compute_api=compute_api, target_server=target_server, wished_server=wished_server) |
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 would rather use another variable name, maybe created_server
? server_change_attributes
method should also use another variable name for the returned object.
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.
Actually, I can't modify only this variable as it is returned by the function.
The function will return target_server
for existing server or updated one. Changing only this variable will cause an undefined variable error on return.
61bd321
to
bade63e
Compare
8b7b5c3
to
e1b2506
Compare
Rebased to relauch CI. |
@abarbare another rebase is needed |
feat: add more tests
e1b2506
to
690654a
Compare
Rebased to relauch CI. |
shipit |
LGTM |
bot_status |
Componentslib/ansible/modules/cloud/scaleway/scaleway_compute.py test/legacy/roles/scaleway_compute/tasks/main.yml test/legacy/roles/scaleway_compute/tasks/security_group.yml Metadatawaiting_on: ansible |
feat: add more tests
SUMMARY
Let the scaleway user assigning a Security Group resource to a compute server
ISSUE TYPE
COMPONENT NAME
scaleway_compute
ANSIBLE VERSION