-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Update node pool tests to use bootstrapped network #9348
Update node pool tests to use bootstrapped network #9348
Conversation
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 ( 2 files changed, 376 insertions(+), 186 deletions(-)) |
Tests analyticsTotal tests: Action takenFound 40 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected testsTestAccContainerClusterDatasource_zonal|TestAccContainerNodePool_withLoggingVariantUpdates|TestAccContainerNodePool_withoutConfidentialBootDisk|TestAccContainerNodePool_withConfidentialBootDisk|TestAccContainerNodePool_withConfidentialNodes|TestAccContainerNodePool_withHostMaintenancePolicy|TestAccContainerNodePool_fastSocket|TestAccContainerNodePool_threadsPerCore|TestAccContainerNodePool_compactPlacement|TestAccContainerNodePool_gvnic|TestAccContainerNodePool_gcfsConfig|TestAccContainerNodePool_customPlacementPolicy|TestAccContainerNodePool_ephemeralStorageConfig|TestAccContainerNodePool_localNvmeSsdBlockConfig|TestAccContainerNodePool_EmptyGuestAccelerator|TestAccContainerNodePool_shieldedInstanceConfig|TestAccContainerNodePool_012_ConfigModeAttr|TestAccContainerNodePool_concurrent|TestAccContainerNodePool_ephemeralStorageLocalSsdConfig|TestAccContainerNodePool_regionalClusters|TestAccContainerNodePool_withNodeConfigScopeAlias|TestAccContainerNodePool_autoscaling|TestAccContainerNodePool_totalSize|TestAccContainerNodePool_resize|TestAccContainerNodePool_withManagement|TestAccContainerNodePool_regionalAutoscaling|TestAccContainerNodePool_withSoleTenantConfig|TestAccContainerNodePool_withGPU|TestAccContainerNodePool_withNodeConfig|TestAccContainerNodePool_version|TestAccContainerNodePool_withTaintsUpdate|TestAccContainerNodePool_withSandboxConfig|TestAccContainerNodePool_withReservationAffinity|TestAccContainerNodePool_withCgroupMode|TestAccContainerNodePool_withWorkloadIdentityConfig|TestAccContainerNodePool_withReservationAffinitySpecific|TestAccContainerNodePool_withLinuxNodeConfig|TestAccContainerNodePool_basicWithClusterId|TestAccContainerNodePool_basic|TestAccContainerClusterDatasource_regional |
Rerun these tests in REPLAYING mode to catch issues
|
Our network is located in Some other tests failed with |
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 ( 2 files changed, 382 insertions(+), 192 deletions(-)) |
Tests analyticsTotal tests: Action takenFound 7 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected testsTestAccContainerNodePool_withHostMaintenancePolicy|TestAccContainerNodePool_withConfidentialNodes|TestAccContainerNodePool_autoscaling|TestAccContainerNodePool_totalSize|TestAccContainerNodePool_withNodeConfigScopeAlias|TestAccContainerNodePool_withReservationAffinitySpecific|TestAccDataSourceGoogleServiceAccountIdToken_impersonation |
Running VCR skipped tests on TC: https://hashicorp.teamcity.com/buildConfiguration/TerraformProviders_GoogleBeta_MmUpstreamTesting_GOOGLEBETA_PACKAGE_CONTAINER/47970?hideTestsFromDependencies=false&hideProblemsFromDependencies=false Update: all tests passed on TC |
Rerun these tests in REPLAYING mode to catch issues
|
/gcbrun |
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 ( 2 files changed, 382 insertions(+), 192 deletions(-)) |
Tests analyticsTotal tests: 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 testsTestAccContainerNodePool_withHostMaintenancePolicy |
|
I can get |
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 ( 2 files changed, 382 insertions(+), 192 deletions(-)) |
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.
LGTM- I haven't verified all the argument positions within the config or anything, but scanned through and no surprises + all the ones I did check were correct.
Tests analyticsTotal tests: 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 testsTestAccBigQueryDataTable_bigtable |
Rerun these tests in REPLAYING mode to catch issues
|
Release Note Template for Downstream PRs (will be copied)