Re: Redundant Router Interfaces

2016-04-01 Thread Wei ZHOU
Hi Dean, the VRRP instance will be set to eth0 (guest device) in router, and eth2 (the first guest device ) in vpc router. I think it is good. -Wei 2016-04-02 0:22 GMT+02:00 Dean Close : > Hi Wei, > > It looks like your patch will prevent redundant routers from bringing up > public interfaces.

[GitHub] cloudstack pull request: maven: Upgrade dependency versions

2016-04-01 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1397#issuecomment-204648798 @swill can you try again with oracle java 1.7.0_80 there are known certificate and cipher suite issues causing build issues when using 1.7.0_90+. JDK8 is not

[GitHub] cloudstack pull request: speedup iptables setup

2016-04-01 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1449#issuecomment-204634361 The debut of `upr`, so let me know if you want me to change the format. Also if you want me to add more tests in future runs, please let me know... Given the

[GitHub] cloudstack pull request: speedup iptables setup

2016-04-01 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1449#issuecomment-204633944 ### CI Results ### 84/85 SUCCESSFUL **1 Failure** (this and another similar test seem to fail consistently in my CI currently)

[GitHub] cloudstack pull request: CLOUDSTACK-9333: Exclude clusters from OV...

2016-04-01 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1457#issuecomment-204624408 @nvazquez, I find the “AlertManager” class terrible for that. Its name gives the idea of something that manages alerts and not clusters parameters

Re: Redundant Router Interfaces

2016-04-01 Thread Dean Close
Hi Wei, It looks like your patch will prevent redundant routers from bringing up public interfaces. We also need to: 1. shift the VRRP instance over to eth0 as eth2 will be down on the backup. 2. Ensure that eth0 and eth1 are up on both redundant virtual routers. Your patch might already solve

[GitHub] cloudstack pull request: BUG-ID:CLOUDSTACK-9331:added code in marv...

2016-04-01 Thread cristofolini
Github user cristofolini commented on the pull request: https://github.com/apache/cloudstack/pull/1458#issuecomment-204588967 Shouldn't the path shown in line 55 of your config file be OS-independent? I'd also suggest fixing the typo in the file name. At least I believe it was mean

[GitHub] cloudstack pull request: CLOUDSTACK-9323: Fix cancel host maintena...

2016-04-01 Thread cristofolini
Github user cristofolini commented on the pull request: https://github.com/apache/cloudstack/pull/1454#issuecomment-20457 @abhinandanprateek Looks like jenkins found some problems with your index. I'd suggest rebasing against the current master making all the needed merges and pus

[GitHub] cloudstack pull request: Cloudstack-8611:Handle SSH if server "for...

2016-04-01 Thread GabrielBrascher
Github user GabrielBrascher commented on the pull request: https://github.com/apache/cloudstack/pull/1459#issuecomment-204571209 Correcting myself, the history from the SshHelper (https://github.com/likitha/cloudstack/commits/CLOUDSTACK-8611/utils/src/com/cloud/utils/ssh/SshHelper.java

Re: Redundant Router Interfaces

2016-04-01 Thread Wei ZHOU
actually not only this issue, but more (1) restart vpc router will break the network if there are multiple tiers in the vpc. (2) check_heartbeat.sh does not work. I will create tickets for them next Monday, and create PR on github. -Wei 2016-04-01 21:14 GMT+02:00 Wei ZHOU : > Dean, > > I just f

[GitHub] cloudstack pull request: speedup iptables setup

2016-04-01 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1449#issuecomment-204537583 @remibergsma The test have already been started. :) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. I

[GitHub] cloudstack pull request: speedup iptables setup

2016-04-01 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1449#issuecomment-204532720 @kollyma Please squash the commits. @swill Let's run integration tests on this before merge. --- If your project is set up for it, you can reply to this ema

Re: Hooking into the SecurityGroups

2016-04-01 Thread Will Stevens
Oh sorry, I misunderstood what you needed to do. The events stream may be able to give you a way to hook in and have some code called on Create and Delete, but you are right, it is probably not the best approach for what you need to do. *Will STEVENS* Lead Developer *CloudOps* *| *Cloud Solution

Re: Redundant Router Interfaces

