lp:~hatch/juju-gui/inspector-relation-status

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp:juju-gui/experimental at revision 1212

Juju GUI Hackers: Pending requested 2013-11-26

Diff: 306 lines (+82/-88)

8 files modified

app/models/models.js (+31/-22)
app/templates/service-relations-list.handlebars (+1/-1)
app/templates/service-relations-viewlet.handlebars (+1/-1)
app/templates/service-relations.handlebars (+0/-40)
app/views/utils.js (+6/-0)
app/views/viewlets/service-relations.js (+3/-7)
lib/views/juju-inspector.less (+11/-0)
test/test_model.js (+29/-17)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Related blueprints

1214. By Jeff Pihach on 2013-11-26

fixed issue where the first error status delta would get ignored for relation errors

1213. By Jeff Pihach on 2013-11-26

lint

1212. By Jeff Pihach on 2013-11-26

Relation status is now shown on the inspector relations tab

1211. By Jeff Pihach on 2013-11-25

Adds ellipsis to unit list headers

The unit list headers could overflow pushing the chevron onto the next line
when the unit status name is too long. This branch maintains the unit number
and chevron in their appropriate places while limiting the length of the
unit type when closed.

R=gary.poster
CC=
https://codereview.appspot.com/32260044

1210. By Jeff Pihach on 2013-11-25

Display all unit status categories in unit list

The inspector unit list will now display all unit error types split out each
into their own section.

It now also has the ability to display all unit pending types but there are
other components which are holding this functionality back. To see it in
action load up the GUI with the 'simPend' flag. It will now show
'installing', 'dying' as well as 'pending'. This is just for illustrative
purposes.

R=rharding, gary.poster
CC=
https://codereview.appspot.com/31190043

1209. By Huw Wilkins on 2013-11-25

Add animations to browser

I had a bit of a realisation about how we could do open animations for the browser panels. It seems to QA accross browsers and even though the panels don't animate on close I think it's worth having.

I'm hoping this might be enough that we don't have to worry about animations on close...

There is one point which might be considered a bug in that the panels will animate in on page load instead of remaining in the open state. I think this is an OK trade off, but others may feel differently.

R=jeff.pihach
CC=
https://codereview.appspot.com/31930043

1208. By Richard Harding on 2013-11-21

Display bundle download counts in the UI.

- Update the model to carry the attribute from the api.
- Update the token to display the "Deployed X times".
- Update the details to also display the count.
- Drive-by to update the attributes in the charm model.
- Update tests with updated api download from the server that includes the
download counts. Required updating tests to the new data.

R=jeff.pihach
CC=
https://codereview.appspot.com/30210044

1207. By Brad Crittenden on 2013-11-21

Send bundle id when d'n'd from sidebar.

This path was initially missed. The Token had to be fixed to include the
ID in the drag data.

Note, for selfish reasons, test/utils.js loadFixtures was updated to retry on
failures. The reason for the occassional 404 in this function when run by
lbox is still unknown. Hopefully the dumb retry will help.

QA Instructions:

# Use juju-quickstart trunk
% cd projects/juju-quickstart
% bzr pull
% make
# Deploy on EC2
% juju switch ec2
% .venv/bin/python juju-quickstart --gui-charm-url cs:~juju-gui/precise/juju-gui-128
% juju set juju-gui "juju-gui-source=lp:~bac/juju-gui/fix-dnd-bundles-wrt-id"
# May want to wait a bit. Perhaps it isn't necessary.
% juju set juju-gui "charmworld-url=http://staging.jujucharms.com"

Find one of Jorge's bundles on http://staging.jujucharms.com.
(wordpress-simple is a fast one.) Look at the deployment counts.

In the GUI search for 'jorge', find the bundle from above, and *drag* it onto
the canvas. Have a light snack. When everything is green and happy, look at
the page again on staging and see that the count has increased by one. NOTE:
in my QA I didn't see the incremented values reflected immediately which is
really strange. A few minutes later they were updated.

R=benji
CC=
https://codereview.appspot.com/29960045

1206. By Richard Harding on 2013-11-20

fakebackend/sandbox no op deployer watches

- The fakebackend and the sandbox need to just ignore calls to watch deployer
status. It's too fast and we don't want to flood the users with a bunch of
notices all at once.
- Had to rename the WatchUpdate to deployerNext since the fakebackend auto
builds the names of the handle methods based off the actual call. So it needed
to be deployerNext though I hate it's no longer worded to tie to 'watch'.

QA:

Simply deploy a bundle in sandbox mode.

- Drag/drop a yaml file into the canvas
- Deploy from the deploy button from bundle details

Note: Dragging/dropping a bundle token is not working. This is also true on
coming soon. This is due to the recent bundle.id tracking code and will be
done in a follow up.

R=bac
CC=
https://codereview.appspot.com/29860044

1205. By Jeff Pihach on 2013-11-20

Adds agent_state_data(SourceData) support

The simulator and juju-core return SourceData along with any
error messages. These are now displayed in the unit details
left breakout panel in the new inspector.

I also removed an unused template from the old inspectors and
modified some css to make the left panel react properly to
varying unit detail header heights.

R=matthew.scott, gary.poster
CC=
https://codereview.appspot.com/29400043