Skip to content

Commit

Permalink
fork o11y test to create another one with injection
Browse files Browse the repository at this point in the history
  • Loading branch information
arvindbr8 committed Dec 3, 2024
1 parent e1e41a3 commit f37789b
Showing 1 changed file with 2 additions and 8 deletions.
10 changes: 2 additions & 8 deletions tests/gamma/csm_observability_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -193,9 +193,7 @@ def initKubernetesClientRunner(self, **kwargs) -> KubernetesClientRunner:
enable_csm_observability=True,
csm_workload_name=CSM_WORKLOAD_NAME_CLIENT,
csm_canonical_service_name=CSM_CANONICAL_SERVICE_NAME_CLIENT,
),
namespace_template="gamma/namespace.yaml",
deployment_template="gamma/client.deployment.yaml",
)
)

# These parameters are more pertaining to the test itself, not to
Expand All @@ -206,9 +204,7 @@ def initKubernetesServerRunner(self, **kwargs) -> GammaServerRunner:
enable_csm_observability=True,
csm_workload_name=CSM_WORKLOAD_NAME_SERVER,
csm_canonical_service_name=CSM_CANONICAL_SERVICE_NAME_SERVER,
),
namespace_template="gamma/namespace.yaml",
deployment_template="gamma/server.deployment.yaml",
)
)

def test_csm_observability(self):
Expand Down Expand Up @@ -288,7 +284,6 @@ def test_csm_observability(self):
# values
with self.subTest("7_check_metrics_labels_histogram_client"):
expected_metric_labels = {
# TODO(arvindbright): Add assetion to CSM_MESH_ID
"csm_mesh_id": ANY,
"csm_remote_workload_canonical_service": CSM_CANONICAL_SERVICE_NAME_SERVER,
"csm_remote_workload_cluster_name": ANY,
Expand Down Expand Up @@ -320,7 +315,6 @@ def test_csm_observability(self):
# values
with self.subTest("8_check_metrics_labels_histogram_server"):
expected_metric_labels = {
# TODO(arvindbright): Add assetion to CSM_MESH_ID
"csm_mesh_id": ANY,
"csm_remote_workload_canonical_service": CSM_CANONICAL_SERVICE_NAME_CLIENT,
"csm_remote_workload_cluster_name": ANY,
Expand Down

0 comments on commit f37789b

Please sign in to comment.