2016-04-01 Thread Wei ZHOU
Dean, I just fixed it yesterday. the commit is --- diff --git a/systemvm/patches/debian/config/opt/cloud/bin/cs/CsAddress.py b/systemvm/patches/debian/config/opt/cloud/bin/cs/CsAddress.py index 5f63c06..5256d03 100755 --- a/systemvm/patches/debian/config/opt/cloud/bin/cs/CsAddress.py +++ b/syst

[GitHub] cloudstack pull request: maven: Upgrade dependency versions

2016-04-01 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1397#issuecomment-204527828 I get the same behavior with Java 7. ``` $ mvn --version Apache Maven 3.0.5 (Red Hat 3.0.5-16) Maven home: /usr/share/maven Java version: 1.7.0

[GitHub] cloudstack pull request: maven: Upgrade dependency versions

2016-04-01 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1397#issuecomment-204520010 I have changed the way my CI environment is being built, so it is built to use Java 7 now. I am testing this PR now, so I will update with status when I have more...

[GitHub] cloudstack pull request: speedup iptables setup

2016-04-01 Thread resmo
Github user resmo commented on the pull request: https://github.com/apache/cloudstack/pull/1449#issuecomment-204500797 LGTM! please merge --- 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

Redundant Router Interfaces

2016-04-01 Thread Dean Close
Hi guys, I had been investigating a possible bug with the way interfaces are managed on virtual routers. The public interfaces are being brought up on backup routers and (because they boot second) they arp the IPs away from the master. I'd been examining an idea for a fix but whilst doing so I

[GitHub] cloudstack pull request: maven: Upgrade dependency versions

2016-04-01 Thread jburwell
Github user jburwell commented on the pull request: https://github.com/apache/cloudstack/pull/1397#issuecomment-204494886 @swill Java8 is not currently supported by the management server. We know that the process will collapse after running for 24-48 hours. Part of the issue is that

[GitHub] cloudstack pull request: maven: Upgrade dependency versions

2016-04-01 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1397#issuecomment-204490175 There were quite a few changes to the way certificates are handled in this PR. Changes to versions of bouncycastle as well as changing the way the Pem files are hand

[GitHub] cloudstack pull request: maven: Upgrade dependency versions

2016-04-01 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1397#issuecomment-204486265 What about this PR forces me to downgrade to Java 7? My Java 8 build environment has been working fine against master until this PR. --- If your project is set up

[GitHub] cloudstack pull request: Cloudstack-8611:Handle SSH if server "for...

2016-04-01 Thread GabrielBrascher
Github user GabrielBrascher commented on the pull request: https://github.com/apache/cloudstack/pull/1459#issuecomment-204484447 @DaanHoogland thanks for the hint, but the problem is not related to line ending. The problem is that there are two (2) different SshHelper classes

[GitHub] cloudstack pull request: maven: Upgrade dependency versions

2016-04-01 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1397#issuecomment-204474193 @swill based on the output you've shared from your CI env -- `Java version: 1.8.0_77, vendor: Oracle Corporation`; you're using Java 8. Please switch to Java 7 (op

[GitHub] cloudstack pull request: maven: Upgrade dependency versions

2016-04-01 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1397#issuecomment-204470246 The above results are from my CI environment which is running the following: ``` $ mvn --version Apache Maven 3.0.5 (Red Hat 3.0.5-16) Maven home: /usr/

[GitHub] cloudstack pull request: maven: Upgrade dependency versions

2016-04-01 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1397#issuecomment-204472158 Here is the core dump I had originally when I tried to build on my mac. [hs_err_pid38330.log.zip](https://github.com/apache/cloudstack/files/200015/hs_err_pid

Re: Hooking into the SecurityGroups

2016-04-01 Thread Nux!
Will, What I want to do is enable some sort of accounting based on iptables counters (I only want to measure traffic to certain destinations, i.e. not count inter-vm traffic in a SG zone). I doubt it's something the events stream can help, but it's a good thing to be aware of nonetheless. Than

Re: Hooking into the SecurityGroups

2016-04-01 Thread Patrick Dube
They are the same events that get pushed to the cloud.event MySQL table. So you get vm creation, vm deletion, etc. Most API driven actions will end up in that table. Using the RabbitMQ configuration, I got it to push the events successfully, but haven't expanded on it yet. Soon™ On Fri, Apr 1, 201

