Merge lp://qastaging/~corey.bryant/charms/trusty/neutron-api/db-stamp into lp://qastaging/~openstack-charmers-archive/charms/trusty/neutron-api/next

Proposed by Corey Bryant
Status: Merged
Merged at revision: 60
Proposed branch: lp://qastaging/~corey.bryant/charms/trusty/neutron-api/db-stamp
Merge into: lp://qastaging/~openstack-charmers-archive/charms/trusty/neutron-api/next
Diff against target: 125 lines (+49/-15)
3 files modified
hooks/neutron_api_hooks.py (+1/-1)
hooks/neutron_api_utils.py (+20/-2)
unit_tests/test_neutron_api_utils.py (+28/-12)
To merge this branch: bzr merge lp://qastaging/~corey.bryant/charms/trusty/neutron-api/db-stamp
Reviewer Review Type Date Requested Status
Liam Young (community) Approve
Review via email: mp+238550@code.qastaging.launchpad.net
To post a comment you must log in.
58. By Corey Bryant

Stamp neutron database before upgrade.

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

UOSCI bot says:
This MP triggered a test on the Ubuntu OSCI system. Here is a summary of results.

#456 neutron-api-next for corey.bryant mp238550
charm_unit_test

This build:
http://10.98.191.181:8080/job/charm_unit_test/456/

MP URL:
https://code.launchpad.net/~corey.bryant/charms/trusty/neutron-api/db-stamp/+merge/238550

Proposed branch:
lp:~corey.bryant/charms/trusty/neutron-api/db-stamp

Results summary:
    UNIT OK: believed to pass, but you should confirm results

UNIT Results (max last 25 lines) from
/var/lib/jenkins/workspace/charm_unit_test/unit-test.456:
Starting tests...
.......................................................
Name Stmts Miss Cover Missing
---------------------------------------------------------
hooks/neutron_api_context 80 6 93% 22, 34-36, 78, 110
hooks/neutron_api_hooks 190 16 92% 120-121, 179-180, 193, 258, 303, 322-333, 408-409
hooks/neutron_api_utils 105 8 92% 216, 249-261
---------------------------------------------------------
TOTAL 375 30 92%
----------------------------------------------------------------------
Ran 55 tests in 3.681s

OK

Ubuntu OSCI Jenkins is currently in development on a Canonical private network, but we plan to publish results to a public instance soon. Tests are triggered if the proposed branch rev changes, or if the MP is placed into "Needs review" status after being otherwise for >= 1hr. Human review of results is still recommended.
http://10.98.191.181:8080/

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

UOSCI bot says:
This MP triggered a test on the Ubuntu OSCI system. Here is a summary of results.

#650 neutron-api-next for corey.bryant mp238550
charm_lint_check

This build:
http://10.98.191.181:8080/job/charm_lint_check/650/

MP URL:
https://code.launchpad.net/~corey.bryant/charms/trusty/neutron-api/db-stamp/+merge/238550

Proposed branch:
lp:~corey.bryant/charms/trusty/neutron-api/db-stamp

Results summary:
    LINT OK: believed to pass, but you should confirm results

LINT Results (max last 25 lines) from
/var/lib/jenkins/workspace/charm_lint_check/make-lint.650:
I: config.yaml: option ssl_key has no default value
I: config.yaml: option os-admin-network has no default value
I: config.yaml: option nsx-controllers has no default value
I: config.yaml: option vip has no default value
I: config.yaml: option nsx-l3-uuid has no default value
I: config.yaml: option ssl_ca has no default value
I: config.yaml: option ssl_cert has no default value
I: config.yaml: option os-internal-network has no default value
I: config.yaml: option os-public-network has no default value
I: config.yaml: option nsx-tz-uuid has no default value

Ubuntu OSCI Jenkins is currently in development on a Canonical private network, but we plan to publish results to a public instance soon. Tests are triggered if the proposed branch rev changes, or if the MP is placed into "Needs review" status after being otherwise for >= 1hr. Human review of results is still recommended.
http://10.98.191.181:8080/

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

UOSCI bot says:
This MP triggered a test on the Ubuntu OSCI system. Here is a summary of results.

#229 neutron-api-next for corey.bryant mp238550
charm_amulet_test

This build:
http://10.98.191.181:8080/job/charm_amulet_test/229/

MP URL:
https://code.launchpad.net/~corey.bryant/charms/trusty/neutron-api/db-stamp/+merge/238550

Proposed branch:
lp:~corey.bryant/charms/trusty/neutron-api/db-stamp

Results summary:
    AMULET FAIL: amulet-test missing

AMULET Results not found.

Ubuntu OSCI Jenkins is currently in development on a Canonical private network, but we plan to publish results to a public instance soon. Tests are triggered if the proposed branch rev changes, or if the MP is placed into "Needs review" status after being otherwise for >= 1hr. Human review of results is still recommended.
http://10.98.191.181:8080/

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

Approve

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