glance-combined needs --sql-connection option

[Rick Harris]

Cactus now has the use_optparse blueprint merged in. It looks like the --filesystem-store-data option is referenced but never scraped from the CLI options.

We need to add the Optparse.Option in `glance-api` and `glance-combined`.

Also, when we add ConfigParser, we'll need to make sure this gets populated.

[Rick Harris (rconradharris)]

While we're at it, looks like glance-combined needs --sql-connection (glance-registry already has it).

[Jay Pipes (jaypipes)]

Nice catch.

On Tue, Feb 1, 2011 at 1:08 PM, Rick Harris <email address hidden> wrote:
> While we're at it, looks like glance-combined needs --sql-connection
> (glance-registry already has it).
>
> --
> You received this bug notification because you are a member of Glance
> Bug Team, which is subscribed to Glance.
> https://bugs.launchpad.net/bugs/711385
>
> Title:
>  Use_optparse lacks --filesystem-store-datadir
>

[Jay Pipes (jaypipes)]

The --filesystem-store-datadir is indeed present in glance-api and glance-connection. Do a --help to see.

Check out the call to glance.store.add_options() in glance-api and glance-combined. That function loops through the stores and asks each if they want to add any options to the parser.

Could you verify this bug again?

[Jay Pipes (jaypipes)]

s/glance-connection/glance-combined/

[Rick Harris (rconradharris)]

Oops, strike the --filesystem-store-data issue; I see it defined in filesystem.add_options.

Leaving this tix around since -sql-connection still looks like a legit issue.