Skip to content

Disable TestAccContainerNodePool_withHostMaintenancePolicy VCR failure (Stockout) - #17890

Merged
c2thorn merged 1 commit into
GoogleCloudPlatform:mainfrom
Capstan:fix-nodepool-stockout
Jul 27, 2026
Merged

Disable TestAccContainerNodePool_withHostMaintenancePolicy VCR failure (Stockout)#17890
c2thorn merged 1 commit into
GoogleCloudPlatform:mainfrom
Capstan:fix-nodepool-stockout

Conversation

@Capstan

@Capstan Capstan commented Jun 6, 2026

Copy link
Copy Markdown
Member

Mitigates hashicorp/terraform-provider-google#25476

Skips TestAccContainerNodePool_withHostMaintenancePolicy in CI.

When requesting maintenance_interval = "PERIODIC", Compute Engine returns persistent [GCE_STOCKOUT] scheduling denials across all available testing data centers (us-central1, us-east1, us-west1, and asia-east1). Investigation details and quota increase suggestions have been logged on the upstream issue.

Release Note Template for Downstream PRs (will be copied)


@modular-magician

modular-magician commented Jun 6, 2026

Copy link
Copy Markdown
Collaborator

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

Diff report

Your PR generated the following diffs in downstream repositories:

Repository Diff Link Changes
google-beta provider View Diff 1 file changed, 2 insertions(+), 2 deletions(-)

Test report

Analytics

Total Tests Passed Skipped Affected
294 277 15 2
Affected Service Packages
  • container

Learn how VCR tests work


Step 1: Replaying Mode

Action taken

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

Click here to see the affected tests
  • TestAccContainerCluster_withLoggingConfig
  • TestAccContainerNodePool_withHostMaintenancePolicy

View the replaying VCR build log


Step 2: Recording Mode

Recording Mode Replaying Rerun Test Name
❌ Error · Log - TestAccContainerCluster_withLoggingConfig
❌ Error · Log - TestAccContainerNodePool_withHostMaintenancePolicy

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.

@Capstan VCR tests complete for 61eeb00!

@Capstan
Capstan force-pushed the fix-nodepool-stockout branch from 61eeb00 to 7290223 Compare June 8, 2026 00:25
@modular-magician

modular-magician commented Jun 8, 2026

Copy link
Copy Markdown
Collaborator

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

Diff report

Your PR generated the following diffs in downstream repositories:

Repository Diff Link Changes
google-beta provider View Diff 1 file changed, 2 insertions(+), 2 deletions(-)

Test report

Analytics

Total Tests Passed Skipped Affected
294 277 15 2
Affected Service Packages
  • container

Learn how VCR tests work


Step 1: Replaying Mode

Action taken

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

Click here to see the affected tests
  • TestAccContainerCluster_withLoggingConfig
  • TestAccContainerNodePool_withHostMaintenancePolicy

View the replaying VCR build log


Step 2: Recording Mode

Recording Mode Replaying Rerun Test Name
❌ Error · Log - TestAccContainerCluster_withLoggingConfig
❌ Error · Log - TestAccContainerNodePool_withHostMaintenancePolicy

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.

@Capstan VCR tests complete for 7290223!

@Capstan
Capstan force-pushed the fix-nodepool-stockout branch from 7290223 to 009fc4b Compare June 8, 2026 14:04
@modular-magician

modular-magician commented Jun 8, 2026

Copy link
Copy Markdown
Collaborator

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

Diff report

Your PR generated the following diffs in downstream repositories:

Repository Diff Link Changes
google-beta provider View Diff 1 file changed, 2 insertions(+), 2 deletions(-)

Test report

Analytics

Total Tests Passed Skipped Affected
294 277 15 2
Affected Service Packages
  • container

Learn how VCR tests work


Step 1: Replaying Mode

Action taken

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

Click here to see the affected tests
  • TestAccContainerCluster_withLoggingConfig
  • TestAccContainerNodePool_withHostMaintenancePolicy

View the replaying VCR build log


Step 2: Recording Mode

