There are several barclamps that still have sqlite code in them.  When we 
decided not to support sqlite, I believe it was just removed from the GUI, and 
the back end code was left in - at least in some cases.

I'm all up for removing it, but it will of course require extra work.


Chris


-----Original Message-----
From: crowbar-bounces On Behalf Of Kanevsky, Arkady
Sent: Friday, May 31, 2013 11:22 AM
To: rha...@suse.de; crowbar
Subject: Re: [Crowbar] SUSE - review cinder and glance pull requests?

Ralf,
Thanks for catching it.
We should remove remnants of sqlite.
Will look into it after transition to database barclamp.
Please, file trello card on it so we will not lose track of  it.
Thanks,
Arkady

-----Original Message-----
From: crowbar-bounces On Behalf Of Ralf Haferkamp
Sent: Friday, May 31, 2013 7:27 AM
To: crowbar
Subject: Re: [Crowbar] SUSE - review cinder and glance pull requests?

On Wed, May 29, 2013 at 03:52:47PM +0200, Vincent Untz wrote:
> Le mercredi 29 mai 2013, à 03:44 -0500, wayne_al...@dell.com a écrit :
> > Ralf, Vincent, could someone give a review to these pending pull requests?
> 
> They look good to me, but I'll let Ralf double-check since I have 
> nearly no experience with the database barclamp.
There is one thing was I wondering about when reviewing those pull requests.
Do we really still care for supporting sqlite based setups?

There is currenlty code in the glance barclamp that silently falls back to 
using sqlite instead of just giving a useful error when something is wrong with 
the Database Barclamp.

--
Ralf

_______________________________________________
Crowbar mailing list
Crowbar@dell.com
https://lists.us.dell.com/mailman/listinfo/crowbar
For more information: http://crowbar.github.com/

_______________________________________________
Crowbar mailing list
Crowbar@dell.com
https://lists.us.dell.com/mailman/listinfo/crowbar
For more information: http://crowbar.github.com/

_______________________________________________
Crowbar mailing list
Crowbar@dell.com
https://lists.us.dell.com/mailman/listinfo/crowbar
For more information: http://crowbar.github.com/

Reply via email to