OpenStack Compute (nova)

Merge lp://staging/~vishvananda/nova/fix-packages into lp://staging/~openstack-ubuntu-packagers/ubuntu/natty/nova/ubuntu

  1. fix-packages
  2. Merge into ubuntu
Proposed by Vish Ishaya
Status: Merged
Merge reported by: Vish Ishaya
Merged at revision: not available
Proposed branch: lp://staging/~vishvananda/nova/fix-packages
Merge into: lp://staging/~openstack-ubuntu-packagers/ubuntu/natty/nova/ubuntu
Diff against target: 85 lines (+49/-0)
5 files modified
debian/control (+24/-0)
debian/nova-vncproxy.install (+1/-0)
debian/nova-vncproxy.logrotate (+4/-0)
debian/nova-vncproxy.upstart (+19/-0)
debian/nova_sudoers (+1/-0)
To merge this branch: bzr merge lp://staging/~vishvananda/nova/fix-packages
Reviewer Review Type Date Requested Status
Devin Carlen (community) Needs Information
Review via email: mp+58851@code.staging.launchpad.net

Description of the change

Fixes bug on volume delete because /bin/dd isn't in sudoers. Also adds packages for nova-vncproxy

I didn't update the changelog because i don't know if it is my responsiblity or the person who merges it.

To post a comment you must log in.
Revision history for this message
Devin Carlen (devcamcar) wrote :

this looks good, but shouldn't it be two separate patches? (dd and nova-vncproxy are unrelated)

review: Needs Information
Revision history for this message
Vish Ishaya (vishvananda) wrote :

Yes, probably, but they both seemed important, and as there is no automerge, I figured soren/monty could look at both together

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