warn/alert on non-default configuration

Bug #1976107 reported by Garrett Neugent
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juju Lint
Won't Fix
Undecided
Unassigned

Bug Description

To ensure that minimal changes are made for a given application (and that default configuration isn't set by the user), it would be great to add a check for this. The following command will show non-default configs for a given application:

juju config $app --format json | jq -c '.settings | to_entries[] | select(.value.source=="user") | [.key, .value.value]'

Revision history for this message
Andrea Ieri (aieri) wrote :

@Garrett are you suggesting we alert on any user-set config option or only on options that have a value set which is identical to the default?

I'm thinking that there are probably many instances of charm options that will almost always require a user config; we wouldn't want to perma-alert on those.

Revision history for this message
Garrett Neugent (thogarre) wrote :

I think as a first step it would be great to identify settings that already match the default.

In terms of non-default settings, i'm not sure how we'd go about this, but it would be good to make sure that these are still validated. maybe a warn instead of a crit? or we can add a known list of common changes, to ignore (e.g nagios host context, netlinks, certs, etc.)

Revision history for this message
Eric Chen (eric-chen) wrote :
Changed in juju-lint:
status: New → Won't Fix
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.