Inconsistencies in tests

2015-04-14 Thread Gaurav Aradhye
Hi all, I feel that the test code in BVT and Regression tests (smoke and component folders) should be more consistent. There are feature developers adding tests into smoke and different test developers adding more tests into component folder, it is evident that inconsistencies are bound to aris

[GitHub] cloudstack pull request: Escone

2015-04-14 Thread manasaveloori
Github user manasaveloori closed the pull request at: https://github.com/apache/cloudstack/pull/160 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feat

RE: [ANNOUNCE] New committer: Gaurav Nandkumar Aradhye

2015-04-14 Thread Koushik Das
Congrats Gaurav! -Original Message- From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] Sent: Tuesday, 14 April 2015 1:14 To: dev Subject: [ANNOUNCE] New committer: Gaurav Nandkumar Aradhye The Project Management Committee (PMC) for Apache CloudStack has asked Gaurav Aradhya to become

Re: [ANNOUNCE] New committer: Funs Kessen

2015-04-14 Thread Wilder Rodrigues
Wt! Great news, dude! Congratulation and let’s keep coking! Cheers, Wilder On 13 Apr 2015, at 21:41, Daan Hoogland wrote: > The Project Management Committee (PMC) for Apache CloudStack > has asked Funs Kessen to become a committer and we are pleased to > announce that they have accepted.

RE: [ANNOUNCE] New committer: Funs Kessen

2015-04-14 Thread Koushik Das
Congrats Funs! -Original Message- From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] Sent: Tuesday, 14 April 2015 1:12 To: dev Subject: [ANNOUNCE] New committer: Funs Kessen The Project Management Committee (PMC) for Apache CloudStack has asked Funs Kessen to become a committer and we

[GitHub] cloudstack pull request: CLOUDSTACK-8374: Adding tests for guest t...

2015-04-14 Thread gauravaradhye
GitHub user gauravaradhye opened a pull request: https://github.com/apache/cloudstack/pull/166 CLOUDSTACK-8374: Adding tests for guest traffic port group verification - Shared and Isolated networks For Isolated network, network and VM is already created in setUpClass, so only addi

Re: [ANNOUNCE] New committer: Funs Kessen

2015-04-14 Thread Rohit Yadav
Congrats Funs! Fun to have Funs around! > On 14-Apr-2015, at 3:00 am, Sebastien Goasguen wrote: > > we are doomed :) > >> On Apr 13, 2015, at 10:05 PM, Erik Weber wrote: >> >> On Mon, Apr 13, 2015 at 9:41 PM, Daan Hoogland >> wrote: >> >>> The Project Management Committee (PMC) for Apache Cloud

Re: [ANNOUNCE] New committer: Gaurav Nandkumar Aradhye

2015-04-14 Thread Rohit Yadav
Congrats Gaurav! Now you can to merge your pending PRs yourself (less work for the rest of us :)). > On 14-Apr-2015, at 1:13 am, Daan Hoogland wrote: > > The Project Management Committee (PMC) for Apache CloudStack > has asked Gaurav Aradhya to become a committer and we are pleased to > announce

Re: cloudmonkey tls

2015-04-14 Thread Rohit Yadav
Hi Marcus, The issue is likely related to Python and/or older versions of requests (the python http library we use). The error is simply saying that SSLv3 is not entertained by the (mgmt) server. To fix this, you can: 1. Upgrade CloudMonkey (and its dependencies, if not using latest cloudmonke

[GitHub] cloudstack pull request: Upgrades from 4.4: disable foreign key ch...

2015-04-14 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/164#issuecomment-92712153 @remibergsma sure, let me test and merge this on 4.4, 4.5 and master --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack pull request: KVM: improve CentOS 7 support (ifconfig)

2015-04-14 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/165#issuecomment-92713667 @remibergsma thanks for the patch will test and merge. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] cloudstack pull request: Upgrades from 4.4: disable foreign key ch...

2015-04-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/164 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] cloudstack pull request: Upgrades from 4.4: disable foreign key ch...

2015-04-14 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/164#issuecomment-92714730 @terbolous @DaanHoogland normally I would oppose the change in db files, but since this only fixes for foreign key constraints and fixes upgrade issue. I'm going ahe

Jenkins build is back to normal : build-4.5-simulator #254

2015-04-14 Thread jenkins
See

Re: [ANNOUNCE] New committer: Gaurav Nandkumar Aradhye

2015-04-14 Thread Gaurav Aradhye
That’s true Rohit! Thanks! :-) Thank you Talluri, Ahmad, Punith, Koushik! Regards, Gaurav Aradhye On Apr 14, 2015, at 2:20 PM, Rohit Yadav wrote: > Congrats Gaurav! Now you can to merge your pending PRs yourself (less work > for the rest of us :)). > >> On 14-Apr-2015, at 1:13 am, Daan Hoogl

Build failed in Jenkins: simulator-singlerun #1103

2015-04-14 Thread jenkins
See Changes: [Rohit Yadav] disable foreign key checks when altering them -- [...truncated 10437 lines...] > Running query: drop database if exists `cloud_usage` >

Re: Inconsistencies in tests

2015-04-14 Thread Sebastien Goasguen
> On Apr 14, 2015, at 9:06 AM, Gaurav Aradhye > wrote: > > Hi all, > > I feel that the test code in BVT and Regression tests (smoke and component > folders) should be more consistent. There are feature developers adding tests > into smoke and different test developers adding more tests into

Jenkins build is still unstable: simulator-4.5-singlerun #202

2015-04-14 Thread jenkins
See

[GitHub] cloudstack pull request: KVM: improve CentOS 7 support (ifconfig)

2015-04-14 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/165#issuecomment-92774873 @remibergsma why not add the dependency to install "net-tools" to rpms? I've tested the patch with 4.5, it worked for me. but applying only on master now. Please adv

Build failed in Jenkins: simulator-singlerun #1104

2015-04-14 Thread jenkins
See Changes: [sanjay.tripathi] CLOUDSTACK-8386: CS is returning sec. storage even if it is full and there are other sec. storage with free space. -- [...truncated 10447 lines...] ==

[GitHub] cloudstack pull request: KVM: improve CentOS 7 support (ifconfig)

2015-04-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/165 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

Build failed in Jenkins: simulator-singlerun #1105

2015-04-14 Thread jenkins
See Changes: [Rohit Yadav] kvm: improve CentOS 7 support [Rohit Yadav] server: ignore params signatureversion and expires when validating API args -- [...truncated 10435 lines...]

RE: [ANNOUNCE] New committer: Gaurav Nandkumar Aradhye

2015-04-14 Thread Raja Pullela
Congratulations Gaurav! -Original Message- From: Rajani Karuturi [mailto:raj...@apache.org] Sent: Tuesday, April 14, 2015 9:10 AM To: dev@cloudstack.apache.org Subject: Re: [ANNOUNCE] New committer: Gaurav Nandkumar Aradhye congratulations Gaurav.. ~Rajani On Tue, Apr 14, 2015 at 1:3

[GitHub] cloudstack pull request: KVM: improve CentOS 7 support (ifconfig)

2015-04-14 Thread NuxRo
Github user NuxRo commented on the pull request: https://github.com/apache/cloudstack/pull/165#issuecomment-92782014 Do note there are slight differences between EL7 ifconfig output and EL6. Since we're making changes, +1 for "ip", ifconfig might not even be available at all in the

Re: [ANNOUNCE] New committer: Gaurav Nandkumar Aradhye

2015-04-14 Thread Gaurav Aradhye
Thanks Raja! Regards, Gaurav Aradhye On Apr 14, 2015, at 5:24 PM, Raja Pullela wrote: > Congratulations Gaurav! > > -Original Message- > From: Rajani Karuturi [mailto:raj...@apache.org] > Sent: Tuesday, April 14, 2015 9:10 AM > To: dev@cloudstack.apache.org > Subject: Re: [ANNOUNCE]

[GitHub] cloudstack pull request: KVM: improve CentOS 7 support (ifconfig)

2015-04-14 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/165#issuecomment-92789451 @NuxRo @remibergsma in that case should we merge this patch on 4.5 as well? I tested this with 4.5 advance zone with Ubuntu 14.04 host and it worked for me, though I

Missing storage label in XenCenter after migrate

2015-04-14 Thread S . Brüseke - proIO GmbH
Hi, can somebody please test this behavior in 4.5.1-rc2? 1. Create instance on XenServer hypervisor via CS. 2. Check "Name" of storage in XenCenter's storage tab for this instance. (should be ROOT-XXX) 3. Migrate primary storage to different storage via CS. 4. Take a look at "Name" of storage in

