[GitHub] cloudstack pull request: This branch implements the CSV and native...

2015-07-19 Thread devdeep
Github user devdeep commented on the pull request: https://github.com/apache/cloudstack/pull/351#issuecomment-122773398 I'll take a look and get back if I have any queries --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. I

Re: [GitHub] cloudstack pull request: CLOUDSTACK-8632: Unable to revert snapsho...

2015-07-19 Thread Sanjeev N
LGTM On Thu, Jul 16, 2015 at 11:34 AM, wilderrodrigues wrote: > Github user wilderrodrigues commented on the pull request: > > https://github.com/apache/cloudstack/pull/590#issuecomment-121843137 > > LGTM > > > --- > If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack pull request: boring resourse leaks again

2015-07-19 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/602#issuecomment-122759444 one of the travis jobs timedout. builds.a.o is down to see what caused the failures code changes looks good :+1: --- If your project is set up for it, you can

Build failed in Jenkins: build-master-slowbuild #1854

2015-07-19 Thread jenkins
See -- Started by timer [EnvInject] - Loading node environment variables. Building remotely on coohq-slave-01 (cloudstack-buildslave-centos6 systemvmbuilder) in workspace

Build failed in Jenkins: build-master-slowbuild #1853

2015-07-19 Thread jenkins
See -- Started by timer [EnvInject] - Loading node environment variables. Building remotely on coohq-slave-01 (cloudstack-buildslave-centos6 systemvmbuilder) in workspace

[GitHub] cloudstack pull request: Dockerfile

2015-07-19 Thread pdion891
GitHub user pdion891 opened a pull request: https://github.com/apache/cloudstack/pull/605 Dockerfile Initial PR for Dockerization of CloudStack. Important Changes: * scripts/vm/systemvm/injectkeys.sh: script execution will pass event if /dev/loop0 not detected and wi

Re: [QUESTION] SQL in old upgrade code

2015-07-19 Thread Daan Hoogland
nevermind, I was reading over an extra underscore ('_')...:( On Sun, Jul 19, 2015 at 4:10 PM, Daan Hoogland wrote: > devs, > > in Upgrade218to22 in private void modifyIndexes(Connection conn), an > index is removed twice without any code inbetween. Does anybody know > of a (my)sql quirk that make

[QUESTION] SQL in old upgrade code

2015-07-19 Thread Daan Hoogland
devs, in Upgrade218to22 in private void modifyIndexes(Connection conn), an index is removed twice without any code inbetween. Does anybody know of a (my)sql quirk that makes this necessary? I am trying to remove the last high impact coverity warning so we can use coverity in our review process. A