juju-gui

GUI fails to deploy keystone

Bug #1214087 reported by Jorge Castro
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
juju-gui
Fix Released
Critical
Jeff Pihach
keystone (Juju Charms Collection)
Fix Released
Undecided
Unassigned

Bug Description

We had a guy on IRC who could not deploy Keystone via the GUI. I think this is another one of those YAML validation things where the CLI deploys but the GUI does not.

To replicate try to deploy keystone. To workaround add an IP address to the VIP field.

Keystone should be deployable with an empty VIP field.

Related branches

lp:~hatch/juju-gui/save-changed-1214087
Juju GUI Hackers: Pending requested
Diff: 826 lines (+268/-67)
17 files modified
app/store/env/fakebackend.js (+28/-2)
app/store/env/go.js (+51/-6)
app/store/env/python.js (+18/-5)
app/views/ghost-inspector.js (+2/-0)
app/views/inspector.js (+2/-0)
app/views/service.js (+1/-0)
app/views/utils.js (+38/-2)
app/views/viewlets/service-ghost.js (+5/-0)
test/assets/mysql-config.yaml (+1/-1)
test/test_env_go.js (+9/-2)
test/test_env_python.js (+9/-10)
test/test_fakebackend.js (+21/-6)
test/test_ghost_inspector.js (+3/-7)
test/test_sandbox_go.js (+50/-12)
test/test_sandbox_python.js (+22/-7)
test/test_service_config_view.js (+4/-3)
test/test_utils.js (+4/-4)
Revision history for this message
Jorge Castro (jorge) wrote :

From the log:

2013-08-18 14:16:11,995 unit:keystone/3: unit.hook.api INFO: FATAL ERROR: ERROR: Config option has no paramter: vip

Revision history for this message
Jorge Castro (jorge) wrote :

Here's the full log:

http://pastebin.ubuntu.com/6003993/

Revision history for this message
Adam Gandelman (gandelman-a) wrote :

The Keystone charm's parsing of config-get JSON output purposely fails if a config setting is set to an empty string. Not sure this is required anymore. In any case, the charm should be using charm-helpers utilities at this point.

Gary Poster (gary)
Changed in juju-gui:
status: New → Triaged
importance: Undecided → Critical
Revision history for this message
Gary Poster (gary) wrote :

I hope that the way around this for the GUI is to only send values that have changed (or, in the case of a deployment, are different that the default). If that doesn't work, we may have to dig a bit more.

We will incorporate this into the inspector work, which we are wrapping up.

Jeff Pihach (hatch)
Changed in juju-gui:
assignee: nobody → Jeff Pihach (hatch)
Gary Poster (gary)
Changed in juju-gui:
status: Triaged → In Progress
status: In Progress → Fix Committed
Gary Poster (gary)
Changed in juju-gui:
status: Fix Committed → Fix Released
Dave Cheney (dave-cheney)
Changed in keystone (Juju Charms Collection):
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.