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

Failing test(s): TestAccNetappVolume_netapp* #18909

Comments

@SarahFrench
Copy link
Member

SarahFrench commented Jul 30, 2024

Impacted tests

  • TestAccNetappVolume_netappVolumeBasicExample_update
  • TestAccNetappbackup_netappBackupExample
  • TestAccNetappbackup_netappBackupFull_update

Affected Resource(s)

  • google_netapp_backup_vault

Failure rates

  • 100% since June/July 2024 across the 3 tests

Message(s)

    vcr_utils.go:152: Step 11/16 error: Error running apply: exit status 1
        Error: Error waiting to create backupVault: Error waiting for Creating backupVault: Error code 3, message: bad request error: "Error creating backup vault - Creation of multiple backup vaults is not supported for a project within a region."
          with google_netapp_backup_vault.backup-vault,
          on terraform_plugin_test.tf line 37, in resource "google_netapp_backup_vault" "backup-vault":
          37: resource "google_netapp_backup_vault" "backup-vault" {

Same for all 3 named tests

Nightly build test history

b/357636823

@github-actions github-actions bot added forward/review In review; remove label to forward service/netapp labels Jul 30, 2024
@SarahFrench
Copy link
Member Author

The example used to generate TestAccNetappbackup_netappBackupExample uses us-central1, and this is the default region used in all tests.

We could make different tests use different regions.

@melinath melinath added size/xs and removed forward/review In review; remove label to forward labels Aug 5, 2024
@melinath melinath added this to the Near-Term Goals milestone Aug 5, 2024
@Mehul3217
Copy link

Can we please get the list of projects that are being used for above tests, looks like some projects requires cleaning up of resources.

@trodge
Copy link
Collaborator

trodge commented Aug 12, 2024

Any cleanup for these resources should be handled by automated sweepers. I agree with @SarahFrench that the most likely solution is going to be putting each test in its own region, unless we know that the sweepers are not working.

@melinath
Copy link
Collaborator

@Mehul3217 responded internally on b/357940045

@Mehul3217
Copy link

We have raised a PR for manual sweepers: GoogleCloudPlatform/magic-modules#11467

@ScottSuarez
Copy link
Collaborator

Doesn't appear to have been resolved with my change.. reopening.. The bug is also tracked internally by the service team.

@ScottSuarez ScottSuarez reopened this Oct 4, 2024
@Mehul3217
Copy link

It should get resolved now, we have cleaned up the required resources.

@SarahFrench
Copy link
Member Author

It looks like these two tests are racing to be the first to provision a backup vault in the region:

Screenshot 2024-10-07 at 10 13 48 Screenshot 2024-10-07 at 10 14 08

And this test is consistently failing

Screenshot 2024-10-07 at 10 15 09

All the failures are still Error creating backup vault - Creation of multiple backup vaults is not supported for a project within a region

@SarahFrench
Copy link
Member Author

SarahFrench commented Oct 7, 2024

Even if some resources needed cleaning up in the past, there will be a problem if tests running at the same time are trying to provision a backup vault in the same region and project. Either tests need to use different regions (have them hardcoded into the test) or we need to ensure the tests provisioning these resources cannot run in parallel.

@Mehul3217
Copy link

Mehul3217 commented Oct 8, 2024

Ack, working on it

created a PR: GoogleCloudPlatform/magic-modules#11939

@ScottSuarez
Copy link
Collaborator

@Mehul3217
Copy link

Ack, working on it
created a PR: GoogleCloudPlatform/magic-modules#11939

If you are using new regions not being sweeped could you add them to the sweeper please? https://github.com/modular-magician/terraform-provider-google/blob/c2cf67f225401493c6d11470be3d6df8747ae816/google/services/netapp/resource_netapp_backup_vault_sweeper.go#L41

Have not added any new locations

@melinath
Copy link
Collaborator

looks like these are now passing relatively consistently - marking this ticket as closed

Copy link

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 28, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.