~jfguedez/charm-juju-lint:bug/1942945

Last commit made on 2021-09-08
Get this branch:
git clone -b bug/1942945 https://git.launchpad.net/~jfguedez/charm-juju-lint
Only Jose Guedez can upload to this branch. If you are Jose Guedez please log in for upload directions.

Branch merges

Branch information

Name:
bug/1942945
Repository:
lp:~jfguedez/charm-juju-lint

Recent commits

62b61df... by Jose Guedez

Adds debug logging to auto_lint.py

Closes-Bug: 1942945

0c46594... by Jose Guedez

Add feature to split nagios checks

Adds a new config item, and capability to split nrpe checks into several different checks depending on the given criteria. This depends on having a juju-lint capable of providing json output (https://code.launchpad.net/~jfguedez/juju-lint/+git/juju-lint/+merge/403129)

Reviewed-on: https://code.launchpad.net/~jfguedez/charm-juju-lint/+git/charm-juju-lint/+merge/403660
Reviewed-by: James Troup <email address hidden>
Reviewed-by: Linda Guo <email address hidden>

0de0ae3... by Jose Guedez

Add features split-nagios-checks and warn-only

The split checks feature is turned off by default, and relies on
juju-lint having the --format flag.

0233b0c... by Jose Guedez

Temporarily limit charmcraft version to avoid behaviour changes

Reviewed-on: https://code.launchpad.net/~jfguedez/charm-juju-lint/+git/charm-juju-lint/+merge/407875
Reviewed-by: James Troup <email address hidden>
Reviewed-by: Edin S <email address hidden>

5536ef7... by Jose Guedez

Temporarily limit charmcraft version to avoid behaviour changes

The behavior of charmcraft changes fundamentally on version 1.2
(see the bug report for details)
As not all the changes/requirements are in place to use this,
we temporarily pin the version of charmcraft similarly to
charm-kubernetes-service-checks

0e847ab... by Paul Goins

Updated test_15 pattern

During recent testing, I got a different style of message with what
appears to be the same meaning:

  2021-07-23 07:00:05 [ERROR] [/var/snap/juju-lint/common/juju-status.json] [manual/manual] Ops charm 'nagios' is missing.

This patch updates the regex so it will accept both the old pattern and this new form.

Reviewed-on: https://code.launchpad.net/~vultaire/charm-juju-lint/+git/charm-juju-lint/+merge/406133
Reviewed-by: Xav Paice <email address hidden>
Reviewed-by: Xiyue Wang <email address hidden>

2ac1608... by Paul Goins

Updated test_15 pattern

During recent testing, I got a different style of message with what
appears to be the same meaning:

  2021-07-23 07:00:05 [ERROR] [/var/snap/juju-lint/common/juju-status.json] [manual/manual] Ops charm 'nagios' is missing.

This patch updates the regex so it will accept both the old pattern and this new form.

1c0c51f... by Jose Guedez

Add logic to convert to jsfy format, as well as charm config

Reviewed-on: https://code.launchpad.net/~jfguedez/charm-juju-lint/+git/charm-juju-lint/+merge/403587
Reviewed-by: Xav Paice <email address hidden>
Reviewed-by: Linda Guo <email address hidden>
Reviewed-by: Xiyue Wang <email address hidden>

056d789... by Jose Guedez

Add logic to convert to jsfy format, as well as charm config

f841cc4... by Jose Guedez

Include /snap/bin in PATH for the auto_lint cronjob

Reviewed-on: https://code.launchpad.net/~jfguedez/charm-juju-lint/+git/charm-juju-lint/+merge/403558
Reviewed-by: Xiyue Wang <email address hidden>
Reviewed-by: Linda Guo <email address hidden>