Recording Mode Replaying Rerun Test Name
❌ Error · Log - TestAccContainerCluster_withLoggingConfig
❌ Error · Log - TestAccContainerNodePool_withHostMaintenancePolicy

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.

@Capstan VCR tests complete for 009fc4b!

@Capstan
Capstan force-pushed the fix-nodepool-stockout branch from 009fc4b to 613de42 Compare June 9, 2026 02:21
@modular-magician

modular-magician commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator

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

Diff report

Your PR generated the following diffs in downstream repositories:

Repository Diff Link Changes
google-beta provider View Diff 1 file changed, 2 insertions(+), 2 deletions(-)

Test report

Analytics

Total Tests Passed Skipped Affected
294 276 15 3
Affected Service Packages
  • container

Learn how VCR tests work


Step 1: Replaying Mode

Action taken

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

Click here to see the affected tests
  • TestAccContainerCluster_withAutoscalingProfile
  • TestAccContainerCluster_withLoggingConfig
  • TestAccContainerNodePool_withHostMaintenancePolicy

View the replaying VCR build log


Step 2: Recording Mode

Recording Mode Replaying Rerun Test Name
✅ Log TestAccContainerCluster_withAutoscalingProfile
❌ Error · Log - TestAccContainerCluster_withLoggingConfig
❌ Error · Log - TestAccContainerNodePool_withHostMaintenancePolicy

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.

@Capstan VCR tests complete for 613de42!

@Capstan
Capstan force-pushed the fix-nodepool-stockout branch from 613de42 to 448ba8a Compare June 10, 2026 06:07
@modular-magician

modular-magician commented Jun 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 448ba8a:

Diff report

Your PR generated the following diffs in downstream repositories:

Repository Diff Link Changes
google-beta provider View Diff 1 file changed, 2 insertions(+), 2 deletions(-)

Test report

Analytics

Total Tests Passed Skipped Affected
295 278 15 2
Affected Service Packages
  • container

Learn how VCR tests work


Step 1: Replaying Mode

Action taken

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

Click here to see the affected tests
  • TestAccContainerCluster_withLoggingConfig
  • TestAccContainerNodePool_withHostMaintenancePolicy

View the replaying VCR build log


Step 2: Recording Mode

Recording Mode Replaying Rerun Test Name
❌ Error · Log - TestAccContainerCluster_withLoggingConfig
❌ Error · Log - TestAccContainerNodePool_withHostMaintenancePolicy

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.

@Capstan VCR tests complete for 448ba8a!

@Capstan
Capstan force-pushed the fix-nodepool-stockout branch from 448ba8a to 4e8e155 Compare June 10, 2026 14:48
@modular-magician

modular-magician commented Jun 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 4e8e155:

Diff report

Your PR generated the following diffs in downstream repositories:

Repository Diff Link Changes
google-beta provider View Diff 1 file changed, 33 insertions(+), 9 deletions(-)

Test report

Analytics

Total Tests Passed Skipped Affected
295 278 15 2
Affected Service Packages
  • container

Learn how VCR tests work


Step 1: Replaying Mode

Action taken

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

Click here to see the affected tests
  • TestAccContainerCluster_withLoggingConfig
  • TestAccContainerNodePool_withHostMaintenancePolicy

View the replaying VCR build log

@Capstan
Capstan force-pushed the fix-nodepool-stockout branch from 4e8e155 to caa81d0 Compare June 11, 2026 05:23
@modular-magician

modular-magician commented Jun 11, 2026

Copy link
Copy Markdown
Collaborator

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

Diff report

Your PR generated the following diffs in downstream repositories:

Repository Diff Link Changes
google-beta provider View Diff 1 file changed, 33 insertions(+), 9 deletions(-)

Test report

Analytics

Total Tests Passed Skipped Affected
295 279 15 1
Affected Service Packages
  • container

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
  • TestAccContainerNodePool_withHostMaintenancePolicy

View the replaying VCR build log


Step 2: Recording Mode

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

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.

@Capstan VCR tests complete for caa81d0!

@Capstan
Capstan force-pushed the fix-nodepool-stockout branch from caa81d0 to 5cb20ab Compare June 11, 2026 07:19
@modular-magician

