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

Fix gcp_compute_snapshot module to work with customer-supplied encryption key #468

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

samyray
Copy link

@samyray samyray commented Nov 18, 2021

SUMMARY

Fixes missing field for a customer-supplied encryption key which prevents this module from creating a snapshot

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

Module gcp_compute_snapshot

ADDITIONAL INFORMATION

Before change:

TASK [gcp : create a snapshot] ***********************************************************************************************
Thursday 18 November 2021  12:23:39 -0500 (0:00:00.062)       0:00:13.518 *****
failed: [test-example-> 127.0.0.1] (item=test-example) => {"ansible_loop_var": "volume_create_item", "changed": false, "msg": "GCP returned error: {'error': {'code': 400, 'message': \"'projects/********/zones/********/disks/test-example' is protected with a customer supplied encryption key, but none was provided.\", 'errors': [{'message': \"'projects/********/zones/********/disks/test-example' is protected with a customer supplied encryption key, but none was provided.\", 'domain': 'global', 'reason': 'resourceIsEncryptedWithCustomerEncryptionKey'}]}}",}

After change:

TASK [gcpr : create a snapshot] ***********************************************************************************************
Thursday 18 November 2021  12:26:18 -0500 (0:00:00.066)       0:00:14.092 *****
changed: [test-example -> 127.0.0.1] => (item=test-example)

@arloginov
Copy link

Just want to check if there is any chance this pull request will be merged?
It is affecting our environment as well

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants