Misconfigured sources does not give any error

Bug #1887947 reported by Benjamin Allot
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
turku-charms
Fix Committed
Medium
James Simpson

Bug Description

The following configuration while invalid, didn't produce any error in logs:

juju config turku-agent-cdimage-master sources='{"etc": {"comment": "etc", "path": "/etc", "frequency": "daily"}'

It should be

juju config turku-agent-cdimage-master sources='{"etc": {"comment": "etc", "path": "/etc", "frequency": "daily"}}'

The only log seen when applying the first line is

unit-turku-agent-cdimage-master-0: 12:21:37 INFO unit.turku-agent-cdimage-master/0.juju-log Hook: config-changed
unit-turku-agent-cdimage-master-0: 12:21:37 INFO unit.turku-agent-cdimage-master/0.juju-log Running turku-update-config

Related branches

Revision history for this message
James Simpson (jsimpso) wrote :

This has been improved with the following MP: https://code.launchpad.net/~jsimpso/turku-charms/improve_logging/+merge/422802

We'll now see more useful feedback, e.g.:
2022-05-18 06:08:38 INFO juju-log Hook: config-changed
2022-05-18 06:08:38 ERROR juju-log Error while parsing config
2022-05-18 06:08:38 ERROR juju-log Expecting ':' delimiter: line 1 column 299 (char 298)

2022-05-18 06:09:38 INFO juju-log Hook: config-changed
2022-05-18 06:09:38 INFO juju-log Successfully parsed config
2022-05-18 06:09:38 INFO juju-log Running turku-update-config

I'm not sure if you want to call this bug resolved or not though, as ideally I believe the charm should end up in a "BLOCKED" state following a JSON parsing error.

Changed in turku-charms:
status: New → Confirmed
importance: Undecided → Medium
Benjamin Allot (ballot)
Changed in turku-charms:
assignee: nobody → James Simpson (jsimpso)
status: Confirmed → Fix Committed
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.