Merge lp://qastaging/~d-jj/ironic/ironic-conductor into lp://qastaging/ubuntu/vivid/ironic

Proposed by JJ Asghar
Status: Superseded
Proposed branch: lp://qastaging/~d-jj/ironic/ironic-conductor
Merge into: lp://qastaging/ubuntu/vivid/ironic
Diff against target: 23 lines (+7/-2) (has conflicts)
1 file modified
debian/ironic-conductor.init.in (+7/-2)
Text conflict in debian/ironic-conductor.init.in
To merge this branch: bzr merge lp://qastaging/~d-jj/ironic/ironic-conductor
Reviewer Review Type Date Requested Status
JJ Asghar (community) Approve
Mark Vanderwiel Pending
Ma Wen Cheng Pending
Review via email: mp+257295@code.qastaging.launchpad.net

This proposal supersedes a proposal from 2015-03-16.

This proposal has been superseded by a proposal from 2015-04-27.

Description of the change

The ironic conductor init script doesn't point to the conductor application, it points to the ironic-api. This fixes that.

To post a comment you must log in.
Revision history for this message
Mark Vanderwiel (vanderwl) wrote : Posted in a previous version of this proposal

Lgtm, thx JJ

review: Approve
Revision history for this message
Ma Wen Cheng (mars914) wrote : Posted in a previous version of this proposal

it is a good fix

review: Approve
Revision history for this message
JJ Asghar (d-jj) :
review: Approve

Unmerged revisions

14. By JJ Asghar

Updated the conductor.init script to look at the conductor instead of the api

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