[GitHub] cloudstack pull request: CLOUDSTACK-8923: Do not send zoneId with ...

Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/911#discussion_r41117252 --- Diff: server/src/com/cloud/network/StorageNetworkManagerImpl.java --- @@ -223,7 +225,7 @@ public StorageNetworkIpRange createIpRange(final Create

[GitHub] cloudstack pull request: Fix bugs on last step on Wizard zone and ...

GitHub user milamberspace opened a pull request: https://github.com/apache/cloudstack/pull/912 Fix bugs on last step on Wizard zone and Adv Search box Javascript error: Uncaught ReferenceError: _ is not defined => Missing 'l' (to call the _l() function) Bug introduce by

[GitHub] cloudstack pull request: Fix bugs on last step on Wizard zone and ...

Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/912#issuecomment-145455703 @remibergsma Probably this is a 'minor' blocker bug for the 4.6 release. --- If your project is set up for it, you can reply to this email and have your r

[GitHub] cloudstack pull request: CLOUDSTACK-8923: Do not send zoneId with ...

Github user borisroman commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/911#discussion_r41118074 --- Diff: server/src/com/cloud/network/StorageNetworkManagerImpl.java --- @@ -223,7 +225,7 @@ public StorageNetworkIpRange createIpRange(final CreateSt

[GitHub] cloudstack pull request: Dockerfile part2

Github user runseb commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/910#discussion_r41119897 --- Diff: tools/docker/Dockerfile.centos6 --- @@ -23,15 +23,19 @@ LABEL Vendor="Apache.org" License="ApacheV2" Version="4.6.0" ENV PKG_URL=http://jen

[GitHub] cloudstack pull request: Dockerfile part2

Github user runseb commented on the pull request: https://github.com/apache/cloudstack/pull/910#issuecomment-145461959 aside of the vhd-util download, +1 LGTM based on code review. Does not impact cloudstack core --- If your project is set up for it, you can reply to this email an

[GitHub] cloudstack pull request: Dockerfile part2

Github user terbolous commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/910#discussion_r41120279 --- Diff: tools/docker/Dockerfile.centos6 --- @@ -23,15 +23,19 @@ LABEL Vendor="Apache.org" License="ApacheV2" Version="4.6.0" ENV PKG_URL=http://

[GitHub] cloudstack pull request: Dockerfile part2

Github user runseb commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/910#discussion_r41120457 --- Diff: tools/docker/Dockerfile.centos6 --- @@ -23,15 +23,19 @@ LABEL Vendor="Apache.org" License="ApacheV2" Version="4.6.0" ENV PKG_URL=http://jen

[GitHub] cloudstack pull request: Dockerfile part2

Github user terbolous commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/910#discussion_r41121330 --- Diff: tools/docker/Dockerfile.centos6 --- @@ -23,15 +23,19 @@ LABEL Vendor="Apache.org" License="ApacheV2" Version="4.6.0" ENV PKG_URL=http://

[GitHub] cloudstack pull request: CLOUDSTACK-8915 - Cannot SSH into VMs dep...

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/908#issuecomment-145473615 Hi @remibergsma and @bvbharat, Please have a look at my comment from 2 days ago. I mentioned that indeed the change would create a regression. So thi

jenkins analysis and rat jobs

H, As you have undoubtetly noticed the jobs at builds.a.o no longer comment in our PRs after they are done. They do however edit the commitstatus which found hovering over a little check near the commit-id. It used to only record jenkins successtatus but since PR912 it also shows the builds.a.o st

Re: jenkins analysis and rat jobs

Thanks Daan. thats nice! Does it also show "in progress" state like travis? ~Rajani On Mon, Oct 5, 2015 at 3:25 PM, Daan Hoogland wrote: > H, > > As you have undoubtetly noticed the jobs at builds.a.o no longer comment in > our PRs after they are done. They do however edit the commitstatus whic

[GitHub] cloudstack pull request: CLOUDSTACK-8832 : Update Nuage VSP plugin...

Github user KrisSterckx commented on the pull request: https://github.com/apache/cloudstack/pull/801#issuecomment-145488778 I would like to bring attention to the reviewers that only a few CS core extensions were made - all other modifications are NuageVsp plugin internal, for Nuage V

[GitHub] cloudstack pull request: Dockerfile part2

Github user pdion891 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/910#discussion_r41133343 --- Diff: tools/docker/Dockerfile.centos6 --- @@ -23,15 +23,19 @@ LABEL Vendor="Apache.org" License="ApacheV2" Version="4.6.0" ENV PKG_URL=http://j

Re: jenkins analysis and rat jobs

I haven't been able to see that but let's keep an eye out On Mon, Oct 5, 2015 at 12:21 PM, Rajani Karuturi wrote: > Thanks Daan. thats nice! > Does it also show "in progress" state like travis? > > ~Rajani > > On Mon, Oct 5, 2015 at 3:25 PM, Daan Hoogland > wrote: > > > H, > > > > As you have u

[GitHub] cloudstack pull request: Dockerfile part2

Github user terbolous commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/910#discussion_r41142674 --- Diff: tools/docker/init.sh_centos6 --- @@ -17,27 +17,35 @@ # under the License. # # update database connection +# /usr/bin/cloudst

Re: jenkins analysis and rat jobs

Hi, That’s great! I’d asked infra about it because it makes Jenkins more visible. Great that it’s there now :-) Regards, Remi On 05/10/15 11:55, "Daan Hoogland" wrote: >H, > >As you have undoubtetly noticed the jobs at builds.a.o no longer comment in >our PRs after they are done. They do h

Adding Host Xen 6.5 on 4.5.2 issue

Hi All, Today I installed the CS 4.5 by configuring Advance Network while creating Zone after filling all the necessary information and clicked the Finish. I could see error while its connects to my xenserver 6.5 it throws error. My xenserver is added with box and updated the traffic label to.

Re: Adding Host Xen 6.5 on 4.5.2 issue

Do we need to apply below mail thread setting even in CS 4.5.2 xenserver bonding setup. http://mail-archives.apache.org/mod_mbox/cloudstack-dev/201403.mbox/%3cd8d70bcdfd3045a09f876df9a3259...@amxpr07mb117.eurprd07.prod.outlook.com%3E Thanks, Keerthi On Mon, Oct 5, 2015 at 8:01 PM, Keerthiraja SJ

[GitHub] cloudstack pull request: another typo that appears when monitoring...

GitHub user radu-stefanache opened a pull request: https://github.com/apache/cloudstack/pull/913 another typo that appears when monitoring the java process via JMX Status : in preogress instead of in progress and some other small typos . You can merge this pull request into a Git re

[GitHub] cloudstack pull request: Dockerfile part2

Github user runseb commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/910#discussion_r41171391 --- Diff: tools/docker/init.sh_centos6 --- @@ -17,27 +17,35 @@ # under the License. # # update database connection +# /usr/bin/cloudstack

[GitHub] cloudstack pull request: Dockerfile part2

Github user runseb commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/910#discussion_r41171193 --- Diff: tools/docker/Dockerfile.centos6 --- @@ -23,15 +23,19 @@ LABEL Vendor="Apache.org" License="ApacheV2" Version="4.6.0" ENV PKG_URL=http://jen

[GitHub] cloudstack pull request: CLOUDSTACK-8832 : Update Nuage VSP plugin...

Github user Pradeepgjain commented on the pull request: https://github.com/apache/cloudstack/pull/801#issuecomment-145624812 @jburwell For testing NuageVsp plugin on cloudstack, first nuage VSP needs to be installed: http://www.nuagenetworks.net/products/virtualized-

BVT report 10/5

Changes in code base since last BVT report - 10/1 - CLOUDSTACK-8848: ensure power state is up to date when handling missing VMs in powerReport - resmo / detail

[GitHub] cloudstack pull request: Fix bugs on last step on Wizard zone and ...

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/912#issuecomment-145655920 @milamberspace Thanks, this we should include indeed. Will try to test in the coming days. --- If your project is set up for it, you can reply to this email and

[GitHub] cloudstack pull request: Fix bugs on last step on Wizard zone and ...

Github user runseb commented on the pull request: https://github.com/apache/cloudstack/pull/912#issuecomment-145661229 @milamberspace @remibergsma there is a duplicate at https://github.com/apache/cloudstack/pull/906 --- If your project is set up for it, you can reply to this email a

[GitHub] cloudstack pull request: BUG-ID: CLOUDSTACK-8922: Unable to delete...

Github user runseb commented on the pull request: https://github.com/apache/cloudstack/pull/905#issuecomment-145661688 @yvsubhash we should try to remove manual testing. Can you write a marvin test for it. thanks. cc @wilderrodrigues --- If your project is set up for it, you can rep

[GitHub] cloudstack pull request: CLOUDSTACK-8929

Github user runseb commented on the pull request: https://github.com/apache/cloudstack/pull/904#issuecomment-145662123 @rags22489664 any comments based on the reviews ? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If y

[GitHub] cloudstack pull request: CLOUDSTACK-8928: While adding VMs to LB r...

Github user runseb commented on the pull request: https://github.com/apache/cloudstack/pull/903#issuecomment-145662476 @nitin-maharana Any automated tests for this ? if not, can you spend the time to write some. thanks --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack pull request: CLOUDSTACK-8919: Slow UI response while l...

Github user runseb commented on the pull request: https://github.com/apache/cloudstack/pull/897#issuecomment-145662988 thanks @nitin-maharana maybe we should have a small performance study to show that it actually speeds things up --- If your project is set up for it, you can reply t

Re: BVT report 10/5

Thanks Raja, couple additional comments in-line. > On Oct 5, 2015, at 9:08 PM, Raja Pullela wrote: > > Changes in code base since last BVT report - 10/1 > - CLOUDSTACK-8848: ensure power state is up to date when handling > missing VMs in powerReport - > resmo

[GitHub] cloudstack pull request: Fix bugs on last step on Wizard zone and ...

Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/912#issuecomment-145668698 @runseb Good catch! Thanks @nitin-maharana to fix part of this bugs. @remibergsma my PR add a bug on Adv search label. --- If your project is set up

Re: BVT report 10/5

Hi, I looked into the reboot of the SSVM and CPVM and can reproduce the failing tests. Created an issue: https://issues.apache.org/jira/browse/CLOUDSTACK-8933 The failure is caused by the fact that on reboot the systemvms are not patched (patchviasocket script), and at boot time they wait for

RE: BVT report 10/5

Oh cool, thanks Remi! This was so hard to debug, doesn't give any clues about what's happening. I will check this with Kishan or if there are any KVM experts, please chime in? -Original Message- From: Remi Bergsma [mailto:rberg...@schubergphilis.com] Sent: Tuesday, October 6, 2015 3

RE: BVT report 10/5

Hi Sebastian, - the LB tests failing on Advanced zone for both KVM and Xen - I will file a bug on this - thought there was a bug around this but can't find it. - on reboot SSVM/CPVM, glad that Remi was able to lead us onto something and I will pursue the next steps. - on internal Lb, will try th