Juju Lint

Merge ~jfguedez/juju-lint:bug/1913405-2 into juju-lint:master

  1. Git
  2. lp:~jfguedez/juju-lint
  3. bug/1913405-2
  4. Merge into master
Proposed by Jose Guedez
Status: Merged
Approved by: Dan Ackerson
Approved revision: 01c70478feded75d55ed87ab479d2ce6c724f0a0
Merged at revision: e8b686b008ee69cacfb4ed5a7f55b90acc53cdcc
Proposed branch: ~jfguedez/juju-lint:bug/1913405-2
Merge into: juju-lint:master
Diff against target: 13 lines (+1/-1)
1 file modified
contrib/canonical-rules.yaml (+1/-1)
Reviewer Review Type Date Requested Status
Dan Ackerson (community) Approve
Linda Guo (community) Approve
Review via email: mp+398662@code.launchpad.net

Commit message

Add logrotated fix again, was reverted by multiple successive merges

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
Linda Guo (lihuiguo) wrote :

LGTM +1

review: Approve
Revision history for this message
Dan Ackerson (dan.ackerson) wrote :

+1

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

Change successfully merged at revision e8b686b008ee69cacfb4ed5a7f55b90acc53cdcc

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 6eaa448..d109037 100644
3--- a/contrib/canonical-rules.yaml
4+++ b/contrib/canonical-rules.yaml
5@@ -88,7 +88,7 @@ subordinates:
6 where: on nagios
7 hw-health:
8 where: host only
9- logrotate:
10+ logrotated:
11 where: all
12
13 operations mandatory: &operations-mandatory-charms

Subscribers

People subscribed via source and target branches