Merge ~arif-ali/juju-lint/+git/juju-lint:fix_openstack_example into juju-lint:master

Proposed by Arif Ali
Status: Merged
Approved by: James Hebden
Approved revision: 8edd291152f47e5d8931e1553105bf7ca4696cf9
Merged at revision: e9499f83aef38e44cb3c27e2f6fb270b5135443a
Proposed branch: ~arif-ali/juju-lint/+git/juju-lint:fix_openstack_example
Merge into: juju-lint:master
Diff against target: 12 lines (+0/-1)
1 file modified
contrib/canonical-openstack-rules.yaml (+0/-1)
Reviewer Review Type Date Requested Status
James Hebden (community) Approve
Review via email: mp+383946@code.launchpad.net

Commit message

remove host-suffixes

The host-suffixes attribute is not there for the 'all except' where
attribute so removing to reduce confusion

To post a comment you must log in.
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

This merge proposal is being monitored by mergebot. Change the status to Approved to merge.

Revision history for this message
James Hebden (ec0) wrote :

Approved, thanks for the MR

review: Approve
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

Change successfully merged at revision e9499f83aef38e44cb3c27e2f6fb270b5135443a

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/contrib/canonical-openstack-rules.yaml b/contrib/canonical-openstack-rules.yaml
2index 1dc8830..14e2ca1 100644
3--- a/contrib/canonical-openstack-rules.yaml
4+++ b/contrib/canonical-openstack-rules.yaml
5@@ -1,7 +1,6 @@
6 subordinates:
7 telegraf:
8 where: all except prometheus # and prometheus-ceph-exporter and prometheus-openstack-exporter
9- host-suffixes: [prometheus]
10 landscape-client:
11 where: all except landscape-server
12 filebeat:

Subscribers

People subscribed via source and target branches