Juju Lint

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

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

Commit message

Warn on unconfigured lacp_bonds and netlinks options

Closes-Bug: 1912344

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 :

Failed to merge change (unable to merge source repository due to conflicts), setting status to needs review.

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

Change successfully merged at revision 5be5c60e019de79340e6dc0e6e2d294648e6f26d

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 904ec4d..e7ee200 100644
3--- a/contrib/canonical-rules.yaml
4+++ b/contrib/canonical-rules.yaml
5@@ -57,6 +57,11 @@ config:
6 ntp:
7 auto_peers:
8 eq: true
9+ nrpe:
10+ lacp_bonds:
11+ isset: true
12+ netlinks:
13+ isset: true
14 landscape-client:
15 disable-unattended-upgrades:
16 eq: true

Subscribers

People subscribed via source and target branches