Merge lp:~soren/nova/get-console-log-fixes into lp:~hudson-openstack/nova/trunk
Proposed by
Soren Hansen
Status: | Merged | ||||||||
---|---|---|---|---|---|---|---|---|---|
Approved by: | Devin Carlen | ||||||||
Approved revision: | 357 | ||||||||
Merged at revision: | 361 | ||||||||
Proposed branch: | lp:~soren/nova/get-console-log-fixes | ||||||||
Merge into: | lp:~hudson-openstack/nova/trunk | ||||||||
Diff against target: |
23 lines (+4/-5) 1 file modified
nova/virt/libvirt_conn.py (+4/-5) |
||||||||
To merge this branch: | bzr merge lp:~soren/nova/get-console-log-fixes | ||||||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Devin Carlen (community) | Approve | ||
Eric Day (community) | Approve | ||
Review via email: mp+38600@code.launchpad.net |
Description of the change
Fix two problems with get_console_log:
* libvirt has this annoying "feature" where it chown()s your console to the uid running libvirt. That gets in the way of reading it. Add a call to "sudo chown ...." right before we read it to make sure it works out well.
* We were looking in the wrong directory for console.log. *blush*
To post a comment you must log in.
lgtm!