[GitHub] cloudstack pull request: CLOUDSTACK-9333: Exclude clusters from OV...

2016-04-01 Thread nvazquez
Github user nvazquez commented on the pull request: https://github.com/apache/cloudstack/pull/1457#issuecomment-204431280 @rafaelweingartner thanks for reviewing! I couldn't find any document with all cluster_details, however we could find out that this property could be defined like

[GitHub] cloudstack pull request: maven: Upgrade dependency versions

2016-04-01 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1397#issuecomment-204429261 @swill I'm guessing an environment issue, likely an empty java keystore; what is the version of your maven, java? Are you using Oracle Java or OpenJDK? Thi

Re: Hooking into the SecurityGroups

2016-04-01 Thread Will Stevens
Patrick, did you guys end up doing anything with these events? I know we were messing with that at one point. Did we learn anything interesting? Is it a viable solution for what Nux is trying to do? *Will STEVENS* Lead Developer *CloudOps* *| *Cloud Solutions Experts 420 rue Guy *|* Montreal *|

Re: Hooking into the SecurityGroups

2016-04-01 Thread Patrick Dube
@swill. You can configure CloudStack to push it's events to both mysql and RabbitMQ or Kafka ( http://docs.cloudstack.apache.org/projects/cloudstack-administration/en/4.8/events.html ) On Fri, Apr 1, 2016 at 8:37 AM, Will Stevens wrote: > Pierre-Luc may have gotten something like this working. I

[GitHub] cloudstack pull request: CLOUDSTACK-9298: Improve performance of r...

2016-04-01 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1425#issuecomment-204397261 Yes I meant to post the results, but I was too quick trying to get thing setup for the next run and I blew away my environment. :( Im a noob... I will be posting t

[GitHub] cloudstack pull request: maven: Upgrade dependency versions

2016-04-01 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1397#issuecomment-204396178 This is failing to build in my CI environment. The following tests are causing the build to fail. It is failing here: ``` [INFO] Apache CloudStack Fr

[GitHub] cloudstack pull request: CLOUDSTACK-9298: Improve performance of r...

2016-04-01 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1425#issuecomment-204382745 @swill Congrats on the first bubble-tested PR :-) I would recommend posting the results though, the results.txt file shows which tests were run. If later we wan

Re: Hooking into the SecurityGroups

2016-04-01 Thread Will Stevens
Pierre-Luc may have gotten something like this working. I think he tried something similar. I think you can do something like redirect the events to redis and then with a tool that uses the API and redis, build an event based overlay. I would have to look into this deeper, but I think PLD did som

Re: Hooking into the SecurityGroups

2016-04-01 Thread Nux!
Thanks a lot Jayapal! Will, that sounds nice but I have not seen anything like this. Another interesting thing might be - as Wido suggested - to use libvirt filters instead of our python scripts and I believe libvirt has it's own way of hooking stuff into it. Could be another thing to explore,

Re: Hooking into the SecurityGroups

2016-04-01 Thread Will Stevens
Slightly off topic, but relevant. Ideally we could easily hook into the event logging and build added logic by simply tying them to specific events. This would limit the hackery and would provide a system that others could use without having to change the core. Has anyone done something like this?

Re: Hooking into the SecurityGroups

2016-04-01 Thread Jayapal Uradi
Hi Nux, You need to look at the vmops file for xenserver and security_group.py for the kvm. Look at the below methods in these file. destroy_network_rules_for_vm, default_network_rules. Thanks, jayapal On 01-Apr-2016, at 4:12 pm, Nux! mailto:n...@li.nux.ro>> wrote: Hi, I want to hook into

Hooking into the SecurityGroups

2016-04-01 Thread Nux!
Hi, I want to hook into the SGs and add a few iptables rules every time a VM is spawned and delete them when the VM is moved/deleted. Has anyone done this before? Any pointers before I go and butcher it? :-) Lucian -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro

[GitHub] cloudstack pull request: BUG-ID:CLOUDSTACK-9331:added code in marv...

