[GitHub] cloudstack issue #872: Strongswan vpn feature

2016-09-19 Thread swill
Github user swill commented on the issue: https://github.com/apache/cloudstack/pull/872 @rhtyd have you gotten this system VM to build? I am having trouble getting it to finish building. My build is failing here... ``` + log DEBUG 'on_exit: clean_vbox' + local

[GitHub] cloudstack issue #1676: CLOUDSTACK-9502: DS template copies don’t get dele...

2016-09-19 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1676 @jburwell We just re-run integration tests. The same environment as 1560. You can now safely back out 1560 and forward merge this one. ''' test DeployVM in anti-affinity groups for

Re: [DISCUSS] Replacing the VR

2016-09-19 Thread Matthew Smart
Abstracting that interaction would also make future implementations MUCH less painful and allows third parties who want their networking products to be CS compatible to write their own implementation of a published API that CS defines. Let them do the legwork for their products, expand CS

[GitHub] cloudstack issue #1677: CLOUDSTACK-8830 : CLONE - VM snapshot fails for 12 m...

2016-09-19 Thread nvazquez
Github user nvazquez commented on the issue: https://github.com/apache/cloudstack/pull/1677 @jburwell I took code from @maneesha-p and added some changes from your review in last PR --- If your project is set up for it, you can reply to this email and have your reply appear on

RE: [DISCUSS] Replacing the VR

2016-09-19 Thread Paul Angus
Hi All, From the looks of this thread and the different preferences that parties have, the way forward looks to be a model that allows multiple different VRs to be used is the answer. This is something that I have had in mind for some time now. By abstracting the roles and configuration of

[GitHub] cloudstack pull request #1677: CLOUDSTACK-8830 : CLONE - VM snapshot fails f...

2016-09-19 Thread nvazquez
GitHub user nvazquez opened a pull request: https://github.com/apache/cloudstack/pull/1677 CLOUDSTACK-8830 : CLONE - VM snapshot fails for 12 min after instance creation Summary: Change in API behavior in vCenter 6.0.0. The name for VM Power On task is null on VMware vCenter 6.0,

Re: [DISCUSS] Replacing the VR

2016-09-19 Thread Syed Ahmed
Hey Guys, Will and I had a discussion in the morning on around VyOS and I have an idea that could work, here me out. The way Cloudstack currently communicates with VR is via the Hypervisor (let's assume KVM and XenServer for now). The management server sends a command as a JSON to the hypervisor

Re: trigger scripts by event

2016-09-19 Thread Stephan Seitz
Thanks Simon! That was exactly what I was looking for! Am Montag, den 19.09.2016, 15:44 + schrieb Simon Weller: > Stephan, > > > ACS has supported publishing events to RabbitMQ and Kafta for a few > releases now.  See this doc: http://docs.cloudstack.apache.org/projec >

[GitHub] cloudstack issue #798: CLOUDSTACK-8830 - [Vmware] VM snapshot fails for 12 m...

2016-09-19 Thread nvazquez
Github user nvazquez commented on the issue: https://github.com/apache/cloudstack/pull/798 Hi @jburwell, yes, actually I was working on other PRs but now I can pick up this one, is it ok? --- If your project is set up for it, you can reply to this email and have your reply appear on

Re: trigger scripts by event

2016-09-19 Thread Simon Weller
Stephan, ACS has supported publishing events to RabbitMQ and Kafta for a few releases now. See this doc: http://docs.cloudstack.apache.org/projects/cloudstack-administration/en/4.9/events.html It's pretty easy to setup and the events make it very easy to trigger third party interactions.

Re: [DISCUSS] Replacing the VR

2016-09-19 Thread Stephan Seitz
Hi! Just to add my 2 cents to that thread: I'ld really like to see something like vyatta or pfsense integrated as "standard" VR. We'd also talked internally about replacing the VR with some more mature "appliance"-like router distro. pfsense e.g. comes AFAIK with no defined API but instead has

Re: [DISCUSS] Replacing the VR

2016-09-19 Thread Will Stevens
Thank you for the feedback Matthew. The Debian 6 EOL thing is something that VyOS has taken very seriously. Everything has been ported to Debian 8 now, but it is still in beta and is not considered 'stable' yet because they have not been able to regression test the entire feature set yet. In

[GitHub] cloudstack pull request #1615: CLOUDSTACK-9438: Fix for CLOUDSTACK-9252 - Ma...

2016-09-19 Thread nvazquez
Github user nvazquez commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1615#discussion_r79419754 --- Diff: test/integration/smoke/test_ssvm.py --- @@ -1197,3 +1205,148 @@ def test_10_destroy_cpvm(self): # Call to verify cloud process is

Re: [DISCUSS] Replacing the VR

2016-09-19 Thread Matthew Smart
We have been using Vyatta (and then Vyos) after the fork for all of our production routers for over four years now. It is rock solid and performs better than the Juniper routers we replaced them with. It is debian based underneath its custom shell whose syntax is very much like Juniper. A sudo

trigger scripts by event

2016-09-19 Thread Stephan Seitz
Hi! We'ld like to add some of the provisioned Networks, IPs and VM into our docu / inventory DB. By now, we're gathering infos from the ACS DB as well as from the API - filtering start/end-date. To get things in a more elegant way, I'ld like to run our skripts event-triggered. So my question is:

[GitHub] cloudstack issue #1560: CLOUDSTACK-9386: DS template copies don’t get dele...

2016-09-19 Thread nvazquez
Github user nvazquez commented on the issue: https://github.com/apache/cloudstack/pull/1560 Thanks @jburwell, I've opened new PR (1676) against 4.9 release branch --- 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 #1676: CLOUDSTACK-9502: DS template copies don’t g...

2016-09-19 Thread nvazquez
GitHub user nvazquez opened a pull request: https://github.com/apache/cloudstack/pull/1676 CLOUDSTACK-9502: DS template copies don’t get deleted in VMware ESXi with multiple clusters and zone wide storage (include CLOUDSTACK-9386 into 4.9 release branch) Include #1560 into 4.9

[GitHub] cloudstack issue #872: Strongswan vpn feature

2016-09-19 Thread swill
Github user swill commented on the issue: https://github.com/apache/cloudstack/pull/872 I will start doing some testing on this today. Thanks... --- 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

[GitHub] cloudstack pull request #1615: CLOUDSTACK-9438: Fix for CLOUDSTACK-9252 - Ma...

2016-09-19 Thread nvazquez
Github user nvazquez commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1615#discussion_r79389208 --- Diff: test/integration/smoke/test_ssvm.py --- @@ -1197,3 +1205,148 @@ def test_10_destroy_cpvm(self): # Call to verify cloud process is

[GitHub] cloudstack issue #1639: CLOUDSTACK-9453: WIP : Marvin optimizations and fixe...

2016-09-19 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1639 Macchinina template doesn't seem to work in vmware hypervisor. When VM deployed eth0 is not created. We probably need to make sure a template is fully operational on all 3 hypervisors to include

[GitHub] cloudstack issue #1675: CLOUDSTACK-9453: WIP

2016-09-19 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1675 Macchinina template doesn't seem to work in vmware hypervisor. When VM deployed eth0 is not created. We probably need to make sure a template is fully operational on all 3 hypervisors to include

[GitHub] cloudstack issue #1602: CLOUDSTACK-9422: Granular 'vmware.create.full.clone'...

2016-09-19 Thread nvazquez
Github user nvazquez commented on the issue: https://github.com/apache/cloudstack/pull/1602 Thanks for your help @jburwell ! --- 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 issue #872: Strongswan vpn feature

2016-09-19 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/872 Thanks @jayapalu I'm rebuilding some infra, I'll get back to you soon. --- 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 issue #1591: Updating Alert codes

2016-09-19 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1591 @jburwell we can merge this, since it's a doc change no tests are needed. --- 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 issue #1591: Updating Alert codes

2016-09-19 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1591 LGTM @jburwell I checked the changes against the values defined in `AlertService` and they check out perfectly. --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack issue #1664: CLOUDSTACK-8676 Deploy user instance from vm snapsho...

2016-09-19 Thread sateesh-chodapuneedi
Github user sateesh-chodapuneedi commented on the issue: https://github.com/apache/cloudstack/pull/1664 @jburwell Yes, currently working on breaking the methods into multiple smaller units, and adding unit tests. Along with these adding the marvin tests from the other PR so that PR

[GitHub] cloudstack issue #872: Strongswan vpn feature

2016-09-19 Thread jayapalu
Github user jayapalu commented on the issue: https://github.com/apache/cloudstack/pull/872 @rhtyd @pdion891 @swill I have squashed the commits. Added the template changes to install strongswan 5.2. Can one you trigger the systemvm template job on this branch. --- If your

[GitHub] cloudstack issue #1639: CLOUDSTACK-9453: WIP : Marvin optimizations and fixe...

2016-09-19 Thread abhinandanprateek
Github user abhinandanprateek commented on the issue: https://github.com/apache/cloudstack/pull/1639 Thanks @karuturi @rhtyd I created a new one for 4.8 here https://github.com/apache/cloudstack/pull/1675 @jburwell --- If your project is set up for it, you can reply to this

[GitHub] cloudstack pull request #1675: CLOUDSTACK-9453: WIP

2016-09-19 Thread abhinandanprateek
GitHub user abhinandanprateek opened a pull request: https://github.com/apache/cloudstack/pull/1675 CLOUDSTACK-9453: WIP 1. Adding small sized macchinina template as built in templates 2. Adding methods so the test writers haev choice to use any of these 3. Updateing

[GitHub] cloudstack issue #1666: CLOUDSTACK-9480, CLOUDSTACK-9495 fix egress rule inc...

2016-09-19 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1666 I think in the latest code change, `fwr` is not defined before and hence `fwr +=` operation will fail. Also, the ICMP rule is still appended and not Inserted in the beginning. Is that ok?

[GitHub] cloudstack issue #1666: CLOUDSTACK-9480, CLOUDSTACK-9495 fix egress rule inc...

2016-09-19 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1666 @murali-reddy recurring failures will be listed as failing since last n runs. The router related tests(lb, network ops) are failing due to this PR. Are the tests running(manual or

[GitHub] cloudstack issue #1666: CLOUDSTACK-9480, CLOUDSTACK-9495 fix egress rule inc...

2016-09-19 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1666 @jburwell Jenkins was failing unrelated unit-test failure. I have triggered new Jenkins build which has passed. Changed the title of the PR to reflect that its fixing issues

[GitHub] cloudstack issue #872: Strongswan vpn feature

2016-09-19 Thread jayapalu
Github user jayapalu commented on the issue: https://github.com/apache/cloudstack/pull/872 @pdion891 Below is the Remote access vpn config, update left with the VR public ip. #ipsec remote access vpn configuration conn L2TP-PSK authby=psk

[GitHub] cloudstack pull request #1615: CLOUDSTACK-9438: Fix for CLOUDSTACK-9252 - Ma...

2016-09-19 Thread koushik-das
Github user koushik-das commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1615#discussion_r79357165 --- Diff: test/integration/smoke/test_ssvm.py --- @@ -1197,3 +1205,148 @@ def test_10_destroy_cpvm(self): # Call to verify cloud process

[GitHub] cloudstack issue #1643: CLOUDSTACK-9460: For long running transactions, if t...

2016-09-19 Thread abhinandanprateek
Github user abhinandanprateek commented on the issue: https://github.com/apache/cloudstack/pull/1643 @jburwell created a new PR for 4.8 here https://github.com/apache/cloudstack/pull/1674. I was getting many conflicts due to which I decided to create a new PR and cherry picked the

[GitHub] cloudstack pull request #1674: CLOUDSTACK-9460: For long running transaction...

2016-09-19 Thread abhinandanprateek
GitHub user abhinandanprateek opened a pull request: https://github.com/apache/cloudstack/pull/1674 CLOUDSTACK-9460: For long running transactions, if the connection is timed out by the mysql server then refresh it You can merge this pull request into a Git repository by running:

Re: CS 4.9 NIO Selector wait time PR-1601

2016-09-19 Thread Rohit Yadav
Thanks for sharing Martin, I'm glad it worked for you. Regards. From: martin kolly Sent: 13 September 2016 00:52:49 To: dev@cloudstack.apache.org; Rohit Yadav Subject: Re: CS 4.9 NIO Selector wait time PR-1601 We could fix the issue by

[GitHub] cloudstack issue #1669: Make CloudStack JSP-free

2016-09-19 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1669 LGTM, I've checkout the PR, builded it and deployed a simulator zone, then ran through a few localizations and did exploratory testing in the UI, all seem to work fine. --- If your

[GitHub] cloudstack pull request #1611: marvin: deploy clusters in separate threads

2016-09-19 Thread rhtyd
Github user rhtyd commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1611#discussion_r79337718 --- Diff: tools/marvin/marvin/deployDataCenter.py --- @@ -224,6 +225,15 @@ def createClusters(self, clusters, zoneId, podId, vmwareDc=None):

[GitHub] cloudstack issue #1511: 4.9/master bountycastle changes

2016-09-19 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1511 @jburwell these are major changes so they need to go into master, targetting 4.10, or 4.11. This PR was created based on Daan's original PR, I'll resume working on this soon. --- If your

[GitHub] cloudstack issue #1639: CLOUDSTACK-9453: WIP : Marvin optimizations and fixe...

2016-09-19 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1639 @abhinandanprateek yes, click on `edit` on your PR you should be able to change the base branch. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack issue #1669: Make CloudStack JSP-free

2016-09-19 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1669 CloudStack UI used dictionary, dictionary2 and index jsp files. The dictionary jsp files simply added a dictionary (java script object or map) by reading through messages properties for

[GitHub] cloudstack issue #1639: CLOUDSTACK-9453: WIP : Marvin optimizations and fixe...

2016-09-19 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1639 @abhinandanprateek https://github.com/blog/2224-change-the-base-branch-of-a-pull-request --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] cloudstack issue #1639: CLOUDSTACK-9453: WIP : Marvin optimizations and fixe...

2016-09-19 Thread abhinandanprateek
Github user abhinandanprateek commented on the issue: https://github.com/apache/cloudstack/pull/1639 @jburwell rebased it @rhtyd is there a way to move a PR to another branch without creating a new one ? --- If your project is set up for it, you can reply to this email and

[GitHub] cloudstack issue #1643: CLOUDSTACK-9460: For long running transactions, if t...

2016-09-19 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1643 @jburwell yes that is possible, @abhinandanprateek please edit the PR and change the base branch, then rebase your PR branch against 4.8 branch. --- If your project is set up for it, you can