Skip to content

Retarget TestAccLustreInstance_withMaintenancePolicy to us-central1-c to mitigate stockouts - #18932

Open
minhvq0899 wants to merge 3 commits into
GoogleCloudPlatform:mainfrom
minhvq0899:patch-1
Open

Retarget TestAccLustreInstance_withMaintenancePolicy to us-central1-c to mitigate stockouts#18932
minhvq0899 wants to merge 3 commits into
GoogleCloudPlatform:mainfrom
minhvq0899:patch-1

Conversation

@minhvq0899

@minhvq0899 minhvq0899 commented Sep 9, 2026

Copy link
Copy Markdown

Update TestAccLustreInstance_withMaintenancePolicy in resource_lustre_instance_test.go to target us-central1-c instead of us-central1-a to alleviate recurring resource exhaustion stockouts in us-central1-a.

Fixes hashicorp/terraform-provider-google#29117

Reference: b/554298111

Release Note Template for Downstream PRs (will be copied)

See Write release notes for guidance.

Retarget TestAccLustreInstance_withMaintenancePolicy to us-central1-c to mitigate stockouts

@google-cla

google-cla Bot commented Sep 9, 2026

Copy link
Copy Markdown

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@modular-magician modular-magician added the awaiting-approval Pull requests that need reviewer's approval to run presubmit tests label Sep 9, 2026
@github-actions
github-actions Bot requested a review from SirGitsalot September 9, 2026 03:25
@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown

Googlers: For automatic test runs see go/terraform-auto-test-runs.

@SirGitsalot, 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.

@modular-magician modular-magician added service/lustre and removed awaiting-approval Pull requests that need reviewer's approval to run presubmit tests labels Sep 10, 2026
@modular-magician

modular-magician commented Sep 10, 2026

Copy link
Copy Markdown
Collaborator

Hi there, I'm the Modular magician. I've detected the following information about your changes for commit e64f5ea:

Diff report

Your PR generated the following diffs in downstream repositories:

Repository Diff Link Changes
google provider View Diff 1 file changed, 1 insertion(+), 1 deletion(-)
google-beta provider View Diff 1 file changed, 1 insertion(+), 1 deletion(-)

Test report

Analytics

Total Tests Passed Skipped Affected
8 7 0 1
Affected Service Packages
  • lustre

Learn how VCR tests work


Step 1: Replaying Mode

Action taken

Found 1 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit.

Click here to see the affected tests
  • TestAccLustreInstance_withMaintenancePolicy

View the replaying VCR build log


Step 2: Recording Mode

Recording Mode Replaying Rerun Test Name
❌ Error · Log - TestAccLustreInstance_withMaintenancePolicy

Caution

Issues requiring attention before PR completion

🔴 Initial Recording Failed: Some tests failed during the recording step. See the table above for details.

Please address these issues to complete your PR. If you believe these detections are incorrect or unrelated to your change, please raise the concern with your reviewer.

View the recording VCR build log or the debug logs folder for detailed results.

@minhvq0899, @SirGitsalot VCR tests complete for e64f5ea!

@SirGitsalot SirGitsalot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Error waiting for Creating Instance: Error code 8, message: resource exhausted: not enough resources available to fulfill the request in us-central1-c 😞

Because this test is not relying on any resources in us-central1 it may be possible to run it in an entirely different region.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Failing test(s): TestAccLustreInstance_withMaintenancePolicy

3 participants