Issue 13900044: Fix settings update in juju-core.
DescriptionFix settings update in juju-core.
When sending ServiceSet requests, the GUI included
a Config parameter where juju-core expects an
Option one.
While I was looking at that code, I decided instead to
switch to the newer ServiceUpdate API call, which
handles both map[string]string and YAML settings.
QA:
- deploy and expose the GUI in a juju-core env;
- switch to this branch:
juju set juju-gui juju-gui-source=lp:~frankban/juju-gui/bug-1229939-service-set
- try to change settings (e.g. juju-gui-debug);
- refresh.
The settings viewlet should reflect the changes and,
after a while (after config-changed runs),
the new settings should be applied.
https://code.launchpad.net/~frankban/juju-gui/bug-1229939-service-set/+merge/187544
(do not edit description out of merge proposal)
Patch Set 1 #
MessagesTotal messages: 2
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
