Merge lp://qastaging/~kirkland/pyjuju/byobu-tmux into lp://qastaging/pyjuju

Proposed by Dustin Kirkland 
Status: Merged
Approved by: Gustavo Niemeyer
Approved revision: 314
Merged at revision: 318
Proposed branch: lp://qastaging/~kirkland/pyjuju/byobu-tmux
Merge into: lp://qastaging/pyjuju
Diff against target: 66 lines (+12/-4)
4 files modified
ensemble/control/debug_hooks.py (+9/-2)
ensemble/providers/common/launch.py (+1/-0)
ensemble/providers/orchestra/tests/data/bootstrap_late_command (+1/-1)
ensemble/providers/orchestra/tests/data/launch_late_command (+1/-1)
To merge this branch: bzr merge lp://qastaging/~kirkland/pyjuju/byobu-tmux
Reviewer Review Type Date Requested Status
Gustavo Niemeyer Approve
William Reade (community) Approve
Review via email: mp+71285@code.qastaging.launchpad.net

Description of the change

Use more familiar keybindings and Ubuntu branding for the tmux-based debug hooks

To post a comment you must log in.
Revision history for this message
William Reade (fwereade) wrote :

All looks good to me, +1

review: Approve
Revision history for this message
Gustavo Niemeyer (niemeyer) wrote :

As long as this was tested in a real run, it looks good to me as well.

Thanks for keeping up with it Dustin.

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

to status/vote changes: