[aks-preview] Extend OTLP live tests with gRPC port coverage for logs and metrics

Adds --opentelemetry-*-port-grpc flags and grpcPort assertions to the live
OpenTelemetry integration scenarios so gRPC is exercised symmetrically for both
the metrics and logs/traces signals (previously only metrics-create asserted
grpcPort). The comprehensive monitoring test also now asserts that disabling
OpenTelemetry metrics/logs clears both httpPort and grpcPort.

Author: Christoph Butz

src/aks-preview/azext_aks_preview/tests/latest/test_aks_commands.py

@@ -16343,7 +16343,7 @@ def test_aks_create_with_azuremonitorlogs_and_opentelemetry(self, resource_group
 
         create_cmd = (
             'aks create --resource-group={resource_group} --name={name} --location={location} --ssh-key-value={ssh_key_value} --node-vm-size={node_vm_size} '
-            '--enable-managed-identity --enable-azure-monitor-logs --enable-opentelemetry-logs-traces --opentelemetry-logs-traces-port-http=8080 '
+            '--enable-managed-identity --enable-azure-monitor-logs --enable-opentelemetry-logs-traces --opentelemetry-logs-traces-port-http=8080 --opentelemetry-logs-traces-port-grpc=8082 '
             '--aks-custom-headers AKSHTTPCustomFeatures=Microsoft.ContainerService/AzureMonitorAppMonitoringPreview --output=json'
         )
         self.cmd(create_cmd, checks=[
@@ -16353,6 +16353,7 @@ def test_aks_create_with_azuremonitorlogs_and_opentelemetry(self, resource_group
             self.check('addonProfiles.omsagent.config.useAADAuth', 'true'),
             self.check('azureMonitorProfile.appMonitoring.openTelemetryLogsAndTraces.enabled', True),
             self.check('azureMonitorProfile.appMonitoring.openTelemetryLogsAndTraces.httpPort', 8080),
+            self.check('azureMonitorProfile.appMonitoring.openTelemetryLogsAndTraces.grpcPort', 8082),
         ])
 
         # delete
@@ -16385,7 +16386,7 @@ def test_aks_update_with_azuremonitorlogs_and_opentelemetry(self, resource_group
         # update: enable-azure-monitor-logs with OpenTelemetry logs
         update_cmd = (
             'aks update --resource-group={resource_group} --name={name} --yes --output=json '
-            '--enable-azure-monitor-logs --enable-opentelemetry-logs-traces --opentelemetry-logs-traces-port-http=9090 '
+            '--enable-azure-monitor-logs --enable-opentelemetry-logs-traces --opentelemetry-logs-traces-port-http=9090 --opentelemetry-logs-traces-port-grpc=9092 '
             '--aks-custom-headers AKSHTTPCustomFeatures=Microsoft.ContainerService/AzureMonitorAppMonitoringPreview'
         )
         self.cmd(update_cmd, checks=[
@@ -16395,6 +16396,7 @@ def test_aks_update_with_azuremonitorlogs_and_opentelemetry(self, resource_group
             self.check('addonProfiles.omsagent.config.useAADAuth', 'true'),
             self.check('azureMonitorProfile.appMonitoring.openTelemetryLogsAndTraces.enabled', True),
             self.check('azureMonitorProfile.appMonitoring.openTelemetryLogsAndTraces.httpPort', 9090),
+            self.check('azureMonitorProfile.appMonitoring.openTelemetryLogsAndTraces.grpcPort', 9092),
         ])
 
         # update: disable OpenTelemetry logs but keep Azure Monitor logs
@@ -16940,8 +16942,8 @@ def test_aks_comprehensive_monitoring_integration(self, resource_group, resource
         create_cmd = (
             'aks create --resource-group={resource_group} --name={name} --location={location} --ssh-key-value={ssh_key_value} --node-vm-size={node_vm_size} '
             '--enable-managed-identity --enable-azure-monitor-logs --enable-azure-monitor-metrics --enable-azure-monitor-app-monitoring '
-            '--enable-opentelemetry-logs-traces --opentelemetry-logs-traces-port-http=8080 '
-            '--enable-opentelemetry-metrics --opentelemetry-metrics-port-http=8081 '
+            '--enable-opentelemetry-logs-traces --opentelemetry-logs-traces-port-http=8080 --opentelemetry-logs-traces-port-grpc=8082 '
+            '--enable-opentelemetry-metrics --opentelemetry-metrics-port-http=8081 --opentelemetry-metrics-port-grpc=8083 '
             '--enable-windows-recording-rules '
             '--aks-custom-headers AKSHTTPCustomFeatures=Microsoft.ContainerService/AzureMonitorAppMonitoringPreview --output=json'
         )
@@ -16974,9 +16976,11 @@ def test_aks_comprehensive_monitoring_integration(self, resource_group, resource
             # OpenTelemetry logs checks
             self.check('azureMonitorProfile.appMonitoring.openTelemetryLogsAndTraces.enabled', True),
             self.check('azureMonitorProfile.appMonitoring.openTelemetryLogsAndTraces.httpPort', 8080),
+            self.check('azureMonitorProfile.appMonitoring.openTelemetryLogsAndTraces.grpcPort', 8082),
             # OpenTelemetry metrics checks
             self.check('azureMonitorProfile.appMonitoring.openTelemetryMetrics.enabled', True),
             self.check('azureMonitorProfile.appMonitoring.openTelemetryMetrics.httpPort', 8081),
+            self.check('azureMonitorProfile.appMonitoring.openTelemetryMetrics.grpcPort', 8083),
         ])
 
         # Phase 2: Update - disable only OpenTelemetry logs (keep everything else)
@@ -17007,6 +17011,9 @@ def test_aks_comprehensive_monitoring_integration(self, resource_group, resource
             self.check('azureMonitorProfile.appMonitoring.autoInstrumentation.enabled', True),
             # OpenTelemetry logs should be disabled
             self.check('azureMonitorProfile.appMonitoring.openTelemetryLogsAndTraces.enabled', False),
+            # Disabling OpenTelemetry logs should clear both ports
+            self.check('azureMonitorProfile.appMonitoring.openTelemetryLogsAndTraces.httpPort', None),
+            self.check('azureMonitorProfile.appMonitoring.openTelemetryLogsAndTraces.grpcPort', None),
             # OpenTelemetry metrics should still be enabled
             self.check('azureMonitorProfile.appMonitoring.openTelemetryMetrics.enabled', True),
         ])
@@ -17039,13 +17046,16 @@ def test_aks_comprehensive_monitoring_integration(self, resource_group, resource
             self.check('azureMonitorProfile.appMonitoring.autoInstrumentation.enabled', True),
             # OpenTelemetry metrics should be disabled
             self.check('azureMonitorProfile.appMonitoring.openTelemetryMetrics.enabled', False),
+            # Disabling OpenTelemetry metrics should clear both ports
+            self.check('azureMonitorProfile.appMonitoring.openTelemetryMetrics.httpPort', None),
+            self.check('azureMonitorProfile.appMonitoring.openTelemetryMetrics.grpcPort', None),
         ])
 
         # Phase 4: Update - re-enable all OpenTelemetry features with different ports
         update_cmd = (
             'aks update --resource-group={resource_group} --name={name} --yes --output=json '
-            '--enable-opentelemetry-logs-traces --opentelemetry-logs-traces-port-http=9090 '
-            '--enable-opentelemetry-metrics --opentelemetry-metrics-port-http=9091 '
+            '--enable-opentelemetry-logs-traces --opentelemetry-logs-traces-port-http=9090 --opentelemetry-logs-traces-port-grpc=9092 '
+            '--enable-opentelemetry-metrics --opentelemetry-metrics-port-http=9091 --opentelemetry-metrics-port-grpc=9093 '
             '--aks-custom-headers AKSHTTPCustomFeatures=Microsoft.ContainerService/AzureMonitorAppMonitoringPreview'
         )
         self.cmd(update_cmd, checks=[
@@ -17070,8 +17080,10 @@ def test_aks_comprehensive_monitoring_integration(self, resource_group, resource
             # OpenTelemetry features should be re-enabled with new ports
             self.check('azureMonitorProfile.appMonitoring.openTelemetryLogsAndTraces.enabled', True),
             self.check('azureMonitorProfile.appMonitoring.openTelemetryLogsAndTraces.httpPort', 9090),
+            self.check('azureMonitorProfile.appMonitoring.openTelemetryLogsAndTraces.grpcPort', 9092),
             self.check('azureMonitorProfile.appMonitoring.openTelemetryMetrics.enabled', True),
             self.check('azureMonitorProfile.appMonitoring.openTelemetryMetrics.httpPort', 9091),
+            self.check('azureMonitorProfile.appMonitoring.openTelemetryMetrics.grpcPort', 9093),
         ])
 
         # Phase 5: Update - disable Azure Monitor metrics (should also disable OpenTelemetry metrics)
@@ -17133,8 +17145,8 @@ def test_aks_comprehensive_monitoring_integration(self, resource_group, resource
         update_cmd = (
             'aks update --resource-group={resource_group} --name={name} --yes --output=json '
             '--enable-azure-monitor-logs --enable-azure-monitor-metrics --enable-azure-monitor-app-monitoring '
-            '--enable-opentelemetry-logs-traces --opentelemetry-logs-traces-port-http=7070 '
-            '--enable-opentelemetry-metrics --opentelemetry-metrics-port-http=7071 '
+            '--enable-opentelemetry-logs-traces --opentelemetry-logs-traces-port-http=7070 --opentelemetry-logs-traces-port-grpc=7072 '
+            '--enable-opentelemetry-metrics --opentelemetry-metrics-port-http=7071 --opentelemetry-metrics-port-grpc=7073 '
             '--enable-windows-recording-rules '
             '--aks-custom-headers AKSHTTPCustomFeatures=Microsoft.ContainerService/AzureMonitorAppMonitoringPreview'
         )
@@ -17161,8 +17173,10 @@ def test_aks_comprehensive_monitoring_integration(self, resource_group, resource
             # All OpenTelemetry features should be enabled with new ports
             self.check('azureMonitorProfile.appMonitoring.openTelemetryLogsAndTraces.enabled', True),
             self.check('azureMonitorProfile.appMonitoring.openTelemetryLogsAndTraces.httpPort', 7070),
+            self.check('azureMonitorProfile.appMonitoring.openTelemetryLogsAndTraces.grpcPort', 7072),
             self.check('azureMonitorProfile.appMonitoring.openTelemetryMetrics.enabled', True),
             self.check('azureMonitorProfile.appMonitoring.openTelemetryMetrics.httpPort', 7071),
+            self.check('azureMonitorProfile.appMonitoring.openTelemetryMetrics.grpcPort', 7073),
         ])
 
         # Phase 8: Final cleanup - disable all monitoring features