environs/config: fails if logging config is not attr=value

[Roger Peppe]

In environs/config/config.go, there is this code:

 levels, err := loggo.ParseConfigurationString(loggingConfig)
 if err != nil {
  return err
 }
 // If there is is no specified level for "unit", then set one.
 if _, ok := levels["unit"]; !ok {
  loggingConfig = loggingConfig + ";unit=DEBUG"
 }

This fails if loggingConfig is "DEBUG" (a valid config) because "DEBUG;unit=DEBUG" is
not a valid logging configuration.

DEBUG seems like a bad default anyway, but perhaps loggo.ParseConfiguration
could be changed to accept the above string as equivalent to "<root>=DEBUG;unit=DEBUG".

[Roger Peppe (rogpeppe)]

This is still a valid bug that really should be fixed.

[Anastasia (anastasia-macmood)]

As this is a tech-debt item, I am lowering its Importance.

[Canonical Juju QA Bot (juju-qa-bot)]

This bug has not been updated in 5 years, so we're marking it Expired. If you believe this is incorrect, please update the status.