nova packages do not include config files when building from github zipballs

Bug #1019423 reported by Doug Hellmann
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Critical
Julien Danjou
OpenStack Core Infrastructure
Fix Released
High
Monty Taylor

Bug Description

The ceilometer gating tests are failing (see below) because the copy of nova being installed does not include the sqlalchemy-migrate configuration file for the DB migrations (nova/db/sqlalchemy/migrate_repo/migrate.cfg). The file is present in git, and packages correctly when an sdist or bdist_egg build is run from a git checkout, but is not present in packages built from the github zipball (https://github.com/openstack/nova/zipball/master#egg=nova).

Example of failing test: https://jenkins.stackforge.org/view/Ceilometer/job/gate-ceilometer-python26/110/console

Changed in ceilometer:
assignee: nobody → Doug Hellmann (doug-hellmann)
status: New → In Progress
Monty Taylor (mordred)
Changed in openstack-ci:
assignee: nobody → Monty Taylor (mordred)
milestone: none → folsom
importance: Undecided → High
status: New → Triaged
Revision history for this message
Doug Hellmann (doug-hellmann) wrote :

As a temporary work-around we're listing the nova dependency for ceilometer as http://nova.openstack.org/tarballs/nova-2012.2~f2~20120629.14648.tar.gz instead of pointing to the github zipball.

Changed in ceilometer:
importance: Undecided → Critical
Revision history for this message
Doug Hellmann (doug-hellmann) wrote :

The work-around is working and ceilometer's gating tests are passing again.

Revision history for this message
Monty Taylor (mordred) wrote :

More permanent fix. Use this:

http://tarballs.openstack.org/nova/nova-master.tar.gz

Which will always be the latest tarball cut from nova master.

Changed in openstack-ci:
status: Triaged → Fix Released
Revision history for this message
StackForge Gerrit (gerrig) wrote : Fix proposed to ceilometer (master)

Fix proposed to branch: master
Review: https://review.stackforge.org/281

Changed in ceilometer:
assignee: Doug Hellmann (doug-hellmann) → Julien Danjou (jdanjou)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Fix proposed to branch: master
Review: https://review.openstack.org/9315

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ceilometer (master)

Reviewed: https://review.openstack.org/9315
Committed: http://github.com/stackforge/ceilometer/commit/fec72610abfbd32480dae73a93f566cbec050b5b
Submitter: Jenkins
Branch: master

commit fec72610abfbd32480dae73a93f566cbec050b5b
Author: Julien Danjou <email address hidden>
Date: Tue Jul 3 13:04:07 2012 +0200

    Reset the correct nova dependency URL

    This fixes bug #1019423

    Change-Id: I8dd36926aa3457a9b985b5b98d7201383ec03518

Changed in ceilometer:
status: In Progress → Fix Committed
Revision history for this message
Graham Binns (gmb) wrote :

Fix released in ceilometer 0.1/folsom.

Changed in ceilometer:
status: Fix Committed → Fix Released
Graham Binns (gmb)
Changed in ceilometer:
milestone: none → 0.1
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.