Merge lp:~hatch/juju-gui/bundle-fail-gracefully into lp:juju-gui/experimental
Proposed by
Jeff Pihach
Status: | Merged |
---|---|
Merged at revision: | 1132 |
Proposed branch: | lp:~hatch/juju-gui/bundle-fail-gracefully |
Merge into: | lp:juju-gui/experimental |
Diff against target: |
222 lines (+125/-23) 2 files modified
app/subapps/browser/views/bundle.js (+48/-8) test/test_bundle_details_view.js (+77/-15) |
To merge this branch: | bzr merge lp:~hatch/juju-gui/bundle-fail-gracefully |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Juju GUI Hackers | Pending | ||
Review via email: mp+190769@code.launchpad.net |
Description of the change
Bundle topology rendering fails gracefully
If the bundle does not provide proper xy position annotations
on any of the services in its deployer file we do not render
the bundle topology and instead remove the bundle topology
tab, selecting the Charms tab to show the charms the
bundle contains.
Because we don't have any bundles with proper positions right
now I added the flag `strictBundle` to enable this feature.
To post a comment you must log in.
Reviewers: mp+190769_ code.launchpad. net,
Message:
Please take a look.
Description:
Bundle topology rendering fails gracefully
If the bundle does not provide proper xy position annotations
on any of the services in its deployer file we do not render
the bundle topology and instead issue a warning.
Because we don't have any bundles with proper positions right
now I added the flag strictBundle to enable this feature.
https:/ /code.launchpad .net/~hatch/ juju-gui/ bundle- fail-gracefully /+merge/ 190769
(do not edit description out of merge proposal)
Please review this at https:/ /codereview. appspot. com/14502061/
Affected files (+119, -23 lines): browser/ views/bundle. js bundle_ details_ view.js
A [revision details]
M app/subapps/
M test/test_