-
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
Add support for enable_relay
field to advanced_datapath_observability_config
#9633
Conversation
Hello! I am a robot. It looks like you are a: Community Contributor @trodge, 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. |
Hi there, I'm the Modular magician. I've detected the following information about your changes: Breaking Change Detection FailedThe breaking change detector crashed during execution. This is usually due to the downstream provider(s) failing to compile. Please investigate or follow up with your reviewer. Diff reportYour PR generated some diffs in downstreams - here they are. Terraform GA: Diff ( 4 files changed, 73 insertions(+)) |
mmv1/third_party/terraform/services/container/resource_container_cluster.go.erb
Outdated
Show resolved
Hide resolved
Tests analyticsTotal tests: Click here to see the affected service packages
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 testsTestAccContainerCluster_withMonitoringConfigAdvancedDatapathObservabilityConfig |
|
Tests analyticsTotal tests: Click here to see the affected service packages
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 testsTestAccContainerCluster_withMonitoringConfigAdvancedDatapathObservabilityConfig |
|
Tests analyticsTotal tests: Click here to see the affected service packages
Action takenFound 2 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected testsTestAccContainerCluster_withMonitoringConfigAdvancedDatapathObservabilityConfigOld|TestAccContainerCluster_withMonitoringConfigAdvancedDatapathObservabilityConfig |
|
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, 169 insertions(+), 2 deletions(-)) |
Tests analyticsTotal tests: Click here to see the affected service packages
Action takenFound 2 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected testsTestAccContainerCluster_withMonitoringConfigAdvancedDatapathObservabilityConfig|TestAccContainerCluster_withMonitoringConfig |
Rerun these tests in REPLAYING mode to catch issues
|
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, 176 insertions(+), 7 deletions(-)) |
Tests analyticsTotal tests: Click here to see the affected service packages
Action takenFound 171 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_withEnablePrivateNodesToggle|TestAccContainerNodePool_withMultiNicNetworkConfig|TestAccContainerNodePool_withSoleTenantConfig|TestAccContainerNodePool_threadsPerCore|TestAccContainerCluster_autoprovisioningDefaultsUpgradeSettings|TestAccContainerNodePool_localNvmeSsdBlockConfig|TestAccContainerNodePool_ephemeralStorageConfig|TestAccContainerNodePool_withNodeConfigScopeAlias|TestAccContainerNodePool_withoutConfidentialBootDisk|TestAccContainerNodePool_withNetworkConfig|TestAccContainerNodePool_withConfidentialNodes|TestAccContainerNodePool_shieldedInstanceConfig|TestAccContainerNodePool_ephemeralStorageLocalSsdConfig|TestAccContainerNodePool_EmptyGuestAccelerator|TestAccContainerNodePool_customPlacementPolicy|TestAccContainerNodePool_fastSocket|TestAccContainerNodePool_regionalClusters|TestAccContainerNodePool_withManagement|TestAccContainerNodePool_regionalAutoscaling|TestAccContainerNodePool_autoscaling|TestAccContainerClusterDatasource_zonal|TestAccContainerNodePool_compactPlacement|TestAccContainerNodePool_enableQueuedProvisioning|TestAccContainerNodePool_gcfsConfig|TestAccContainerNodePool_withGPU|TestAccContainerNodePool_totalSize|TestAccContainerNodePool_resize|TestAccContainerNodePool_concurrent|TestAccContainerNodePool_012_ConfigModeAttr|TestAccContainerNodePool_version|TestAccContainerNodePool_gvnic|TestAccContainerNodePool_withCgroupMode|TestAccContainerNodePool_withConfidentialBootDisk|TestAccContainerNodePool_withLinuxNodeConfig|TestAccContainerNodePool_maxPodsPerNode|TestAccContainerCluster_additional_pod_ranges_config_on_create|TestAccContainerNodePool_nodeLocations|TestAccContainerCluster_withPrivateClusterConfigPrivateEndpointSubnetwork|TestAccContainerCluster_additional_pod_ranges_config_on_update|TestAccContainerCluster_withPrivateEndpointSubnetwork|TestAccContainerCluster_with_PodCIDROverprovisionDisabled|TestAccContainerCluster_autopilot_minimal|TestAccContainerCluster_stackType_withSingleStack|TestAccContainerCluster_stackType_withDualStack|TestAccContainerNodePool_withSandboxConfig|TestAccContainerNodePool_withTaintsUpdate|TestAccContainerCluster_withIPAllocationPolicy_existingSecondaryRanges|TestAccContainerCluster_withIPAllocationPolicy_specificIPRanges|TestAccContainerCluster_withIPAllocationPolicy_specificSizes|TestAccContainerNodePool_withWorkloadIdentityConfig|TestAccContainerNodePool_withMachineAndDiskUpdate|TestAccContainerNodePool_withReservationAffinitySpecific|TestAccContainerNodePool_basicWithClusterId|TestAccContainerNodePool_basic|TestAccContainerNodePool_withNodeConfig|TestAccContainerNodePool_withLoggingVariantUpdates|TestAccContainerCluster_withEnablePrivateEndpointToggle|TestAccContainerCluster_nodeAutoprovisioningDefaults|TestAccContainerCluster_withoutConfidentialBootDisk|TestAccContainerCluster_nodeAutoprovisioning|TestAccContainerNodePool_withReservationAffinity|TestAccContainerCluster_autopilot_net_admin|TestAccContainerCluster_customPlacementPolicy|TestAccContainerCluster_withLoggingVariantInNodePool|TestAccContainerCluster_withConfidentialBootDisk|TestAccContainerCluster_withConfidentialBootDiskNodeConfig|TestAccContainerCluster_withNodePoolConflictingNameFields|TestAccContainerCluster_deleteExclusionWindow|TestAccContainerCluster_deleteMaintenanceExclusionOptions|TestAccContainerCluster_updateMaintenanceExclusionOptions|TestAccContainerCluster_withWorkloadALTSConfig|TestAccContainerCluster_network|TestAccContainerCluster_withMaintenanceExclusionOptions|TestAccContainerCluster_withRecurringMaintenanceWindow|TestAccContainerCluster_withNodePoolNodeConfig|TestAccContainerCluster_withMaintenanceWindow|TestAccContainerCluster_withTPUConfig|TestAccContainerCluster_withMaintenanceExclusionWindow|TestAccContainerCluster_withNodePoolMultiple|TestAccContainerCluster_withNodePoolUpdateVersion|TestAccContainerCluster_withNodePoolAutoscaling|TestAccContainerCluster_backend|TestAccContainerCluster_withNodePoolCIA|TestAccContainerCluster_withSecurityPostureConfig|TestAccContainerCluster_withProtectConfig|TestAccContainerCluster_withNodePoolResize|TestAccContainerCluster_withNodePoolBasic|TestAccContainerCluster_withFleetConfig|TestAccContainerCluster_withReleaseChannelEnabled|TestAccContainerCluster_withNodeConfig|TestAccContainerCluster_withLoggingVariantInNodeConfig|TestAccContainerCluster_withSandboxConfig|TestAccContainerCluster_withWorkloadMetadataConfig|TestAccContainerCluster_withNodeConfigReservationAffinitySpecific|TestAccContainerCluster_withNodeConfigShieldedInstanceConfig|TestAccContainerCluster_withNodeConfigReservationAffinity|TestAccContainerCluster_withIPv4Error|TestAccContainerCluster_updateVersion|TestAccContainerCluster_withIncompatibleMasterVersionNodeVersion|TestAccContainerCluster_withGatewayApiConfig|TestAccContainerCluster_withMasterAuthorizedNetworksConfig|TestAccContainerCluster_withNodeConfigScopeAlias|TestAccContainerCluster_withPrivateClusterConfigMissingCidrBlock_withAutopilot|TestAccContainerCluster_withBootDiskKmsKey|TestAccContainerCluster_withDNSConfig|TestAccContainerCluster_withIntraNodeVisibility|TestAccContainerCluster_withVersion|TestAccContainerCluster_withNodePoolDefaults|TestAccContainerCluster_withPrivateClusterConfigGlobalAccessEnabledOnly|TestAccContainerCluster_regionalWithNodePool|TestAccContainerCluster_withMasterAuthorizedNetworksDisabled|TestAccContainerCluster_withLoggingVariantUpdates|TestAccContainerCluster_withGcpPublicCidrsAccessEnabledToggle|TestAccContainerCluster_regional|TestAccContainerCluster_withPrivateClusterConfigMissingCidrBlock|TestAccContainerCluster_withPrivateClusterConfigBasic|TestAccContainerCluster_withTelemetryEnabled|TestAccContainerCluster_withEnableKubernetesBetaAPIsOnExistingCluster|TestAccContainerCluster_withTpu|TestAccContainerCluster_withResourceUsageExportConfig|TestAccContainerCluster_withEnableKubernetesBetaAPIs|TestAccContainerCluster_errorNoClusterCreated|TestAccContainerCluster_withAdvancedDatapath|TestAccContainerCluster_withEnableKubernetesAlpha|TestAccContainerCluster_withCostManagementConfig|TestAccContainerCluster_withReleaseChannelEnabledDefaultVersion|TestAccContainerCluster_withMonitoringConfigAdvancedDatapathObservabilityConfig|TestAccContainerCluster_withExternalIpsConfig|TestAccContainerCluster_withMeshCertificatesConfig|TestAccContainerCluster_withDatabaseEncryption|TestAccContainerCluster_withAutopilotNetworkTags|TestAccContainerCluster_errorAutopilotLocation|TestAccContainerClusterCustomServiceAccount_withAutopilot|TestAccContainerCluster_withAutopilot|TestAccContainerCluster_withFlexiblePodCIDR|TestAccContainerCluster_regionalWithNodeLocations|TestAccContainerCluster_withMonitoringConfig|TestAccContainerCluster_autoprovisioningDefaultsManagement|TestAccContainerCluster_withSoleTenantGroup|TestAccContainerCluster_withIdentityServiceConfig|TestAccContainerCluster_nodeAutoprovisioningDefaultsShieldedInstance|TestAccContainerCluster_withLoggingConfig|TestAccContainerCluster_nodeAutoprovisioningDefaultsImageType|TestAccContainerCluster_withWorkloadIdentityConfig|TestAccContainerCluster_nodeAutoprovisioningDefaultsDiskSizeGb|TestAccContainerCluster_networkingModeRoutes|TestAccContainerCluster_nodeAutoprovisioningDefaultsDiskType|TestAccContainerCluster_withShieldedNodes|TestAccContainerCluster_withMultiNetworking|TestAccContainerCluster_withFQDNNetworkPolicy|TestAccContainerCluster_nodeAutoprovisioningDefaultsBootDiskKmsKey|TestAccContainerCluster_withBinaryAuthorizationEvaluationModeClassic|TestAccContainerCluster_withAutoscalingProfile|TestAccContainerCluster_withBinaryAuthorizationEvaluationModeAutopilot|TestAccContainerEngineVersions_filtered|TestAccContainerCluster_nodeAutoprovisioningNetworkTags|TestAccContainerCluster_withNotificationConfig|TestAccContainerEngineVersions_basic|TestAccContainerCluster_withNetworkPolicyEnabled|TestAccContainerCluster_withAuthenticatorGroupsConfig|TestAccContainerCluster_withBinaryAuthorizationEnabledBool|TestAccContainerCluster_withDeletionProtection|TestAccContainerCluster_withAddons|TestAccContainerCluster_withMasterAuthConfig_NoCert|TestAccContainerCluster_misc|TestAccContainerCluster_withILBSubsetting|TestAccContainerCluster_withConfidentialNodes|TestAccContainerCluster_withFilteredNotificationConfig|TestAccContainerCluster_basic|TestAccContainerClusterDatasource_regional |
|
…ity_config` Signed-off-by: Michal Siwinski <[email protected]>
Hi there, I'm the Modular magician. I've detected the following information about your changes: Breaking Change(s) DetectedThe following breaking change(s) were detected within your pull request.
If you believe this detection to be incorrect please raise the concern with your reviewer. Diff reportYour PR generated some diffs in downstreams - here they are. Terraform GA: Diff ( 4 files changed, 175 insertions(+), 8 deletions(-)) |
Tests analyticsTotal tests: Click here to see the affected service packages
|
I'm not sure if I understand what's the breaking change detected here. MinItems did not change, though MaxItems did. I believe change to MaxItems will not be breaking, as implementation only used the 0-index element of Therefore I think it's fine to merge it in the current state. |
@trodge can you take a look at this PR again? |
…ity_config` (GoogleCloudPlatform#9633) Signed-off-by: Michal Siwinski <[email protected]>
…ity_config` (GoogleCloudPlatform#9633) Signed-off-by: Michal Siwinski <[email protected]>
…ity_config` (GoogleCloudPlatform#9633) Signed-off-by: Michal Siwinski <[email protected]>
…ity_config` (GoogleCloudPlatform#9633) Signed-off-by: Michal Siwinski <[email protected]>
This field is an alternative to
relay_mode
field inadvanced_datapath_observability_config
to enable Relay.https://cloud.google.com/kubernetes-engine/docs/reference/rest/v1/projects.locations.clusters#Cluster.AdvancedDatapathObservabilityConfig
Release Note Template for Downstream PRs (will be copied)