Merge lp://staging/~openstack-ubuntu-packagers/glance/fix-sql-connection into lp://staging/~openstack-ubuntu-packagers/glance/ubuntu

lgtm

Looks good.

I don't think it's extraneous: http://www.sqlalchemy.org/docs/dialects/sqlite.html (search for "four slashes").

If it doesn't match the default config, then that's what needs fixing, probably.

Hey Soren,

You are correct. Good catch.

After a bit more investigation it looks like we have a couple things going that were causing my problems.

Yesterday the glance deb package source was changed so that it actually tries to run 'glance-manage db_sync' during an installation (bzr 29). On a clean machine this is actually causes an error.

See this ticket for details:

https://bugs.launchpad.net/glance/+bug/778463

In any case I no longer think we need this. At least not without changing the default string in Glance as well.

Dan

2011/5/6 Dan Prince <email address hidden>:
> Yesterday the glance deb package source was changed so that it actually tries to run 'glance-manage db_sync' during an installation (bzr 29). On a clean machine this is actually causes an error.
>
> See this ticket for details:
>
> https://bugs.launchpad.net/glance/+bug/778463

That looks completely unrelated. I'll fix that up right away.

> In any case I no longer think we need this.

Oh? What changed?

--
Soren Hansen        | http://linux2go.dk/
Ubuntu Developer    | http://www.ubuntu.com/
OpenStack Developer | http://www.openstack.org/

One more thing to note. Maybe the point of confusion here is that patches/sql_conn.patch uses the extra slash where the default glance.conf.sample file in the actual glance source tree doesn't.

2011/5/6 Dan Prince <email address hidden>:
> In any case I no longer think we need this. At least not without changing the default string in Glance as well.

Having read this again, I'm not sure at all what "this" refers to here?

--
Soren Hansen        | http://linux2go.dk/
Ubuntu Developer    | http://www.ubuntu.com/
OpenStack Developer | http://www.openstack.org/

> 2011/5/6 Dan Prince <email address hidden>:
> > In any case I no longer think we need this. At least not without changing
> the default string in Glance as well.
>
> Having read this again, I'm not sure at all what "this" refers to here?

By 'this' I meant the change from 4 to 3 slashes. It looks like Jay was trying to make this match what is in the glance source tree which uses 3 slashes.

So I agree with you in that we should change the glance source to match what we do in the PPA packages using 4 slashes.

OK, no problem. We can change the Glance etc/ configs. Sorry, I
thought 3 slashes was correct, not 4. My mistake.

On Fri, May 6, 2011 at 8:49 AM, Dan Prince <email address hidden> wrote:
>> 2011/5/6 Dan Prince <email address hidden>:
>> > In any case I no longer think we need this. At least not without changing
>> the default string in Glance as well.
>>
>> Having read this again, I'm not sure at all what "this" refers to here?
>
> By 'this' I meant the change from 4 to 3 slashes. It looks like Jay was trying to make this match what is in the glance source tree which uses 3 slashes.
>
> So I agree with you in that we should change the glance source to match what we do in the PPA packages using 4 slashes.
> --
> https://code.launchpad.net/~openstack-ubuntu-packagers/glance/fix-sql-connection/+merge/60070
> Your team OpenStack Ubuntu packagers is subscribed to branch lp:~openstack-ubuntu-packagers/glance/ubuntu.
>