canonical / juju-lint

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

False positive: nrpe lacp_bonds/netlinks errors on LMA model #237

Open zxhdaze opened 6 months ago

zxhdaze commented 6 months ago

False positives found on LMA model:

2023-08-23 11:54:51 [ERROR] [lma-bundle.yaml] [manual/manual] Application nrpe has incorrect setting for 'lacp_bonds': Should not be '' 2023-08-23 11:54:51 [ERROR] [lma-bundle.yaml] [manual/manual] Application nrpe has incorrect setting for 'netlinks': Should not be ''

Per https://bugs.launchpad.net/juju-lint/+bug/2011568, it seems that these can be avoided via deploying the nrpe app with an lxd or container suffix. Neither of those is accurate for the LMA model since those are typically VMs running on the infra nodes.

Should a new suffix be added to cover this case?