How do the committers generally merge the pull requests?

2015-04-14 Thread Gaurav Aradhye
Need to some advice from established committers :-) How do you merge the pull request? Through button on github? Or applying patch locally and then committing it? Where to add “This closes ###” comment exactly? Please help me out with steps! Regards, Gaurav Aradhye

Jenkins build is still unstable: simulator-4.5-singlerun #203

2015-04-14 Thread jenkins
See

question about JDBC and mysql

2015-04-14 Thread Jeronimo Garcia
Hi List. Does cloudstack uses transactions when writing to mysql ? i happen to see data and soon enough disappears , like if i would be using READ_UNCOMMITTED. The whole thing is that on a snapshot failure , the snapshot ID didn't get status of "Error" it simply vanished from the snapshots table

[GitHub] cloudstack pull request: CLOUDSTACK-8374: Adding tests for guest t...

2015-04-14 Thread gauravaradhye
Github user gauravaradhye closed the pull request at: https://github.com/apache/cloudstack/pull/166 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feat

[GitHub] cloudstack pull request: CLOUDSTACK-8374: Adding tests for guest t...

2015-04-14 Thread gauravaradhye
Github user gauravaradhye commented on the pull request: https://github.com/apache/cloudstack/pull/166#issuecomment-92815608 This is committed. Closing. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does

Re: How do the committers generally merge the pull requests?

2015-04-14 Thread Sebastien Goasguen
> On Apr 14, 2015, at 2:53 PM, Gaurav Aradhye > wrote: > > Need to some advice from established committers :-) > > How do you merge the pull request? Through button on github? Or applying > patch locally and then committing it? > Where to add “This closes ###” comment exactly? > > Please hel

Re: How do the committers generally merge the pull requests?

2015-04-14 Thread Gaurav Aradhye
Thanks Sebastien for detailed steps! Regards, Gaurav Aradhye On Apr 14, 2015, at 6:30 PM, Sebastien Goasguen wrote: > >> On Apr 14, 2015, at 2:53 PM, Gaurav Aradhye >> wrote: >> >> Need to some advice from established committers :-) >> >> How do you merge the pull request? Through button o

Re: How do the committers generally merge the pull requests?

2015-04-14 Thread Gaurav Aradhye
Should I add this to any existing/new wiki page? Regards, Gaurav Aradhye On Apr 14, 2015, at 6:34 PM, Gaurav Aradhye wrote: > Thanks Sebastien for detailed steps! > > Regards, > Gaurav Aradhye > > On Apr 14, 2015, at 6:30 PM, Sebastien Goasguen wrote: > >> >>> On Apr 14, 2015, at 2:53 PM,

Re: How do the committers generally merge the pull requests?

2015-04-14 Thread Daan Hoogland
I would actually review on github and then apply the git pull command line shown in the mail. Next I would test and push to wip-us... I documented this in the qa pages ( a while ago, might be up for revision) On Tue, Apr 14, 2015 at 8:07 AM, Gaurav Aradhye wrote: > Should I add this to any exist

[GitHub] cloudstack pull request: CLOUDSTACK-8377: Marvin - base library - ...

2015-04-14 Thread gauravaradhye
Github user gauravaradhye commented on the pull request: https://github.com/apache/cloudstack/pull/156#issuecomment-92825267 You are right Sanjeev. For all the other classes it is done this way, without using map, but sometimes we need to use the object methods and that are not availa

[GitHub] cloudstack pull request: CLOUDSTACK-8377: Marvin - base library - ...

2015-04-14 Thread gauravaradhye
Github user gauravaradhye closed the pull request at: https://github.com/apache/cloudstack/pull/156 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feat

Re: [GitHub] cloudstack pull request: CLOUDSTACK-8374: Adding tests for guest t...

2015-04-14 Thread Gaurav Aradhye
Sure Sebastien, I missed squashing. Thanks. Regards, Gaurav Aradhye On Apr 14, 2015, at 6:50 PM, runseb wrote: > Github user runseb commented on the pull request: > >https://github.com/apache/cloudstack/pull/166#issuecomment-92825534 > >in cases like this, make sure to squash commits.

[GitHub] cloudstack pull request: Make the routers iptables config persiste...

