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
1diff --git a/contrib/canonical-rules.yaml b/contrib/canonical-rules.yaml
2index 0961671..7ee3dc5 100644
3--- a/contrib/canonical-rules.yaml
4+++ b/contrib/canonical-rules.yaml
5@@ -22,10 +22,6 @@ openstack config:
6 global-physnet-mtu:
7 eq: 9000
8 nova-compute:
9- live-migration-permit-auto-converge:
10- eq: true
11- live-migration-permit-post-copy:
12- eq: true
13 cpu-model:
14 isset: true
15 ceph-osd-replication-count:

Subscribers

People subscribed via source and target branches