tools/conf/generate_sample.sh seems to be broken

Bug #1182611 reported by Morgan Fainberg
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Undecided
Anton Frolov

Bug Description

Trying to generate a new sample config (to reflect changes in options) doesn't seem to work properly. There are many options that are missing from the new sample config when the script is run.

The script reports no errors and has no logging so there is no direct indication of what might be wrong with it at this time.

Revision history for this message
Eric Harney (eharney) wrote :

There is a substantial delta between Cinder's extract_opts.py and Nova's -- probably need to start there and see if we can sync their changes over.

Changed in cinder:
assignee: nobody → Avishay Traeger (avishay-il)
status: New → Confirmed
Changed in cinder:
assignee: Avishay Traeger (avishay-il) → nobody
Anton Frolov (anton0)
Changed in cinder:
assignee: nobody → Anton Frolov (anton0)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

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

Changed in cinder:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.openstack.org/31180
Committed: http://github.com/openstack/cinder/commit/27e261376fc03655a5f90603078a3a8623ddf32f
Submitter: Jenkins
Branch: master

commit 27e261376fc03655a5f90603078a3a8623ddf32f
Author: Anton Frolov <email address hidden>
Date: Fri May 31 14:27:49 2013 +0400

    Add error reporting to generate_sample.sh on import failure.

    Currently tools/conf/generate_sample.sh collect options from project
    modules by importing each module and looking at it's global variables
    for instances of classes derived from cfg.Opt (or lists of such
    instances). Sometimes this import can fail due to various reasons,
    cfg.DuplicateOptError for example. And if import fails no options
    from imported module written to sample config file. Error reporting
    will at least warn user that some of options didn't get into sample
    config.

    Change-Id: I831464b1c1575c106d866a936a50ec74b2d44e59
    Fixes: bug #1182611

Changed in cinder:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in cinder:
milestone: none → havana-2
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in cinder:
milestone: havana-2 → 2013.2
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.