canonical / juju-lint

Run checks against a juju model
GNU General Public License v3.0
0 stars 3 forks source link

Incorrect logrotated charm name in canonical-rules.yaml #274

Closed zxhdaze closed 5 months ago

zxhdaze commented 5 months ago

/snap/juju-lint/current/contrib/canonical-rules.yaml defines logrotate charm, instead of logrotated -- note missing 'd'.

84 hw-health: 85 where: host only 86 logrotate: <--- HERE 87 where: all 88 89 operations mandatory: &operations-mandatory-charms 90 - elasticsearch

As a result, juju-lint reports false-positive about missing logrotated relations:

2021-01-27 08:25:53 [ERROR] [/tmp/juju_status.json] [manual/manual] -> logrotate [aodh, barbican, ceilometer, ceph-mon, ceph-osd, ceph-radosgw, cinder, designate, designate-bind, easyrsa, etcd, glance, gnocchi, heat, keystone, memcached, mysql-innodb-cluster, neutron-api, nova-cloud-controller, nova-compute, octa via, openstack-dashboard, openstack-service-checks, placement, prometheus-ceph-exporter, prometheus-openstack-exporter, rabbitmq-server, ubuntu, vault]

$ snap info juju-lint | grep installed installed: 1.0.2.dev7+g39fd901 (25) 13MB classic