Re: note to devcloud users on db.properties.override

2013-11-01 Thread Darren Shepherd
Yeah, I see there was a commit a3cec3802c4c56173c33ccc1e08af059547fe90e that broke it again. I'll reverted that commit and ran the following to sanity test and its seems to be working now mvn -T 1.5C -DskipTests clean install chmod +x plugins/network-elements/juniper-contrail/test/resources/mysql

Re: note to devcloud users on db.properties.override

2013-11-01 Thread Pedro Roque Marques
Darren, On Oct 31, 2013, at 9:27 AM, Darren Shepherd wrote: > This is fixed now. Maybe i'm doing something really dumb... but i still see the problem. I do see the commit afaf637 in the branch history. Steps to reproduce: 1. plugins/network-elements/juniper-contrail/test/resources/mysql_db_st

Re: note to devcloud users on db.properties.override

2013-10-31 Thread Hugo Trippaers
On 31 okt. 2013, at 17:27, Darren Shepherd wrote: > This is fixed now. Sorry about that. I really do go through extra > length to test my changes. I'm finding it difficult to ensure quality > because there is so much variation. You have to test everything in so > many contexts. It shouldn't

Re: note to devcloud users on db.properties.override

2013-10-31 Thread Darren Shepherd
This is fixed now. Sorry about that. I really do go through extra length to test my changes. I'm finding it difficult to ensure quality because there is so much variation. You have to test everything in so many contexts. It shouldn't be like that I hope to help reduced the amount of variation

Re: note to devcloud users on db.properties.override

2013-10-31 Thread Darren Shepherd
Ugh, I see what I did now. When I tested that code I changed the code I didn't do a "mvn install" before running "mvn ... deploydb." That's annoying, now I want to figure out why maven doesn't pickup changes unless I do an install. So I effectively tested with the old code. Darren On Thu, Oct

Re: note to devcloud users on db.properties.override

2013-10-31 Thread Darren Shepherd
I'll fix that. Gimme ten mintues. I specifically looked at that code and thought I didn't change the behavior, but I guess I screwed it up. Just a general comment. There's too many ways to do the same thing in CloudStack. Especially the database. The way databases are setup for developers sho

note to devcloud users on db.properties.override

2013-10-31 Thread Hugo Trippaers
Heya, With commit 1460196496d73e0db25c7beb2392cfaf9d591ed7 Darren improved the loading of the db.properties file, however this also affected the DatabaseCreator used by the deploydb procedure to refresh the database. Effectively the db.properies.override is ignored at the moment and the db.pro