Skip to content

Commit

Permalink
test_admin_client.py: Use new publish appId metrics flag
Browse files Browse the repository at this point in the history
Signed-off-by: Yuan Jing Vincent Yan <yyan82@bloomberg.net>
  • Loading branch information
kaikulimu committed Jul 29, 2024
1 parent c7e2f8e commit e6c4df5
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion src/integration-tests/test_admin_client.py
Original file line number Diff line number Diff line change
Expand Up @@ -158,7 +158,6 @@ def test_breathing(single_node: Cluster) -> None:
admin.stop()


@tweak.broker.app_config.stats.app_id_tag_domains([tc.DOMAIN_FANOUT])
def test_queue_stats(single_node: Cluster) -> None:
"""
Test: queue metrics via admin command.
Expand All @@ -184,6 +183,12 @@ def test_queue_stats(single_node: Cluster) -> None:
- Safeguarding mechanism prevents from getting stats too often.
"""

domains = {domain.name: domain for domain in single_node.configurator.domains}
domains[
tc.DOMAIN_FANOUT
].definition.parameters.mode.fanout.publish_app_id_metrics = True
single_node.deploy_domains()

# Preconditions
admin = AdminClient()
admin.connect(*single_node.admin_endpoint)
Expand Down

0 comments on commit e6c4df5

Please sign in to comment.