Skip to content

Commit c7d36fb

Browse files
Apply suggestions from code review
Co-authored-by: dkirov-dd <[email protected]>
1 parent 412f035 commit c7d36fb

File tree

3 files changed

+3
-5
lines changed

3 files changed

+3
-5
lines changed

redis_enterprise/assets/dashboards/redis_enterprise_proxy.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
{
1+
{
22
"title": "Redis Enterprise - Proxy",
33
"description": "Redis Enterprise Proxy presents a single, usable endpoint that hides topological complexity",
44
"widgets": [

redis_enterprise_prometheus/assets/configuration/spec.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,4 +7,4 @@ files:
77
- template: init_config/default
88
- template: instances
99
options:
10-
- template: instances/default
10+
- template: instances/openmetrics

redis_enterprise_prometheus/tests/test_redis_enterprise_prometheus.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -99,10 +99,8 @@ def test_instance_invalid_group_check(aggregator, dd_run_check, mock_http_respon
9999
dd_run_check(check)
100100
except ConfigurationError:
101101
assert True
102-
except Exception:
103-
assert True
104102

105-
aggregator.assert_service_check(f"{RedisEnterprisePrometheusCheck.__NAMESPACE__}.group_bogus", count=0)
103+
aggregator.assert_service_check(f"{RedisEnterprisePrometheusCheck.__NAMESPACE__}.more_groups", count=0)
106104

107105

108106
@pytest.mark.unit

0 commit comments

Comments
 (0)