~jfguedez/juju-lint:bug/1903973

Branch merges

Propose for merging

Merged into juju-lint:master at revision 3b85de9d81bf29c7769febd0a60c7a41fc424a34

Edin S (community): Approve on 2021-09-20

Juju Lint maintainers: Pending requested 2021-09-20

Diff: 340 lines (+158/-47)

5 files modified

contrib/includes/base.yaml (+1/-1)
jujulint/lint.py (+22/-8)
jujulint/util.py (+25/-3)
tests/conftest.py (+6/-0)
tests/test_jujulint.py (+104/-35)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related charm recipes

c686905... by Jose Guedez on 2021-09-17

Add the "metal only" condition for subordinates

* This new condition differentiates from containers/VMs.
* Use this new conditon for the 'hw-health' charm in the rules
* Also some refactoring of tests and test fixtures.

Closes-Bug: 1903973

bd261d5... by Jose Guedez on 2021-09-17

Refactor logging to make code more readable

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

d219e5e... by Jose Guedez on 2021-09-17

Refactor logging to make code more readable

737b1c0... by Jose Guedez on 2021-09-17

Add support for includes, as well as several reference rules files

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

31d208a... by Jose Guedez on 2021-09-15

Add support for includes, as well as several reference rules files

The include feature effectively can be as a substitute for overrides/overlays
which are not supported in YAML.

Several rule files have been provided to cover typical use cases:

* openstack-bionic-ovn.yaml
* openstack-bionic-ovs.yaml
* openstack-focal-ovn.yaml
* openstack-focal-ovs.yaml
* kubernetes.yaml

These new files are based on modular includes and native YAML features
like anchors and merge operations. The canonical-rules.yaml file
remains for legacy reasons.

Closes-Bug: 1896551
Closes-Bug: 1904038
Closes-Bug: 1916045
Closes-Bug: 1939434

e4a0112... by Jose Guedez on 2021-09-15

Fix issue with 'neq' logic (empty string), and some lint errors

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

4c8331f... by Jose Guedez on 2021-09-15

Add support for CMR applications (SAAS)

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

cb3ab92... by Jose Guedez on 2021-09-14

Add support for CMR applications (SAAS)

Closes-Bug: 1897262

eb55efa... by Jose Guedez on 2021-09-14

Add missing charms to the rules

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

e46d22d... by Jose Guedez on 2021-09-14

Add missing charms to the rules