lp:~frankban/juju-gui/bug-1168308-add-unit

Branch merges

Propose for merging

No branches dependent on this one.

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

Juju GUI Hackers: Pending requested 2013-04-12

Diff: 121 lines (+100/-0)

2 files modified

app/store/env/go.js (+59/-0)
test/test_env_go.js (+41/-0)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #1168308: Implement add_unit in GoEnvironment

High

Fix Released

Link a bug report

Related blueprints

544. By Francesco Banconi on 2013-04-12

Merged trunk.

543. By Francesco Banconi on 2013-04-12

Changes as per review.

542. By Francesco Banconi on 2013-04-12

Implemented add_unit in the Go env.

541. By Francesco Banconi on 2013-04-12

Add tests for add_unit.

540. By Gary Poster on 2013-04-11

Fix bug 1167967 (hopefully)

This branch attempts to fix the referenced bug. I was unable to duplicate the symptoms locally exactly, but duplicated them closely enough that I am optimistic that these changes address the issue. Regardless, they seem like reasonable changes.

I also fixed a couple of landscape issues I saw with newly created services without annotations.

The aspect of this branch I wonder most if reviewers (particularly Ben) will like is the change to the route url algorithm. If it's not clear why I chose it, I'm happy to discuss my rationale and other options.

Thank you.

R=bcsaller, matthew.scott
CC=
https://codereview.appspot.com/8680043

539. By j.c.sackett on 2013-04-11

Adds "popular" charms to sidebar.

-Another block in sidebar creates the "popular" charm container.
-"Popular" charms also added to cache.
-"Popular" block added to sidebar template.
-Added "popular" to sidebar json test data, and re-pretty-printed it.
-Updated test for the additional charm in cache; although "popular" has two
charms, one of them (cassandra) is also in our "new" category.

R=rharding, jeff.pihach
CC=
https://codereview.appspot.com/8561048

538. By Richard Harding on 2013-04-11

Adjust design per review, add truncate helper

- Per UX reviews adjust some of the design of the browser
- Add a truncate handlebars helper to aid in fitting content nicely

R=j.c.sackett
CC=
https://codereview.appspot.com/8662043

537. By Gary Poster on 2013-04-11

Fix sandbox setconfig and associated test

The sandbox setconfig test was taking over a second--a pretty clear indication that it was waiting on the fake delta stream rather than a direct response. Investigation revealed that the sandbox method was incorrectly using the ASYNCOP helper rather than the OP helper. This small branch fixes the setconfig problem and adjusts the tests. The test is very fast now (not yellow or red in the mocha output).

R=bac, matthew.scott
CC=
https://codereview.appspot.com/8663043

536. By Madison Scott-Clary on 2013-04-11

Trivial: add relation works with touch.

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

535. By Francesco Banconi on 2013-04-11

Fix exposed attribute handling in delta stream

This branch fixes the service "exposed" and unit "relation_errors"
attributes handling when the delta comes from juju-core.
The _process_delta function allows for overriding those attrs
in the case a delta stream arrives from PyJuju. The juju-core
handlers, instead, must update model instances with partial changes,
e.g. when handling annotations. This was the cause of the regression.

Also improved the cleanUpEntityTags helper.

R=teknico, gary.poster
CC=
https://codereview.appspot.com/8657043