Merge lp://qastaging/~ddellav/charms/trusty/keystone/ssl-spec-fix into lp://qastaging/~openstack-charmers-archive/charms/trusty/keystone/next

Proposed by David Della Vecchia
Status: Merged
Merged at revision: 190
Proposed branch: lp://qastaging/~ddellav/charms/trusty/keystone/ssl-spec-fix
Merge into: lp://qastaging/~openstack-charmers-archive/charms/trusty/keystone/next
Diff against target: 52 lines (+9/-2)
2 files modified
hooks/keystone_hooks.py (+3/-0)
unit_tests/test_keystone_hooks.py (+6/-2)
To merge this branch: bzr merge lp://qastaging/~ddellav/charms/trusty/keystone/ssl-spec-fix
Reviewer Review Type Date Requested Status
David Ames (community) Approve
Corey Bryant Pending
Review via email: mp+274952@code.qastaging.launchpad.net

Description of the change

Fixing SSL deploy issue with missing system user.

To post a comment you must log in.
Revision history for this message
uosci-testing-bot (uosci-testing-bot) wrote :

charm_lint_check #12170 keystone-next for ddellav mp274952
    LINT OK: passed

Build: http://10.245.162.77:8080/job/charm_lint_check/12170/

Revision history for this message
uosci-testing-bot (uosci-testing-bot) wrote :

charm_unit_test #11296 keystone-next for ddellav mp274952
    UNIT OK: passed

Build: http://10.245.162.77:8080/job/charm_unit_test/11296/

Revision history for this message
uosci-testing-bot (uosci-testing-bot) wrote :

charm_amulet_test #7438 keystone-next for ddellav mp274952
    AMULET OK: passed

Build: http://10.245.162.77:8080/job/charm_amulet_test/7438/

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

Take note, the amulet test doesn't exercise SSL features.

Revision history for this message
David Ames (thedac) wrote :

Approved

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