-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Netapp backup vault #9687
Netapp backup vault #9687
Conversation
Hello! I am a robot. It looks like you are a: Community Contributor @ScottSuarez, a repository maintainer, has been assigned to review your changes. If you have not received review feedback within 2 business days, please leave a comment on this PR asking them to take a look. You can help make sure that review is quick by doing a self-review and by running impacted tests locally. |
Hi there, I'm the Modular magician. I've detected the following information about your changes: Diff reportYour PR generated some diffs in downstreams - here they are. Terraform GA: Diff ( 6 files changed, 930 insertions(+), 2 deletions(-)) |
Tests analyticsTotal tests: Action takenFound 3 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected testsTestAccDataprocClusterIamPolicy|TestAccNetappbackupVault_netappBackupVaultExample_update|TestAccNetappbackupVault_netappBackupVaultExample |
Rerun these tests in REPLAYING mode to catch issues
|
Okay, I just realized that the test harness will run resource_netapp_backup_vault_test.go and resource_netapp_backup_vault_generated_test.go in parallel, with different suffixes. That will not work, as the user is only allowed to create one backup vault per region. Therefore, the test sometimes passed and sometimes fails, due to timing issues. Currently investigating on how to tell the test harness to run the tests sequentially, or will have to use a different region for one of the tests. |
Lets use different regions! :D |
- disabled example test. Now only custom test ist running and doesn't generate region conflict - Small documentation fix
Got guidance from @shuyama1. My problem was that generated test and my custom test both created resources in same region and only one resource per region is allowed. Shuya recommended to disable generated test and just use a proper custom test. I disabled generated test and reverted custom test back to use us-central1. |
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.
Some questions and nits.
mmv1/third_party/terraform/services/netapp/resource_netapp_backup_vault_test.go
Show resolved
Hide resolved
Hi there, I'm the Modular magician. I've detected the following information about your changes: Diff reportYour PR generated some diffs in downstreams - here they are. Terraform GA: Diff ( 5 files changed, 862 insertions(+), 2 deletions(-)) |
Tests analyticsTotal tests: Click here to see the affected service packages
Action takenFound 1 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected testsTestAccNetappbackupVault_netappBackupVaultExample_update |
Rerun these tests in REPLAYING mode to catch issues
|
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 the work !
* netapp-backup-vault: First commit * Updated test file * make the tests use different regions * Revert testing back to us-central1 - disabled example test. Now only custom test ist running and doesn't generate region conflict - Small documentation fix * make lint-yaml happy * make lint-yaml happy2 * make yamlking happy3 * Move example above parameters
* netapp-backup-vault: First commit * Updated test file * make the tests use different regions * Revert testing back to us-central1 - disabled example test. Now only custom test ist running and doesn't generate region conflict - Small documentation fix * make lint-yaml happy * make lint-yaml happy2 * make yamlking happy3 * Move example above parameters
* netapp-backup-vault: First commit * Updated test file * make the tests use different regions * Revert testing back to us-central1 - disabled example test. Now only custom test ist running and doesn't generate region conflict - Small documentation fix * make lint-yaml happy * make lint-yaml happy2 * make yamlking happy3 * Move example above parameters
* netapp-backup-vault: First commit * Updated test file * make the tests use different regions * Revert testing back to us-central1 - disabled example test. Now only custom test ist running and doesn't generate region conflict - Small documentation fix * make lint-yaml happy * make lint-yaml happy2 * make yamlking happy3 * Move example above parameters
Fixes hashicorp/terraform-provider-google#16810
This PR is to support provisioning NetApp Backup Vaults for Google Cloud NetApp Volumes through GCP terraform provider.
Release Note Template for Downstream PRs (will be copied)