Re: Redundant VPC Virtual Router's state is BACKUP & BACKUP

2015-11-01 Thread Remi Bergsma
Thanks, we will look into this tomorrow! Regards, Remi Sent from my iPhone > On 01 Nov 2015, at 03:48, giraffeg forestg wrote: > > Hi all. > > My environment: > > CloudStack master(2015/10/31) 4.6.0-snapshot > > Hypervisor CentOS6/KVM > > SystemVM > build #654

[GitHub] cloudstack pull request: CLOUDSTACK-8825, CLOUDSTACK-8824 : Fixed ...

2015-11-01 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/787#discussion_r43580727 --- Diff: engine/schema/src/com/cloud/capacity/dao/CapacityDaoImpl.java --- @@ -98,9 +98,9 @@ private static final String

Re: Redundant VPC Virtual Router's state is BACKUP & BACKUP

2015-11-01 Thread Nux!
CC-ing dev ml. -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro - Original Message - > From: "giraffeg forestg" > To: us...@cloudstack.apache.org > Sent: Sunday, 1 November, 2015 02:48:17 > Subject: Redundant VPC Virtual Router's state is

[GitHub] cloudstack pull request: CLOUDSTACK-8825, CLOUDSTACK-8824 : Fixed ...

2015-11-01 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/787#issuecomment-152812742 @anshul1886 for an integration test only a definition of an environment with zwps that can be evaluated by marvin is needed. All the use cases are covered by

Nexus repository dependencies

2015-11-01 Thread Ramzi Youssefi
Hi, I was trying to use Nexus as a local repository for cloudstack deployment. Everything went fine until this happened: [WARNING] The POM for net.juniper.contrail:juniper-contrail-api:jar:1.0-SNAPSHOT is missing, no dependency information available I have tried everything to make it work but

New Defects reported by Coverity Scan for cloudstack

2015-11-01 Thread scan-admin
Hi, Please find the latest report on new defect(s) introduced to cloudstack found with Coverity Scan. 1 new defect(s) introduced to cloudstack found with Coverity Scan. 2 defect(s), reported by Coverity Scan earlier, were marked fixed in the recent build analyzed by Coverity Scan. New

[GitHub] cloudstack pull request: CLOUDSTACK-8825, CLOUDSTACK-8824 : Fixed ...

2015-11-01 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/787#issuecomment-152849621 LGTM, both scenario's tested manually. --- 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

Build failed in Jenkins: simulator-4.5-singlerun #275

2015-11-01 Thread jenkins
See -- [...truncated 109 lines...] Started by an SCM change Started by an SCM change Started by an SCM change Started by an SCM change Started by an SCM change Started by an SCM

[GitHub] cloudstack pull request: Merge pull request #1 from apache/master

2015-11-01 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1017#issuecomment-152850429 @bprakg is this a mistake? can you close it? --- 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: CID 1338016: multi catch instead of pokem...

2015-11-01 Thread DaanHoogland
GitHub user DaanHoogland opened a pull request: https://github.com/apache/cloudstack/pull/1018 CID 1338016: multi catch instead of pokemon You can merge this pull request into a Git repository by running: $ git pull https://github.com/DaanHoogland/cloudstack CID-1338016

[GitHub] cloudstack pull request: Merge pull request #1 from apache/master

2015-11-01 Thread bprakg
Github user bprakg commented on the pull request: https://github.com/apache/cloudstack/pull/1017#issuecomment-152856278 Sorry Daan, it was a mistake, closing it. --- 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

[GitHub] cloudstack pull request: Merge pull request #1 from apache/master

2015-11-01 Thread bprakg
Github user bprakg closed the pull request at: https://github.com/apache/cloudstack/pull/1017 --- 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

[GitHub] cloudstack pull request: Use java.io.tmpdir instead of hardcoded /...

2015-11-01 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1012#issuecomment-152861124 The value is different for each user; the JVM will return the value from %TEMP environment variable. To give you an example, in my pc the returned

[GitHub] cloudstack pull request: Merge pull request #1 from apache/master

2015-11-01 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1017#issuecomment-152859902 np, human! --- 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

[GitHub] cloudstack pull request: findbugs: ! equals instead of !=

2015-11-01 Thread DaanHoogland
GitHub user DaanHoogland opened a pull request: https://github.com/apache/cloudstack/pull/1019 findbugs: ! equals instead of != simple pmd warning fixed You can merge this pull request into a Git repository by running: $ git pull https://github.com/DaanHoogland/cloudstack PMD

[GitHub] cloudstack pull request: Use java.io.tmpdir instead of hardcoded /...

2015-11-01 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1012#issuecomment-152861243 I noticed you have already changed the code as a suggested. I can give you now a LGTM. --- If your project is set up for it, you can reply to this

[GitHub] cloudstack pull request: findbugs: ! equals instead of !=

2015-11-01 Thread karuturi
Github user karuturi commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1019#discussion_r43601287 --- Diff: plugins/network-elements/bigswitch/src/com/cloud/network/bigswitch/BigSwitchBcfApi.java --- @@ -296,7 +296,7 @@ private void

[GitHub] cloudstack pull request: findbugs: ! equals instead of !=

2015-11-01 Thread resmo
Github user resmo commented on the pull request: https://github.com/apache/cloudstack/pull/1019#issuecomment-152939410 LGTM --- 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

[GitHub] cloudstack pull request: CLOUDSTACK-8647 : LDAP: Auto Import and T...

2015-11-01 Thread sarathkouk
Github user sarathkouk commented on the pull request: https://github.com/apache/cloudstack/pull/919#issuecomment-152932441 @remibergsma : Yes, it is. I made all the changes you suggested. Please go through and let me know if it looks good. --- If your project is set up for it,

[GitHub] cloudstack pull request: CLOUDSTACK-8860: improve error messages i...

2015-11-01 Thread bvbharatk
Github user bvbharatk commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/864#discussion_r43601603 --- Diff: engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java --- @@ -1090,8 +1091,8 @@ public void orchestrateStart(final String

[GitHub] cloudstack pull request: CLOUDSTACK-8860: improve error messages i...

2015-11-01 Thread bvbharatk
Github user bvbharatk commented on the pull request: https://github.com/apache/cloudstack/pull/864#issuecomment-152940799 @wido, sorry for the late response, was busy with other work. I have replied to your question inline. @remibergsma Thanks for taking time and

[GitHub] cloudstack pull request: findbugs: ! equals instead of !=

2015-11-01 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1019#discussion_r43602513 --- Diff: plugins/network-elements/bigswitch/src/com/cloud/network/bigswitch/BigSwitchBcfApi.java --- @@ -296,7 +296,7 @@ private void