2015-04-14 Thread wilderrodrigues
GitHub user wilderrodrigues opened a pull request: https://github.com/apache/cloudstack/pull/167 Make the routers iptables config persistent through reboot/crash - After configuration save the ipdated in files * /etc/iptables/router_rules.v4 and /etc/iptables/router_rules.v

Build failed in Jenkins: simulator-singlerun #1106

2015-04-14 Thread jenkins
See Changes: [Gaurav Aradhye] CLOUDSTACK-8374: Adding tests for guest traffic port group verification - Shared and Isolated networks [Gaurav Aradhye] CLOUDSTACK-8374: Adding tests for guest traffic port group verification -

[GitHub] cloudstack pull request: CLOUDSTACK-8374: Adding tests for guest t...

2015-04-14 Thread runseb
Github user runseb commented on the pull request: https://github.com/apache/cloudstack/pull/166#issuecomment-92825534 in cases like this, make sure to squash commits. and if pep8 does not pass, then ask for revision to PR --- If your project is set up for it, you can reply to this em

[GitHub] cloudstack pull request: Make the routers iptables config persiste...

2015-04-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/167 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

Can you please give a edit permission to wiki?

2015-04-14 Thread Go Chiba
Hi guys, Can you please give a edit permission to wiki for below account? Account ID: smzksts He is one of the Japanese board member and we'd like to translate 2015 Plans that @sebgoa described to share with Japan users... from my iPhone

[VOTE] Apache Cloudstack 4.4.3

2015-04-14 Thread Daan Hoogland
Hi All, I've created a 4.4.3 release, with the following artifacts up for a vote: Git Branch and Commit SH: https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/4.4 Commit: f2826ac60af3d9aae508a505370e1b280cb6f599 List of changes: https://git-wip-us.apache.org/repos/a

Build failed in Jenkins: simulator-singlerun #1107

2015-04-14 Thread jenkins
See Changes: [wrodrigues] Make the routers persistent -- [...truncated 10451 lines...] > Processing SQL file at

Re: Can you please give a edit permission to wiki?

2015-04-14 Thread Daan Hoogland
On Tue, Apr 14, 2015 at 10:22 AM, Go Chiba wrote: > smzksts is in -- Daan

RE: [ANNOUNCE] New committer: Funs Kessen

2015-04-14 Thread Paul Angus
Awesome. Congratulations Funs! Regards Paul Angus Cloud Architect S: +44 20 3603 0540 | M: +447711418784 | T: CloudyAngus paul.an...@shapeblue.com -Original Message- From: Rohit Yadav [mailto:rohit.ya...@shapeblue.com] Sent: 14 April 2015 09:49 To: dev@cloudstack.apache.org Subject: Re:

Build failed in Jenkins: simulator-singlerun #1108

2015-04-14 Thread jenkins
See Changes: [wido] CLOUDSTACK-8313: Allow overprovisioning of local storage pools -- [...truncated 10435 lines...] > Running query: drop database if exists `cloud_usage` ===

[ANNOUNCE] pull request builder

2015-04-14 Thread Daan Hoogland
people, there is now a pull request builder active at builds.apache.org. It will detect and build pull requests. It is not yet enabled for simulator or regression tests but those are next steps. regards, -- Daan

Re: [DISCUSS] 4.4.3 rc

2015-04-14 Thread Daan Hoogland
btw, closing this vote and probably the next as well as Remi want more KVM fixes back ported. On Thu, Apr 9, 2015 at 6:00 AM, Wido den Hollander wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > > > On 04/09/2015 12:26 PM, Remi Bergsma wrote: >> Hi Daan, >> >> If your doing another RC,

Re: [ANNOUNCE] New committer: Funs Kessen