modular-magician commented Jun 11, 2026

Copy link
Copy Markdown
Collaborator

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

Diff report

Your PR generated the following diffs in downstream repositories:

Repository Diff Link Changes
google-beta provider View Diff 1 file changed, 31 insertions(+), 7 deletions(-)

Test report

Analytics

Total Tests Passed Skipped Affected
295 279 15 1
Affected Service Packages
  • container

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
  • TestAccContainerNodePool_withHostMaintenancePolicy

View the replaying VCR build log


Step 2: Recording Mode

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

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.

@Capstan VCR tests complete for 5cb20ab!

@modular-magician

modular-magician commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator

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

Diff report

Your PR generated the following diffs in downstream repositories:

Repository Diff Link Changes
google-beta provider View Diff 1 file changed, 32 insertions(+), 7 deletions(-)

Test report

Analytics

Total Tests Passed Skipped Affected
304 288 15 1
Affected Service Packages
  • container

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
  • TestAccContainerCluster_regionalWithNodePool

View the replaying VCR build log


Step 2: Recording Mode

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

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.

@Capstan VCR tests complete for 1aa3860!

@Capstan Capstan changed the title Fix TestAccContainerNodePool_withHostMaintenancePolicy VCR failure (Stockout) Disable TestAccContainerNodePool_withHostMaintenancePolicy VCR failure (Stockout) Jun 18, 2026
@Capstan
Capstan marked this pull request as ready for review June 18, 2026 14:01
@github-actions

Copy link
Copy Markdown

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

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

When requesting 'maintenance_interval = PERIODIC', Google Compute Engine rejects VM placement requests with [GCE_STOCKOUT] across all available data centers (Central, East, West, and Asia).

This indicates the shared CI test project (ci-test-project-188019) lacks the required scheduling allowlist grants or quota allocation for specialized GKE periodic maintenance host pools.

This commit skips the test in CI to resolve permanent verification blocks while retaining the test schema and isolated VPC networking for local verification.
@Capstan
Capstan force-pushed the fix-nodepool-stockout branch from 1aa3860 to e665dfb Compare June 18, 2026 14:05
@modular-magician

modular-magician commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator

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

Diff report

Your PR generated the following diffs in downstream repositories:

Repository Diff Link Changes
google-beta provider View Diff 1 file changed, 32 insertions(+), 7 deletions(-)

Test report

Analytics

Total Tests Passed Skipped Affected
304 288 15 1
Affected Service Packages
  • container

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
  • TestAccContainerCluster_regionalWithNodePool

View the replaying VCR build log


Step 2: Recording Mode

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

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.

@Capstan, @malhotrasagar2212 VCR tests complete for e665dfb!

@github-actions

Copy link
Copy Markdown

@malhotrasagar2212 This PR has been waiting for review for 3 weekdays. Please take a look! Use the label disable-review-reminders to disable these notifications.

@malhotrasagar2212 malhotrasagar2212 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@Capstan The test changes cannot be determined since the test is being skipped. Did you verify the test run locally to ensure the changes are correct and tests are passing?

@github-actions

github-actions Bot commented Jul 7, 2026

Copy link
Copy Markdown

@Capstan, this PR is waiting for action from you. If no action is taken, this PR will be closed in 28 days.

Please address any comments or change requests, or re-request review from a core reviewer if no action is required.

Image showing the re-request review button

This notification can be disabled with the disable-automatic-closure label.

@github-actions

Copy link
Copy Markdown

@Capstan, this PR is waiting for action from you. If no action is taken, this PR will be closed in 14 days.

Please address any comments or change requests, or re-request review from a core reviewer if no action is required.

Image showing the re-request review button

This notification can be disabled with the disable-automatic-closure label.

@c2thorn c2thorn 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.

the test is already failing in our TC environment do to Stockout.

I need this PR for a related pr #18397 so I'm going to subsume this PR and approve it

@c2thorn
c2thorn added this pull request to the merge queue Jul 27, 2026
Merged via the queue into GoogleCloudPlatform:main with commit c6d1371 Jul 27, 2026
26 of 28 checks passed
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.

4 participants