False positive: nrpe lacp_bonds/netlinks errors on LMA model

Bug #2033317 reported by Paul Goins
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juju Lint
Incomplete
Medium
Unassigned

Bug Description

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?

Tags: bseng-1514
Eric Chen (eric-chen)
Changed in juju-lint:
status: New → Triaged
importance: Undecided → Medium
tags: added: bseng-1514
Revision history for this message
Andrea Ieri (aieri) wrote :

Yes, the suffixes logic was added in bug 1944406, and can be seen in lint.py, line 534 and later.
The idea is that rules would only apply to applications matching the base charm name or the base charm name + listed suffix.
This mechanism is very fragile, but it's supposed to work: if you deploy nrpe in tha LMA model with any suffix that is *not* -host or -physical, the lacp_bonds and netlinks configuration checks will be skipped.

Unfortunately making juju-lint able to dynamically determine the type of nrpe unit and act accordingly is at the moment likely to be very complex.

As an alternative, one could specify different rulesets for the LMA model, of course.

Would either of the workarounds above be sufficient for the time being?

Changed in juju-lint:
status: Triaged → Incomplete
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.