2015-04-14 Thread Ahmad Emneina
Nice work Funs! Congrats. On Tue, Apr 14, 2015 at 8:27 AM, Paul Angus wrote: > Awesome. Congratulations Funs! > > Regards > > Paul Angus > Cloud Architect > S: +44 20 3603 0540 | M: +447711418784 | T: CloudyAngus > paul.an...@shapeblue.com > > -Original Message- > From: Rohit Yadav [mai

Re: Missing storage label in XenCenter after migrate

2015-04-14 Thread raja sekhar
Bruseke, When the disks are live migrated from one primary to another this problem is arising, i'am also getting same problem, the disk labels are changed to "Cloud-".it seems to be a bug. how ever it doesn't happens when the vm is down. On Tue, Apr 14, 2015 at 6:11 PM, S. Brüseke - proIO GmbH <

RE: [ANNOUNCE] pull request builder

2015-04-14 Thread Stephen Turner
Nice! -- Stephen Turner -Original Message- From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] Sent: 14 April 2015 17:07 To: dev Subject: [ANNOUNCE] pull request builder people, there is now a pull request builder active at builds.apache.org. It will detect and build pull requests.

RE: [ANNOUNCE] New committer: Gaurav Nandkumar Aradhye

2015-04-14 Thread Somesh Naidu
Congratulations Gaurav! Regards, Somesh -Original Message- From: Gaurav Aradhye [mailto:gaurav.arad...@clogeny.com] Sent: Tuesday, April 14, 2015 8:08 AM To: dev@cloudstack.apache.org Subject: Re: [ANNOUNCE] New committer: Gaurav Nandkumar Aradhye Thanks Raja! Regards, Gaurav Aradhye

RE: [ANNOUNCE] New committer: Funs Kessen

2015-04-14 Thread Somesh Naidu
Congratulations Funs! Regards, Somesh -Original Message- From: Ahmad Emneina [mailto:aemne...@gmail.com] Sent: Tuesday, April 14, 2015 12:45 PM To: dev@cloudstack.apache.org Subject: Re: [ANNOUNCE] New committer: Funs Kessen Nice work Funs! Congrats. On Tue, Apr 14, 2015 at 8:27 AM, P

[GitHub] cloudstack pull request: KVM: improve CentOS 7 support (ifconfig)

2015-04-14 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/165#issuecomment-92988843 @bhaisaab we even want this in 4.4 @SBP as we are experimenting with KVM. --- If your project is set up for it, you can reply to this email and have your reply a

Re: Can you please give a edit permission to wiki?

2015-04-14 Thread Go Chiba
Thanks! from my iPhone 2015/04/15 0:33、Daan Hoogland のメッセージ: >> On Tue, Apr 14, 2015 at 10:22 AM, Go Chiba wrote: >> smzksts > > > is in > > -- > Daan

RE: question about JDBC and mysql

2015-04-14 Thread Somesh Naidu
Yes, transactions are used and AFAIK, transaction level is set to READ COMMITED. You would benefit from setting the logging level to TRACE to get more insight as most of the logging at the DAO layer is set to TRACE. Somesh CloudPlatform Escalations Citrix Systems, Inc. -Original Message---

[GitHub] cloudstack pull request: Make the routers iptables config persiste...

2015-04-14 Thread resmo
Github user resmo commented on the pull request: https://github.com/apache/cloudstack/pull/167#issuecomment-93027681 There is a Debian package for this functionality called `iptables-persistent`. It is implemented using a init script. The rules are expected to be in `/etc/ipta

[GitHub] cloudstack pull request: Make the routers iptables config persiste...

2015-04-14 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/167#issuecomment-93040925 Hi Rene, I tried to get it working with iptables-persistent package but spent about 2 days and it took me nowhere. I can send an email with s

[GitHub] cloudstack pull request: Make the routers iptables config persiste...

2015-04-14 Thread resmo
Github user resmo commented on the pull request: https://github.com/apache/cloudstack/pull/167#issuecomment-93054007 hi @wilderrodrigues, never had any problems with it but didn't use it on a "router". It would be great if you can provide more details. I expected you didn't know about

KVM plugs public nic twice in non-VPC router

2015-04-14 Thread Remi Bergsma
Hi, We’re testing KVM and noticed that a virtual router (non-VPC) gets two pubic ip nics plugged. This is in 4.4.2 when using Open vSwitch on CentOS 7 (advanced zone). The reason for this, is that the first nic gets plugged by provisioning the nic. Then, a second nic is plugged that gets the s

Re: [DISCUSS] 4.4.3 rc

2015-04-14 Thread Nux!
Sure, go right ahead. -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro - Original Message - > From: "Daan Hoogland" > To: "dev" > Sent: Tuesday, 14 April, 2015 17:17:30 > Subject: Re: [DISCUSS] 4.4.3 rc > btw, closing this vote and probably the next as well as Re

