-
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
Support for GKE dual-stack clusters #7459
Support for GKE dual-stack clusters #7459
Conversation
Oops! It looks like you're using an unknown release-note type in your changelog entries:
Please only use the types listed in https://github.com/GoogleCloudPlatform/magic-modules/blob/master/.ci/RELEASE_NOTES_GUIDE.md. |
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 ( 4 files changed, 590 insertions(+), 27 deletions(-)) |
Tests analyticsTotal tests: Action takenFound 156 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected testsTestAccContainerNodePool_compactPlacement|TestAccContainerNodePool_gvnic|TestAccContainerCluster_withMaintenanceExclusionWindow|TestAccContainerNodePool_gcfsConfig|TestAccContainerNodePool_ephemeralStorageConfig|TestAccContainerNodePool_concurrent|TestAccContainerNodePool_shieldedInstanceConfig|TestAccContainerNodePool_EmptyGuestAccelerator|TestAccContainerNodePool_012_ConfigModeAttr|TestAccContainerNodePool_regionalClusters|TestAccContainerNodePool_resize|TestAccContainerNodePool_version|TestAccContainerNodePool_autoscaling|TestAccContainerNodePool_totalSize|TestAccContainerNodePool_regionalAutoscaling|TestAccContainerNodePool_withManagement|TestAccContainerNodePool_withNodeConfigScopeAlias|TestAccContainerNodePool_withGPU|TestAccContainerNodePool_withEnablePrivateNodesToggle|TestAccContainerNodePool_withUpgradeSettings|TestAccContainerNodePool_withNetworkConfig|TestAccContainerNodePool_withLinuxNodeConfig|TestAccContainerNodePool_withKubeletConfig|TestAccContainerNodePool_withWorkloadIdentityConfig|TestAccContainerNodePool_withSandboxConfig|TestAccContainerNodePool_withReservationAffinity|TestAccContainerNodePool_withNodeConfig|TestAccContainerNodePool_withLoggingVariantUpdates|TestAccContainerNodePool_withReservationAffinitySpecific|TestAccContainerNodePool_maxPodsPerNode|TestAccContainerNodePool_nodeLocations|TestAccContainerCluster_stackType_withDefaultStack|TestAccContainerNodePool_basicWithClusterId|TestAccContainerNodePool_basic|TestAccContainerCluster_stackType_withSingleStack|TestAccContainerCluster_withEnablePrivateEndpointToggle|TestAccContainerCluster_stackType_withDualStack|TestAccContainerCluster_withPrivateEndpointSubnetwork|TestAccContainerCluster_withIntraNodeVisibility|TestAccContainerCluster_withProtectConfig|TestAccContainerCluster_withPrivateClusterConfigMissingCidrBlock_withAutopilot|TestAccContainerCluster_withIPAllocationPolicy_specificSizes|TestAccContainerCluster_withIPAllocationPolicy_specificIPRanges|TestAccContainerCluster_deleteExclusionWindow|TestAccContainerCluster_withTPUConfig|TestAccContainerCluster_updateMaintenanceExclusionOptions|TestAccContainerCluster_withGatewayApiConfig|TestAccContainerCluster_withIPAllocationPolicy_existingSecondaryRanges|TestAccContainerCluster_deleteMaintenanceExclusionOptions|TestAccContainerCluster_withDNSConfig|TestAccContainerCluster_withMaintenanceExclusionOptions|TestAccContainerCluster_withEnableKubernetesAlpha|TestAccContainerCluster_regionalWithNodeLocations|TestAccContainerCluster_withPrivateClusterConfigBasic|TestAccContainerCluster_withTpu|TestAccContainerCluster_withAdvancedDatapath|TestAccContainerCluster_withMasterAuthorizedNetworksDisabled|TestAccContainerCluster_withResourceUsageExportConfig|TestAccContainerCluster_withRecurringMaintenanceWindow|TestAccContainerCluster_withCostManagementConfig|TestAccContainerCluster_regionalWithNodePool|TestAccContainerCluster_regional|TestAccContainerCluster_withDatabaseEncryption|TestAccContainerCluster_withMaintenanceWindow|TestAccContainerCluster_withNodePoolNodeConfig|TestAccContainerCluster_withMeshCertificatesConfig|TestAccContainerCluster_withGcpPublicCidrsAccessEnabledToggle|TestAccContainerCluster_withExternalIpsConfig|TestAccContainerCluster_withMasterAuthorizedNetworksConfig|TestAccContainerCluster_withTelemetryEnabled|TestAccContainerCluster_withNodePoolMultiple|TestAccContainerCluster_withReleaseChannelEnabled|TestAccContainerCluster_withNodePoolAutoscaling|TestAccContainerCluster_errorNoClusterCreated|TestAccContainerCluster_withReleaseChannelEnabledDefaultVersion|TestAccContainerCluster_withNodePoolResize|TestAccContainerCluster_withNodePoolCIA|TestAccContainerCluster_withNodePoolBasic|TestAccContainerCluster_withNodePoolUpdateVersion|TestAccContainerCluster_backend|TestAccContainerCluster_network|TestAccContainerCluster_withSandboxConfig|TestAccContainerCluster_withWorkloadMetadataConfig|TestAccContainerCluster_errorCleanDanglingCluster|TestAccContainerCluster_withNetworkPolicyEnabled|TestAccContainerCluster_withAuthenticatorGroupsConfig|TestAccContainerCluster_withMasterAuthConfig_NoCert|TestAccContainerCluster_withNodeConfigReservationAffinity|TestAccContainerCluster_withNodeConfigShieldedInstanceConfig|TestAccContainerCluster_autoprovisioningDefaultsManagement|TestAccContainerCluster_withNodeConfigScopeAlias|TestAccContainerCluster_withILBSubsetting|TestAccContainerCluster_withNodePoolDefaults|TestAccContainerCluster_nodeAutoprovisioningDefaultsShieldedInstance|TestAccContainerCluster_withLoggingVariantUpdates|TestAccContainerCluster_nodeAutoprovisioningDefaultsImageType|TestAccContainerCluster_withConfidentialNodes|TestAccContainerCluster_withLoggingVariantInNodePool|TestAccContainerCluster_withLoggingVariantInNodeConfig|TestAccContainerCluster_withNodeConfigReservationAffinitySpecific|TestAccContainerCluster_withNodeConfig|TestAccContainerCluster_withFilteredNotificationConfig|TestAccContainerCluster_withNotificationConfig|TestAccContainerCluster_nodeAutoprovisioningDefaultsDiskType|TestAccContainerCluster_misc|TestAccContainerCluster_withAddons|TestAccContainerCluster_networkingModeRoutes|TestAccContainerCluster_updateVersion|TestAccContainerCluster_nodeAutoprovisioningDefaultsDiskSizeGb|TestAccContainerCluster_withVersion|TestAccContainerCluster_withBinaryAuthorizationEvaluationModeClassic|TestAccContainerCluster_withFlexiblePodCIDR|TestAccContainerCluster_basic|TestAccContainerCluster_withBinaryAuthorizationEvaluationModeAutopilot|TestAccContainerCluster_withBinaryAuthorizationEnabledBoolLegacy|TestAccContainerCluster_withBinaryAuthorizationEnabledBool|TestAccContainerCluster_withBootDiskKmsKey|TestAccContainerCluster_withAutoscalingProfile|TestAccContainerCluster_nodeAutoprovisioningDefaultsBootDiskKmsKey|TestAccContainerCluster_withSoleTenantGroup|TestAccContainerCluster_withMonitoringConfig|TestAccContainerCluster_withLoggingConfig|TestAccContainerCluster_withIdentityServiceConfig|TestAccContainerCluster_withWorkloadIdentityConfig|TestAccContainerCluster_withAutopilotNetworkTags|TestAccContainerCluster_errorAutopilotLocation|TestAccContainerCluster_withShieldedNodes|TestAccContainerCluster_withAutopilot|TestAccContainerClusterCustomServiceAccount_withAutopilot|TestAccContainerCluster_nodeAutoprovisioningNetworkTags|TestAccContainerCluster_autoprovisioningDefaultsUpgradeSettings|TestAccContainerCluster_nodeAutoprovisioningDefaults|TestAccContainerCluster_nodeAutoprovisioning|TestAccComputeNetworkPeeringRoutesConfig_networkPeeringRoutesConfigGkeExample|TestAccGKEHubMembershipIamBindingGenerated|TestAccGKEBackupBackupPlanIamPolicyGenerated|TestAccGKEBackupBackupPlanIamMemberGenerated|TestAccGKEBackupBackupPlanIamBindingGenerated|TestAccGKEHubMembershipIamPolicyGenerated|TestAccGKEHubMembershipIamMemberGenerated|TestAccFirebaserulesRelease_BasicRelease|TestAccContainerClusterDatasource_regional|TestAccContainerClusterDatasource_zonal|TestAccDataSourceDnsManagedZone_basic|TestAccDNSResponsePolicy_update|TestAccDNSResponsePolicy_dnsResponsePolicyBasicExample|TestAccDNSManagedZone_privateUpdate|TestAccGKEBackupBackupPlan_gkebackupBackupplanCmekExample|TestAccWorkstationsWorkstationConfig_workstationConfigEncryptionKeyExample|TestAccDNSManagedZone_dnsManagedZonePrivateGkeExample|TestAccGKEBackupBackupPlan_gkebackupBackupplanAutopilotExample|TestAccGKEBackupBackupPlan_gkebackupBackupplanBasicExample|TestAccGKEBackupBackupPlan_update|TestAccGKEHubMembership_gkehubMembershipBasicExample|TestAccGKEBackupBackupPlan_gkebackupBackupplanFullExample|TestAccGKEHubMembership_gkehubMembershipIssuerExample |
Tests passed during RECORDING mode: Tests failed during RECORDING mode: Please fix these to complete your PR |
All failed tests seem to be caused by a quota issue? Error: googleapi: Error 403: Insufficient project quota to satisfy request: resource "FIREWALLS": request requires '3.0' and is short '3.0'. project has a quota of '200.0' with '0.0' available. View and manage quotas at https://console.cloud.google.com/iam-admin/quotas?usage=USED&project=ci-test-project-188019., forbidden |
Would you mind resolving the conflicts first? I'll kick off another run. |
Oops! It looks like you're using an unknown release-note type in your changelog entries:
Please only use the types listed in https://github.com/GoogleCloudPlatform/magic-modules/blob/master/.ci/RELEASE_NOTES_GUIDE.md. |
Merge Conflict 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 ( 4 files changed, 279 insertions(+), 32 deletions(-)) |
Tests analyticsTotal tests: Action takenFound 47 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected testsTestAccGKEBackupBackupPlan_gkebackupBackupplanBasicExample|TestAccContainerCluster_withIdentityServiceConfig|TestAccContainerCluster_withWorkloadIdentityConfig|TestAccContainerCluster_withAutopilotNetworkTags|TestAccContainerNodePool_localNvmeSsdBlockConfig|TestAccContainerCluster_withShieldedNodes|TestAccContainerCluster_nodeAutoprovisioningNetworkTags|TestAccContainerCluster_autoprovisioningDefaultsUpgradeSettings|TestAccContainerCluster_nodeAutoprovisioningDefaults|TestAccContainerClusterCustomServiceAccount_withAutopilot|TestAccContainerCluster_withVersion|TestAccContainerCluster_misc|TestAccContainerCluster_networkingModeRoutes|TestAccContainerCluster_basic|TestAccContainerCluster_nodeAutoprovisioning|TestAccContainerCluster_withSandboxConfig|TestAccContainerCluster_nodeAutoprovisioningDefaultsImageType|TestAccContainerCluster_nodeAutoprovisioningDefaultsDiskType|TestAccContainerCluster_withNodeConfigReservationAffinity|TestAccContainerCluster_withBootDiskKmsKey|TestAccContainerCluster_nodeAutoprovisioningDefaultsDiskSizeGb|TestAccContainerCluster_withBinaryAuthorizationEvaluationModeClassic|TestAccContainerCluster_withNodeConfigReservationAffinitySpecific|TestAccContainerCluster_withNodePoolDefaults|TestAccContainerCluster_withLoggingVariantUpdates|TestAccContainerCluster_withBinaryAuthorizationEvaluationModeAutopilot|TestAccContainerCluster_nodeAutoprovisioningDefaultsBootDiskKmsKey|TestAccContainerCluster_withLoggingVariantInNodePool|TestAccContainerCluster_withBinaryAuthorizationEnabledBoolLegacy|TestAccContainerCluster_withBinaryAuthorizationEnabledBool|TestAccContainerCluster_withAutoscalingProfile|TestAccContainerCluster_withLoggingVariantInNodeConfig|TestAccContainerCluster_withSoleTenantGroup|TestAccContainerCluster_withNodeConfig|TestAccContainerCluster_withMonitoringConfig|TestAccContainerCluster_updateVersion|TestAccContainerCluster_withLoggingConfig|TestAccContainerCluster_withILBSubsetting|TestAccContainerCluster_withConfidentialNodes|TestAccContainerCluster_withFilteredNotificationConfig|TestAccContainerCluster_withNotificationConfig|TestAccContainerCluster_withAddons|TestAccGKEBackupBackupPlanIamPolicyGenerated|TestAccGKEBackupBackupPlanIamMemberGenerated|TestAccGKEHubMembershipIamBindingGenerated|TestAccGKEBackupBackupPlanIamBindingGenerated|TestAccDataSourceDnsManagedZone_basic |
Tests passed during RECORDING mode: Tests failed during RECORDING mode: Several tests got terminated during RECORDING mode. |
/gcbrun |
Oops! It looks like you're using an unknown release-note type in your changelog entries:
Please only use the types listed in https://github.com/GoogleCloudPlatform/magic-modules/blob/master/.ci/RELEASE_NOTES_GUIDE.md. |
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 ( 4 files changed, 279 insertions(+), 32 deletions(-)) |
Tests analyticsTotal tests: Action takenFound 15 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected testsTestAccContainerCluster_withAutopilotNetworkTags|TestAccContainerCluster_nodeAutoprovisioningNetworkTags|TestAccContainerClusterCustomServiceAccount_withAutopilot|TestAccContainerCluster_updateVersion|TestAccContainerCluster_nodeAutoprovisioningDefaultsDiskSizeGb|TestAccContainerCluster_withSoleTenantGroup|TestAccContainerCluster_withMonitoringConfig|TestAccContainerCluster_withILBSubsetting|TestAccContainerCluster_withLoggingConfig|TestAccContainerCluster_withConfidentialNodes|TestAccContainerCluster_withFilteredNotificationConfig|TestAccContainerCluster_withNodeConfigReservationAffinity|TestAccContainerCluster_misc|TestAccDataSourceDnsManagedZone_basic|TestAccComposerEnvironment_withEncryptionConfigComposer1 |
Tests passed during RECORDING mode: Tests failed during RECORDING mode: Please fix these to complete your PR |
Oops! It looks like you're using an unknown release-note type in your changelog entries:
Please only use the types listed in https://github.com/GoogleCloudPlatform/magic-modules/blob/master/.ci/RELEASE_NOTES_GUIDE.md. |
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 ( 3 files changed, 252 insertions(+), 9 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 testsTestAccContainerClusterCustomServiceAccount_withAutopilot|TestAccComposerEnvironment_withEncryptionConfigComposer2|TestAccDataSourceDnsManagedZone_basic |
Tests passed during RECORDING mode: Tests failed during RECORDING mode: Please fix these to complete your PR |
Only one test case failed, but it dose not seem related to the PR |
mmv1/third_party/terraform/tests/resource_container_cluster_test.go.erb
Outdated
Show resolved
Hide resolved
mmv1/third_party/terraform/tests/resource_container_cluster_test.go.erb
Outdated
Show resolved
Hide resolved
mmv1/third_party/terraform/tests/resource_container_cluster_test.go.erb
Outdated
Show resolved
Hide resolved
mmv1/third_party/terraform/tests/resource_container_cluster_test.go.erb
Outdated
Show resolved
Hide resolved
mmv1/third_party/terraform/resources/resource_container_cluster.go.erb
Outdated
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 ( 4 files changed, 256 insertions(+), 9 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 testsTestAccNetworkServicesGateway_update|TestAccComposerEnvironment_withEncryptionConfigComposer2|TestAccDataSourceDnsManagedZone_basic |
Tests passed during RECORDING mode: Tests failed during RECORDING mode: Please fix these to complete your PR |
mmv1/third_party/terraform/resources/resource_container_cluster.go.erb
Outdated
Show resolved
Hide resolved
mmv1/third_party/terraform/website/docs/r/container_cluster.html.markdown
Outdated
Show resolved
Hide resolved
mmv1/third_party/terraform/tests/resource_container_cluster_test.go.erb
Outdated
Show resolved
Hide resolved
mmv1/third_party/terraform/resources/resource_container_cluster.go.erb
Outdated
Show resolved
Hide resolved
Plus, I assume the go.sum modification is no longer needed(?), since we bumped Go API version to 0.114 in a recent PR. |
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 ( 4 files changed, 155 insertions(+), 5 deletions(-)) |
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 ( 3 files changed, 150 insertions(+), 4 deletions(-)) |
Tests analyticsTotal tests: Action takenFound 4 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected testsTestAccFirebaserulesRelease_BasicRelease|TestAccNetworkServicesGateway_update|TestAccComposerEnvironment_withEncryptionConfigComposer2|TestAccDataSourceDnsManagedZone_basic |
Tests passed during RECORDING mode: Tests failed during RECORDING mode: Please fix these to complete your PR |
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.
Thank you!
Support for GKE dual-stack clusters. Bump Go API version to 0.112
hashicorp/terraform-provider-google#13842
https://cloud.google.com/kubernetes-engine/docs/reference/rest/v1beta1/projects.locations.clusters#stacktype
If this PR is for Terraform, I acknowledge that I have:
make test
andmake lint
in the generated providers to ensure it passes unit and linter tests.Release Note Template for Downstream PRs (will be copied)