All nova upstart configuration use 'su' instead of 'start-stop-daemon'

Bug #1086833 reported by James Page
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
nova (Ubuntu)
Fix Released
Medium
Yolanda Robla
Raring
Fix Released
Medium
Yolanda Robla

Bug Description

The way the nova upstart configurations chuid/chgid is by using su; using start-stop-daemon is a much cleaner way and avoids creating user sessions on the installed system.

See quantum for reference.

ProblemType: Bug
DistroRelease: Ubuntu 12.10
Package: nova-compute (not installed)
ProcVersionSignature: Ubuntu 3.5.0-19.30-generic 3.5.7
Uname: Linux 3.5.0-19-generic x86_64
NonfreeKernelModules: fglrx
ApportVersion: 2.6.1-0ubuntu6
Architecture: amd64
Date: Wed Dec 5 15:08:12 2012
MarkForUpload: True
SourcePackage: nova
UpgradeStatus: Upgraded to quantal on 2012-12-04 (0 days ago)

Related branches

Revision history for this message
James Page (james-page) wrote :
Changed in nova (Ubuntu Raring):
assignee: nobody → Yolanda Robla (yolanda.robla)
James Page (james-page)
Changed in nova (Ubuntu Raring):
importance: Undecided → Medium
James Page (james-page)
Changed in nova (Ubuntu Raring):
status: New → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package nova - 2013.1~g2-0ubuntu1

---------------
nova (2013.1~g2-0ubuntu1) raring; urgency=low

  [ Chuck Short ]
  * New upstream release.
  * debian/patches/ubuntu-show-tests.patch: Dropped no longer needed.
  * debian/nova-xcp-plugins.install: Fix xcp-plugins empty packages
  * debian/control: Drop python-nose in favor or testrepository
  * debian/control: Add python-coverage as a build dep.
  * debian/rules, debian/control: Run pep8 tests.
  * debian/*.init: Remove they are not needed and take up space
  * debian/control, debian/nova-cells.{install, logrotate, upstart}: Add
    cells support.
  * debian/patches/fix-ubuntu-tests.patch: temporarily disable failing tests.
  * debian/control, debian/nova-baremetal.{install, logrotate, upstart}: Add
    nova baremetal support.
  * debian/control: Remove python-support.

  [ Adam Gandelman ]
  * debian/*.manpages: Install Sphinx-generated manpages instead of
    our own.
  * debian/nova-compute-*.conf: Specify the newly required compute_driver
    flag in addition to libvirt_type.
  * debian/control: Specify required python-webob and python-stevedore
    versions.

  [ Yolanda Robla ]
  * debian/*.upstart: Use start-stop-daemon instead of su for chuid
    (LP: #1086833).
  * debian/rules: Remove override of dh_installinit for discriminating
    between Debian and Ubuntu.
  * debian/nova-common.docs: Installing changelogs from rules
  * debian/rules: Replacing perms in /etc/nova/logging.conf for 0644
  * debian/control: adduser dependency on nova-compute.
  * debian/control: added section oldlibs and priority extra on
    nova-ajax-console-proxy.
  * debian/nova-xvpvncproxy.postrm: removing because of duplicates.

  [ James Page ]
  * d/control: Add ~ to python-sqlalchemy-ext versioned dependencies to
    make backporting easier.
  * d/control: Updated nova-volume description and depdendencies to
    mark it as a transitional package, moved to oldlibs/extra.
  * d/p/fix-libvirt-tests.patch: Dropped; accepted upstream.
  * d/control: Added python-stevedore to BD's.
  * d/*.postrm: Dropped postrm's that just run update-rc.d; this is not
    required when deploying upstart configurations only.
  * d/nova-scheduler.manpages: Add man page for nova-rpc-zmq-receiver.
  * d/rules: Install upstream changelog with a policy compliant name.
  * d/control: Mark nova-compute-xcp as virtual package.
  * d/control: nova-api-os-volume; Depend on cinder-api and mark as
    transitional package.
  * d/nova-api-os-volume.lintian-overrides: Dropped - no longer required.
 -- Chuck Short <email address hidden> Fri, 11 Jan 2013 13:06:56 -0600

Changed in nova (Ubuntu Raring):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.