2016-04-01 Thread cloudsadhu
Github user cloudsadhu commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1458#discussion_r58188259 --- Diff: tools/marvin/marvin/config/Baremtal_AdvZone_Vmware.cfg --- @@ -0,0 +1,191 @@ +{ +"zones": [ +{ +"name

[GitHub] cloudstack pull request: BUG-ID:CLOUDSTACK-9331:added code in marv...

2016-04-01 Thread cloudsadhu
Github user cloudsadhu commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1458#discussion_r58188294 --- Diff: tools/marvin/marvin/config/Baremtal_AdvZone_Vmware.cfg --- @@ -0,0 +1,191 @@ +{ +"zones": [ +{ +"name

[GitHub] cloudstack pull request: BUG-ID:CLOUDSTACK-9331:added code in marv...

2016-04-01 Thread cloudsadhu
Github user cloudsadhu commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1458#discussion_r58188213 --- Diff: tools/marvin/marvin/config/Baremtal_AdvZone_Vmware.cfg --- @@ -0,0 +1,191 @@ +{ +"zones": [ +{ +"name

[GitHub] cloudstack pull request: BUG-ID:CLOUDSTACK-9331:added code in marv...

2016-04-01 Thread shwetaag
Github user shwetaag commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1458#discussion_r58182115 --- Diff: tools/marvin/marvin/config/Baremtal_AdvZone_Vmware.cfg --- @@ -0,0 +1,191 @@ +{ +"zones": [ +{ +"name":

[GitHub] cloudstack pull request: BUG-ID:CLOUDSTACK-9331:added code in marv...

2016-04-01 Thread shwetaag
Github user shwetaag commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1458#discussion_r58181661 --- Diff: tools/marvin/marvin/config/Baremtal_AdvZone_Vmware.cfg --- @@ -0,0 +1,191 @@ +{ +"zones": [ +{ +"name":

[GitHub] cloudstack pull request: BUG-ID:CLOUDSTACK-9331:added code in marv...

2016-04-01 Thread shwetaag
Github user shwetaag commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1458#discussion_r58180544 --- Diff: tools/marvin/marvin/config/Baremtal_AdvZone_Vmware.cfg --- @@ -0,0 +1,191 @@ +{ +"zones": [ +{ +"name":

Re: [SSL CERTS] Importing ROOT and INTERMEDIATE certs for SSVM

2016-04-01 Thread Nux!
Ilya, See the last few lines of this post, I had a similar problem a while back: http://www.nux.ro/archive/2014/03/Run_your_own_realhostip.html -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro - Original Message - > From: "ilya" > To: dev@cloudstack.apache.org >

[GitHub] cloudstack pull request: Cloudstack 8611 Handle SSH if server "for...

2016-04-01 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1459#issuecomment-204303829 @GabrielBrascher what shows is that unchanged line are changed. This could mean that you are using different line endings. Are you working on a windows machine

[GitHub] cloudstack pull request: BUG-ID:CLOUDSTACK-9331:added code in marv...

2016-04-01 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1458#issuecomment-204301035 tools/marvin/marvin/config/Baremtal_AdvZone_Vmware.cfg does not have a license. @cloudsadhu please add it. --- If your project is set up for it, you can reply

[GitHub] cloudstack pull request: Cloudstack 8611 Handle SSH if server "for...

2016-04-01 Thread GabrielBrascher
Github user GabrielBrascher commented on the pull request: https://github.com/apache/cloudstack/pull/1459#issuecomment-204293184 Not sure why it shows as if I had changed all the SshHelper class (the diff shows removed lines that I have not changed). I will check on that. --- If yo

[GitHub] cloudstack pull request: Cloudstack 8611 Handle SSH if server "for...

2016-04-01 Thread GabrielBrascher
GitHub user GabrielBrascher opened a pull request: https://github.com/apache/cloudstack/pull/1459 Cloudstack 8611 Handle SSH if server "forget" to send exit status Continuing the work started by @likitha, I cherry-picked the commit (b9181c689e0e7b5f1e28c81d73710196dfabd0ba) fro

Re: Permissions issues on cwiki with my login credentials

2016-04-01 Thread Daan Hoogland
On Fri, Apr 1, 2016 at 7:50 AM, Shweta Agarwal < shweta.agarw...@accelerite.com> wrote: > shwetaag ​is added​ -- Daan