Juju Lint

Merge ~aieri/juju-lint:bug/1905608 into juju-lint:master

  1. Git
  2. lp:~aieri/juju-lint
  3. bug/1905608
  4. Merge into master
Proposed by Andrea Ieri
Status: Merged
Approved by: Andrea Ieri
Approved revision: f4c325a7c8231e53cf0b215ee21a1b7b4812eff2
Merged at revision: 7ffb3edf99f00eb490559990683330168e14f33e
Proposed branch: ~aieri/juju-lint:bug/1905608
Merge into: juju-lint:master
Diff against target: 15 lines (+0/-4)
1 file modified
contrib/canonical-rules.yaml (+0/-4)
Reviewer Review Type Date Requested Status
Dan Ackerson (community) Approve
BootStack Reviewers Pending
Review via email: mp+398090@code.launchpad.net

Commit message

Don't warn on live migration settings
Closes-Bug: 1905608

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
Dan Ackerson (dan.ackerson) wrote :

lgtm

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

Change successfully merged at revision 7ffb3edf99f00eb490559990683330168e14f33e

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/contrib/canonical-rules.yaml b/contrib/canonical-rules.yaml
index 0961671..7ee3dc5 100644
--- a/contrib/canonical-rules.yaml
+++ b/contrib/canonical-rules.yaml
@@ -22,10 +22,6 @@ openstack config:
22 global-physnet-mtu:22 global-physnet-mtu:
23 eq: 900023 eq: 9000
24 nova-compute:24 nova-compute:
25 live-migration-permit-auto-converge:
26 eq: true
27 live-migration-permit-post-copy:
28 eq: true
29 cpu-model:25 cpu-model:
30 isset: true26 isset: true
31 ceph-osd-replication-count:27 ceph-osd-replication-count:

Subscribers

People subscribed via source and target branches