nova-manage providing filename to default_flagfile breaks devstack

[Scott Moser]

Ubuntu is carrying a patch nova-manage_flagfile_location.patch, for quite some time.
(timestamp: Tue 2010-11-23 12:17:09 +0100)

I've just noticed that if you try to run out of a source directory with that patch applied and without specifying --flagfile, that you will essentially get no flag file. This is different than the default case, where running like './bin/nova' will use './bin/nova.conf'.

Obviously the ./bin/nova.conf path is somewhat silly, but many dev scripts (devstack, nova.sh) expect that that file will be used when invoking nova.

As a result, I can't run 'devstack' right now.

ProblemType: Bug
DistroRelease: Ubuntu 11.10
Package: nova-common (not installed)
ProcVersionSignature: User Name 3.0.0-12.19-server 3.0.4
Uname: Linux 3.0.0-12-server x86_64
ApportVersion: 1.23-0ubuntu2
Architecture: amd64
Date: Sat Oct 8 00:37:00 2011
Ec2AMI: ami-000000ac
Ec2AMIManifest: FIXME
Ec2AvailabilityZone: nova
Ec2InstanceType: m1.large
Ec2Kernel: unavailable
Ec2Ramdisk: unavailable
ProcEnviron:
 PATH=(custom, user)
 LANG=en_US.UTF-8
 SHELL=/bin/bash
SourcePackage: nova
UpgradeStatus: No upgrade log present (probably fresh install)

[Scott Moser (smoser)]

I suggest we replace the current 'nova-manage_flagfile_location.patch' with this one.

We should also try to get this upstream.
The change here is basically to try to use /etc/nova/nova.conf only if it exists and is readable.
While that still will break you if you're running nova-manage from a source tree as root on a system with nova installed, i find that scenario unlikely.

[Ubuntu Foundations Team Bug Bot (crichton)]

The attachment "suggested patch" of this bug report has been identified as being a patch. The ubuntu-reviewers team has been subscribed to the bug report so that they can review the patch. In the event that this is in fact not a patch you can resolve this situation by removing the tag 'patch' from the bug report and editing the attachment so that it is not flagged as a patch. Additionally, if you are member of the ubuntu-sponsors please also unsubscribe the team from this bug report.

[This is an automated message performed by a Launchpad user owned by Brian Murray. Please contact him regarding any issues with the action taken in this bug report.]

[Dave Walker (davewalker)]

@smoser, this patch targets developers rather than consumers right?

That being the case, as the Ubuntu nova packages are releasing soon - is this patch of more use in P-Series?

Ie, what is the impact if this doesn't land in Oneiric?

Thanks.

[Scott Moser (smoser)]

Dave,
  I hit this when trying to use an ubuntu git branch as the source of the nova component for devstack.
  Because of this bug, i wasted time figuring out what was going wrong, rather than spending that time figuring out whether or not an issue with trunk applied to Ubuntu. It is probably sufficient to have this fixed in 'P', as there will no doubt be other issues using devstack against a diablo release (as devstack will concentrate on functioning with trunk I would guess).

[Launchpad Janitor (janitor)]

This bug was fixed in the package nova - 2012.1~e1~20111020.11229-0ubuntu1

---------------
nova (2012.1~e1~20111020.11229-0ubuntu1) precise; urgency=low

  * New upstream release.
  * Dropped patches, already applied upstream:
    - debian/patches/backport-iscsitarget-choice.patch
    - debian/patches/backport-libvirt-console-pipe.patch
    - debian/patches/backport-lxc-container-console-fix.patch
    - debian/patches/backport-recreate-gateway-using-dhcp.patch
    - debian/patches/backport-snapshot-cleanup.patch
    - debian/patches/block-migration-needs-copy-backingfile.patch
    - debian/patches/fix-iscsi-target-path.patch
    - debian/patches/fix-lp838581-removed-db_pool-complexities.patch
    - debian/patches/fix-lp863305-images-permission.patch
    - debian/patches/fqdn-in-local-hostname-of-ec2-metadata.patch
    - debian/patches/use-netcat-instead-of-socat.patc
   * debian/control:
     - Add python-feedparser as a build dependency.
     - Bump standards version to 3.9.2
     - Point to the essex branch.
   * debian/patches/nova-manage_flagfile_location.patch:
     Update patch to take in account of devstack (LP: #870405)
 -- Chuck Short <email address hidden> Fri, 21 Oct 2011 14:37:26 -0400