Re: [DISCUSS] 4.4.3 rc

2015-04-14 Thread Marcus
Who don't we release the 4.4.3 that was voted on? I think bugfix releases should be small and often, there's no reason to wait to roll a new set of fixes if we have a release that fixes bugs and has passed vote, is there? On Tue, Apr 14, 2015 at 2:25 PM, Nux! wrote: > Sure, go right ahead. > > --

Re: [DISCUSS] 4.4.3 rc

2015-04-14 Thread Daan Hoogland
the first one was ignored. since then we had reports about issues. I don't generally care except for the nil-database-upgrade code we need to write. On Tue, Apr 14, 2015 at 4:43 PM, Marcus wrote: > Who don't we release the 4.4.3 that was voted on? I think bugfix > releases should be small and oft

Re: [DISCUSS] 4.4.3 rc

2015-04-14 Thread Remi Bergsma
Hi Marcus, I totally agree with you: we need fast releases often! But let’s not release this when people won’t be able to upgrade to 4.5 and up. Since that fix went in, I asked Daan to merge some KVM stuff that was already fixed as well. We should create a new release today, if possible. I’ll w

[GitHub] cloudstack pull request: CLOUDSTACK-6697: bigswitch networking plu...

2015-04-14 Thread kwanggithub
Github user kwanggithub commented on the pull request: https://github.com/apache/cloudstack/pull/151#issuecomment-93120538 Thanks to all who helped shepherd this effort! Happy to see this merged! @karuturi , absolutely - I will fix the find bug issues, together with many PMD issues

[GitHub] cloudstack pull request: findbugs and PMD fixes

2015-04-14 Thread kwanggithub
GitHub user kwanggithub opened a pull request: https://github.com/apache/cloudstack/pull/168 findbugs and PMD fixes findbugs and PMD fixes You can merge this pull request into a Git repository by running: $ git pull https://github.com/kwanggithub/cloudstack feature_bigswitch A

Re: [ANNOUNCE] pull request builder

2015-04-14 Thread Rajani Karuturi
wonderful!! colors like travis would be nice :) ~Rajani On Tue, Apr 14, 2015 at 9:36 PM, Daan Hoogland wrote: > people, > > there is now a pull request builder active at builds.apache.org. It > will detect and build pull requests. It is not yet enabled for > simulator or regression tests but t

Re: [ANNOUNCE] pull request builder

2015-04-14 Thread Abhinandan Prateek
Daan, I have been fiddling with simulator and provisioning tests recently, ping me in case I can take up some related tasks. -abhi > On 14-Apr-2015, at 9:36 pm, Daan Hoogland wrote: > > people, > > there is now a pull request builder active at builds.apache.org. It > will detect and build pull

[GitHub] cloudstack pull request: Make the routers iptables config persiste...

2015-04-14 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/167#issuecomment-93194201 Hi @resmo On the cherry pick: I'm afraid it cannot be done. Why? Besides the IP tables we now have also the whole router configuration (e.g. guest ne

[GitHub] cloudstack pull request: Make the routers iptables config persiste...

2015-04-14 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/167#issuecomment-93206215 @resmo @wilderrodrigues I would avoid cherry-picking this feature on 4.5; AFAIK if a router reboots, cloudstack should re-apply any iptables rules and patch it again

[GitHub] cloudstack pull request: CLOUDSTACK-8335: use libvirt from central...

2015-04-14 Thread bhaisaab
Github user bhaisaab commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/163#discussion_r28395092 --- Diff: plugins/hypervisors/kvm/pom.xml --- @@ -45,7 +38,7 @@ ${project.version} - org.libvirt + com.gi

[GitHub] cloudstack pull request: findbugs and PMD fixes

2015-04-14 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/168#issuecomment-93206517 I can see more than 200 issues fixed after this patch on master. I will merge it in sometime --- If your project is set up for it, you can reply to this email and h

[GitHub] cloudstack pull request: Make the routers iptables config persiste...

2015-04-14 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/167#issuecomment-93206684 If the router is restarted from ACS management server, yes, it will apply all setting. However, if one connects to the router and reboots it - or the router c

[GitHub] cloudstack pull request: findbugs and PMD fixes

2015-04-14 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/168#issuecomment-93207386 @kwanggithub Thanks a lot for taking care of these issues. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] cloudstack pull request: findbugs and PMD fixes

