False positive: nrpe lacp_bonds/netlinks errors on LMA model
Bug #2033317 reported by
Paul Goins
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:/
Should a new suffix be added to cover this case?
Changed in juju-lint: | |
status: | New → Triaged |
importance: | Undecided → Medium |
tags: | added: bseng-1514 |
To post a comment you must log in.
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?