Split out unit_config config.yaml parameter into multiple parameters

Bug #1292416 reported by Evan
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ubuntu CI Engine
Confirmed
Undecided
Unassigned

Bug Description

8:43 AM <ev> vila, doanac: on the unit_config vs juju set deathmatch, another thing I'd like to be able to do is change the CI_LAUNCHPAD_* config details without having to redeploy. juju set would make that possible. Though I was thinking this morning that there's no reason to get rid of the unit_config file, just the base64'd config parameter
8:44 AM <ev> with it being replaced by separate config parameters for each item in it

10:11 AM <hazmat> ev, configuration inheritance should work.. its a merge of the dicts
10:12 AM <ev> hazmat: oooh, via "inherits: "? Grand.
10:13 AM <hazmat> ev, inherits between deployers.. each with a named service that has options.. the child.. gets the parents config options that aren't overridden by the child.
10:14 AM <hazmat> ev, nutshell yes

So this would allow us to say juju set imagebuild-worker launchpad_user=ev and have the change set and the service restarted, all without having to redeploy.

Tags: airline
Evan (ev)
description: updated
description: updated
Revision history for this message
Andy Doan (doanac) wrote :

we can do that. the reason we originally used unit_config was that every time we needed a new config option you had to update multiple charms to deal with it. Things are more stable now, but getting granular settings like that makes our charms useless to anyone other than us. However, they are pretty much that way already, so i'm not sure it matters.

Changed in uci-engine:
milestone: none → backlog
Evan (ev)
no longer affects: ubuntu-ci-services-itself
Vincent Ladeuil (vila)
Changed in uci-engine:
status: New → Confirmed
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.