2015-04-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/168 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] cloudstack pull request: findbugs and PMD fixes

2015-04-14 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/168#issuecomment-93207332 @kwanggithub thanks for the fixes, merged on master now. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as we

Re: [ANNOUNCE] New committer: Funs Kessen

2015-04-14 Thread Ian Duffy
Congratulations Funs! :) On 14 April 2015 at 18:17, Somesh Naidu wrote: > Congratulations Funs! > > Regards, > Somesh > > > -Original Message- > From: Ahmad Emneina [mailto:aemne...@gmail.com] > Sent: Tuesday, April 14, 2015 12:45 PM > To: dev@cloudstack.apache.org > Subject: Re: [ANNOUNC

[GitHub] cloudstack pull request: Issue:Domian admin can view templates in ...

2015-04-14 Thread manasaveloori
GitHub user manasaveloori opened a pull request: https://github.com/apache/cloudstack/pull/169 Issue:Domian admin can view templates in other domains list templates by domain admin is listing all the templates of another domains as well. This test will verify that a domain admin

[GitHub] cloudstack pull request: Issue:Domian admin can view templates in ...

2015-04-14 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/169#discussion_r28395596 --- Diff: test/integration/component/test_escalation_listTemplatesDomainAdmin1.py --- @@ -0,0 +1,94 @@ + --- End diff -- Please inc

[GitHub] cloudstack pull request: Issue:Domian admin can view templates in ...

2015-04-14 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/169#discussion_r28395604 --- Diff: test/integration/component/test_escalation_listTemplatesDomainAdmin1.py --- @@ -0,0 +1,94 @@ + +from marvin.cloudstackTestCase imp

[GitHub] cloudstack pull request: Issue:Domian admin can view templates in ...

2015-04-14 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/169#discussion_r28395630 --- Diff: test/integration/component/test_escalation_listTemplatesDomainAdmin1.py --- @@ -0,0 +1,94 @@ + +from marvin.cloudstackTestCase imp

[GitHub] cloudstack pull request: Issue:Domian admin can view templates in ...

2015-04-14 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/169#discussion_r28395658 --- Diff: test/integration/component/test_escalation_listTemplatesDomainAdmin1.py --- @@ -0,0 +1,94 @@ + +from marvin.cloudstackTestCase imp

[GitHub] cloudstack pull request: CLOUDSTACK-8335: use libvirt from central...

2015-04-14 Thread kishankavala
Github user kishankavala commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/163#discussion_r28395781 --- Diff: plugins/hypervisors/kvm/pom.xml --- @@ -45,7 +38,7 @@ ${project.version} - org.libvirt + co

[GitHub] cloudstack pull request: Issue:Domian admin can view templates in ...

2015-04-14 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/169#discussion_r28395789 --- Diff: test/integration/component/test_escalation_listTemplatesDomainAdmin1.py --- @@ -0,0 +1,94 @@ + +from marvin.cloudstackTestCase imp

[GitHub] cloudstack pull request: Issue:Domian admin can view templates in ...

2015-04-14 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/169#discussion_r28395840 --- Diff: test/integration/component/test_escalation_listTemplatesDomainAdmin1.py --- @@ -0,0 +1,94 @@ + +from marvin.cloudstackTestCase imp

[GitHub] cloudstack pull request: Issue:Domian admin can view templates in ...

2015-04-14 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/169#discussion_r28395900 --- Diff: test/integration/component/test_escalation_listTemplatesDomainAdmin1.py --- @@ -0,0 +1,94 @@ + +from marvin.cloudstackTestCase imp

[GitHub] cloudstack pull request: Issue:Domian admin can view templates in ...

2015-04-14 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/169#discussion_r28395963 --- Diff: test/integration/component/test_escalation_listTemplatesDomainAdmin1.py --- @@ -0,0 +1,94 @@ + +from marvin.cloudstackTestCase imp

4.5 clustered mgmt server

2015-04-14 Thread Marcus
Is anyone running 4.5 mgmt server in cluster setup? I've run into an issue where the channel between the two servers is null, and I think I've tracked it down to "coverity fixes" from 78bfaa79cf3eaa170ef9422bf8fb1825c0cecfc1. The method returns 'ch', which is null in the error. The coverity fix ch