Merge lp://qastaging/~corey.bryant/charms/trusty/nova-compute/use-stable into lp://qastaging/~openstack-charmers-archive/charms/trusty/nova-compute/trunk

Proposed by Corey Bryant
Status: Merged
Merged at revision: 87
Proposed branch: lp://qastaging/~corey.bryant/charms/trusty/nova-compute/use-stable
Merge into: lp://qastaging/~openstack-charmers-archive/charms/trusty/nova-compute/trunk
Diff against target: 12 lines (+1/-1)
1 file modified
tests/basic_deployment.py (+1/-1)
To merge this branch: bzr merge lp://qastaging/~corey.bryant/charms/trusty/nova-compute/use-stable
Reviewer Review Type Date Requested Status
Liam Young (community) Approve
Review via email: mp+239471@code.qastaging.launchpad.net
To post a comment you must log in.
Revision history for this message
Ryan Beisner (1chb1n) wrote :

UOSCI bot says:
charm_lint_check #798 trusty-nova-compute for corey.bryant mp239471
    LINT OK: believed to pass, but you should confirm results

LINT Results (max last 4 lines) from
/var/lib/jenkins/workspace/charm_lint_check/make-lint.798:
I: config.yaml: option config-flags has no default value
I: config.yaml: option instances-path has no default value
W: config.yaml: option disable-neutron-security-groups has no default value
I: config.yaml: option migration-auth-type has no default value

Full lint output: http://paste.ubuntu.com/8707142/
Build: http://10.98.191.181:8080/job/charm_lint_check/798/

Revision history for this message
Ryan Beisner (1chb1n) wrote :

UOSCI bot says:
charm_unit_test #606 trusty-nova-compute for corey.bryant mp239471
    UNIT OK: believed to pass, but you should confirm results

UNIT Results (max last 4 lines) from
/var/lib/jenkins/workspace/charm_unit_test/unit-test.606:
----------------------------------------------------------------------
Ran 56 tests in 3.373s

OK (SKIP=5)

Full unit output: http://paste.ubuntu.com/8707143/
Build: http://10.98.191.181:8080/job/charm_unit_test/606/

Revision history for this message
Liam Young (gnuoy) wrote :

lgtm

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches