Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(2549)

Issue 55230044: Update deployer to 0.3.1

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 2 months ago by frankban
Modified:
10 years, 2 months ago
Reviewers:
mp+203999, jeff.pihach, gary.poster
Visibility:
Public.

Description

Update deployer to 0.3.1 Update the juju-deployer package to the newer version. As a consequence, improve the bundle deployment feedback. Tests: `make unittest`. QA: 1) deploy the charm by bootstrapping juju and then running `make deploy`. 2) wait for the service to be ready/started. 3) Open the GUI. 4) Try to deploy this bundle file: http://pastebin.ubuntu.com/6844154/ 5) You should see an error in the GUI: The deployment errored: Invalid config charm cs:~openstack-ubuntu-testing/precise/mysql-4 no-such-option=42 6) Try to deploy this bundle file: http://pastebin.ubuntu.com/6844187/ 7) You should see an error in the GUI: The deployment errored: Invalid relation in config, service no-such not found, rel mediawiki:db <-> no-such:db Done, thank you! https://code.launchpad.net/~frankban/charms/precise/juju-gui/deployer-031/+merge/203999 (do not edit description out of merge proposal)

Patch Set 1 #

Patch Set 2 : Update deployer to 0.3.1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
A [revision details] View 1 chunk +2 lines, -0 lines 0 comments Download
D deps/juju-deployer-0.3.1-prerelease.tar.gz View 0 chunks +-1 lines, --1 lines 0 comments Download
A deps/juju-deployer-0.3.1.tar.gz View 0 chunks +-1 lines, --1 lines 0 comments Download
M server-requirements.pip View 1 chunk +1 line, -3 lines 0 comments Download

Messages

Total messages: 5
frankban
Please take a look.
10 years, 2 months ago (2014-01-30 15:26:07 UTC) #1
gary.poster
LGTM
10 years, 2 months ago (2014-01-30 15:35:04 UTC) #2
jeff.pihach
LGTM QA OK (from Makyo)
10 years, 2 months ago (2014-01-30 19:03:48 UTC) #3
frankban
*** Submitted: Update deployer to 0.3.1 Update the juju-deployer package to the newer version. As ...
10 years, 2 months ago (2014-01-31 08:59:33 UTC) #4
frankban
10 years, 2 months ago (2014-01-31 09:39:13 UTC) #5
Thank you!
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b