[GitHub] cloudstack issue #1811: CLOUDSTACK-9649: In the management server log there ...

2016-12-06 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1811 @borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your re

[GitHub] cloudstack issue #1811: CLOUDSTACK-9649: In the management server log there ...

2016-12-06 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1811 @blueorangutan package --- 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 an

[GitHub] cloudstack pull request #1794: added more guest os

2016-12-06 Thread koushik-das
Github user koushik-das commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1794#discussion_r91235138 --- Diff: setup/db/db/schema-4910to41000.sql --- @@ -45,4 +45,48 @@ CREATE TABLE `cloud`.`vlan_details` ( `display` tinyint(1) NOT NULL DEFAULT

[GitHub] cloudstack issue #1758: CLOUDSTACK-9588: Add Load Balancer functionality in ...

2016-12-06 Thread nitin-maharana
Github user nitin-maharana commented on the issue: https://github.com/apache/cloudstack/pull/1758 @rajesh-battala : Thanks for giving the details. I will go through the video. If i find a case where it is really required then I will modify the PR. --- If your project is set up for it

[GitHub] cloudstack issue #1659: CLOUDSTACK-9339 Virtual Routers don't handle Multipl...

2016-12-06 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1659 component test results. If any one wish to run component test test_multiple_public_interfaces.py, add a new public ip range in the test_daya.py, in the 'publiciprange' dict object.

[GitHub] cloudstack issue #1758: CLOUDSTACK-9588: Add Load Balancer functionality in ...

2016-12-06 Thread rajesh-battala
Github user rajesh-battala commented on the issue: https://github.com/apache/cloudstack/pull/1758 Here is the link https://www.youtube.com/watch?v=qybCr1klhKk&t=1108s I have seen in this demo video about using that extra page On Tue, Dec 6, 2016 at 4:33 PM, Nitin Kumar Maha

[GitHub] cloudstack issue #1815: Fix docker image build for cloudstack-management

2016-12-06 Thread pdion891
Github user pdion891 commented on the issue: https://github.com/apache/cloudstack/pull/1815 Thanks @rhtyd, let merge it so we have the simulator back online on docker hub and let's improve our Docker images... @jburwell, @rhtyd , do I merge it or you do it? I'm just wondering

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-06 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1749 @rhtyd It took a few force pushes, but it worked. :) --- 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 n

[GitHub] cloudstack issue #1659: CLOUDSTACK-9339 Virtual Routers don't handle Multipl...

2016-12-06 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1659 Trillian test result (tid-597) Environment: vmware-55u3 (x2), Advanced Networking with Mgmt server 7 Total time taken: 36723 seconds Marvin logs: https://github.com/blueoranguta

[GitHub] cloudstack issue #1809: CLOUDSTACK-9646: No usage is generated for uploaded ...

2016-12-06 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1809 Trillian test result (tid-599) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 26799 seconds Marvin logs: https://github.com/blueoranguta

[GitHub] cloudstack issue #1799: CLOUDSTACK-9632: Upgrade bouncy castle to version 1....

2016-12-06 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1799 We've enough code review, and test results. Test LGTM, some failures with vpc/rvr are known intermittent failures. The Travis failure is caused by `component/test_volumes`, which I found is failin

[GitHub] cloudstack issue #1807: CLOUDSTACK-9633: Revert addition of `vhd` extention ...

2016-12-06 Thread swill
Github user swill commented on the issue: https://github.com/apache/cloudstack/pull/1807 @rhtyd If you merge this you will break #1600. We may want to do some more testing on the upgrade path, but I believe @syed has done some testing of the upgrade path. Maybe we can put together a

[GitHub] cloudstack issue #1799: CLOUDSTACK-9632: Upgrade bouncy castle to version 1....

2016-12-06 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1799 Trillian test result (tid-596) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 25569 seconds Marvin logs: https://github.com/blueoranguta

[GitHub] cloudstack issue #1579: CLOUDSTACK-9403 : Support for shared networks in Nua...

2016-12-06 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1579 @prashanthvarma we're waiting for a final LGTM from @murali-reddy can you do a `push -f` to re-kick Travis. (though I checked the failure is not due to your PR). @murali-reddy can you please revie

[GitHub] cloudstack issue #1807: CLOUDSTACK-9633: Revert addition of `vhd` extention ...

2016-12-06 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1807 @serg38 I think we can do that, but will be also need some tooling to rename the actual files on the SRs? /cc @abhinandanprateek --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack issue #1807: CLOUDSTACK-9633: Revert addition of `vhd` extention ...

2016-12-06 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1807 @syed Thanks, so just to confirm... with this PR, is your work with #1600 works and is not broken? --- If your project is set up for it, you can reply to this email and have your reply appear on

Re: Usage server development

2016-12-06 Thread Rohit Yadav
Hi Patrick, You're right, in the 'how to build cloudstack' wiki we've not mentioned this. You can run it like this: mvn clean install -P usage -pl usage # This will work given that other dependencies are already built/installed mvn -P usage -pl usage -Drun -Dpid=$$ # This runs the usage ser

[GitHub] cloudstack issue #1727: CLOUDSTACK-9539: Support changing Service offering f...

2016-12-06 Thread nvazquez
Github user nvazquez commented on the issue: https://github.com/apache/cloudstack/pull/1727 @koushik-das @rhtyd I pushed changes for supporting custom service offerings the way @koushik-das and @serg38 proposed --- If your project is set up for it, you can reply to this email and hav

[GitHub] cloudstack issue #1763: CLOUDSTACK-9594: API "list templates templatefilter=...

2016-12-06 Thread sudhansu7
Github user sudhansu7 commented on the issue: https://github.com/apache/cloudstack/pull/1763 @abhinandanprateek Below is test scenario and result. I have also reverted the changes introduced in CLOUDSTACK-9376. Test Scenario: Domain: SUDDOMAIN Domain Admin: s

Usage server development

2016-12-06 Thread Patrick Dube
Hey all, Since 4.9, usage is broken if you have a project that has resources. Basically, it is an autoboxing NPE because a project account has no role (UsageDaoImpl line 116). I know where to fix it, but I can't find any documentation about running the usage server in a dev setup, so I can't test

[GitHub] cloudstack issue #1807: CLOUDSTACK-9633: Revert addition of `vhd` extention ...

2016-12-06 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1807 @syed @rhtyd @abhinandanprateek Why cant we simply rename all install paths in DB for existing snapshots instead. It can be as simple as adding this query to schema upgrade path UPDATE cloud.

[GitHub] cloudstack issue #1799: CLOUDSTACK-9632: Upgrade bouncy castle to version 1....

2016-12-06 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1799 Trillian test result (tid-594) Environment: xenserver-65sp1 (x2), Advanced Networking with Mgmt server 6 Total time taken: 34189 seconds Marvin logs: https://github.com/blueoran

[GitHub] cloudstack issue #1807: CLOUDSTACK-9633: Revert addition of `vhd` extention ...

2016-12-06 Thread syed
Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1807 @abhinandanprateek the snapshot path is something which is internal to Cloudstack, as an end user, I would not be able to tell the difference before and after the upgrade, unless I missed something

[GitHub] cloudstack issue #1807: CLOUDSTACK-9633: Revert addition of `vhd` extention ...

2016-12-06 Thread syed
Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1807 @rhtyd I understand that we want to avoid breaking upgrades. But I have upgraded my setup and I did not find any problems. I am trying to understand what will break if upgrades happen, given the fa

[GitHub] cloudstack issue #1741: Updated StrongSwan VPN Implementation

2016-12-06 Thread swill
Github user swill commented on the issue: https://github.com/apache/cloudstack/pull/1741 Rebased against master and fixed merge conflicts... --- 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 th

[GitHub] cloudstack issue #1799: CLOUDSTACK-9632: Upgrade bouncy castle to version 1....

2016-12-06 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1799 Trillian test result (tid-592) Environment: vmware-55u3 (x2), Advanced Networking with Mgmt server 7 Total time taken: 36740 seconds Marvin logs: https://github.com/blueoranguta

[GitHub] cloudstack issue #1818: CLOUDSTACK-9655 The template which is registered in ...

2016-12-06 Thread cloudmonger
Github user cloudmonger commented on the issue: https://github.com/apache/cloudstack/pull/1818 ### ACS CI BVT Run **Sumarry:** Build Number 159 Hypervisor xenserver NetworkType Advanced Passed=101 Failed=4 Skipped=7 _Link to logs Folder (searc

[GitHub] cloudstack issue #1741: Updated StrongSwan VPN Implementation

2016-12-06 Thread kiwiflyer
Github user kiwiflyer commented on the issue: https://github.com/apache/cloudstack/pull/1741 @jayapalu Can you expand on your findings regarding public ip order? Did you pull in the latest PR with the fixes from @swill? --- If your project is set up for it, you can reply to this ema

[GitHub] cloudstack issue #1579: CLOUDSTACK-9403 : Support for shared networks in Nua...

2016-12-06 Thread prashanthvarma
Github user prashanthvarma commented on the issue: https://github.com/apache/cloudstack/pull/1579 @rhtyd Just to make sure, are we waiting for a LGTM to merge this PR ? May be, @jburwell can comment and/or give a LGTM as he has reviewed the PR :) --- If your project is set u

[GitHub] cloudstack issue #1799: CLOUDSTACK-9632: Upgrade bouncy castle to version 1....

2016-12-06 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1799 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests --- If your project is set up for it, you can reply to this email and have your reply ap

[GitHub] cloudstack issue #1799: CLOUDSTACK-9632: Upgrade bouncy castle to version 1....

2016-12-06 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1799 @blueorangutan test --- 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

[GitHub] cloudstack issue #1799: CLOUDSTACK-9632: Upgrade bouncy castle to version 1....

2016-12-06 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1799 @rhtyd unsupported parameters provided. Supported mgmt server os are: `centos6, centos7, ubuntu`. Supported hypervisors are: `kvm-centos6, kvm-centos7, kvm-ubuntu, xenserver-65sp1, xenserv

[GitHub] cloudstack issue #1799: CLOUDSTACK-9632: Upgrade bouncy castle to version 1....

2016-12-06 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1799 @blueorangutan test On 6 Dec 2016 18:11, "Blue Orangutan Build Things" wrote: Packaging result: ✔centos6 ✔centos7 ✔debian. JID-330 — You are re

[GitHub] cloudstack issue #1799: CLOUDSTACK-9632: Upgrade bouncy castle to version 1....

2016-12-06 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1799 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-330 --- 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 d

[GitHub] cloudstack issue #1799: CLOUDSTACK-9632: Upgrade bouncy castle to version 1....

2016-12-06 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1799 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your reply appe

[GitHub] cloudstack issue #1659: CLOUDSTACK-9339 Virtual Routers don't handle Multipl...

2016-12-06 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1659 @rhtyd please proceed with merge. I have not added the test suite(test_multiple_public_interfaces.py) part of this PR as smoke tests, because it takes around an hour for execution so they a

[GitHub] cloudstack issue #1799: CLOUDSTACK-9632: Upgrade bouncy castle to version 1....

2016-12-06 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1799 @blueorangutan package --- 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

[GitHub] cloudstack-www pull request #29: Added Delimiter Corp to known users

2016-12-06 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack-www/pull/29 --- 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 issue #1659: CLOUDSTACK-9339 Virtual Routers don't handle Multipl...

2016-12-06 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1659 Thanks @murali-reddy @borisstoyanov I'll proceed with merging this now. @murali-reddy do you want to run the component test on this PR or on 1753/1754? --- If your project is set up for it, you c

[GitHub] cloudstack issue #1758: CLOUDSTACK-9588: Add Load Balancer functionality in ...

2016-12-06 Thread nitin-maharana
Github user nitin-maharana commented on the issue: https://github.com/apache/cloudstack/pull/1758 @rajesh-battala : From my observation, I found whatever we can do in Add Load Balancer tab, the same can be performed in load balancing section after selecting any IP. Do you think any sc

[GitHub] cloudstack pull request #1818: CLOUDSTACK-9655 The template which is registe...

2016-12-06 Thread karuturi
GitHub user karuturi opened a pull request: https://github.com/apache/cloudstack/pull/1818 CLOUDSTACK-9655 The template which is registered in all zones will be deleted by deleting 1 template on any zone added extra warning message if it's a cross-zone template ("This is a cross

[GitHub] cloudstack issue #1805: CLOUDSTACK-9637: Template create from snapshot does ...

2016-12-06 Thread cloudmonger
Github user cloudmonger commented on the issue: https://github.com/apache/cloudstack/pull/1805 ### ACS CI BVT Run **Sumarry:** Build Number 157 Hypervisor xenserver NetworkType Advanced Passed=97 Failed=6 Skipped=7 _Link to logs Folder (search

[GitHub] cloudstack issue #1741: Updated StrongSwan VPN Implementation

2016-12-06 Thread jayapalu
Github user jayapalu commented on the issue: https://github.com/apache/cloudstack/pull/1741 @rhtyd While testing this feature public ip order change on reboot issue blocked this. My suggestion is that public ip out of order is different issue. If vpn functionalities working e

[GitHub] cloudstack issue #1805: CLOUDSTACK-9637: Template create from snapshot does ...

2016-12-06 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1805 I've manually verified the bug fix, looks OK ``` (local) SBCM5> list templates name=newTemp templatefilter=all { "count": 1, "template": [ { "accoun

[GitHub] cloudstack issue #1771: CLOUDSTACK-9611: Dedicating a Guest VLAN range to Pr...

2016-12-06 Thread nitin-maharana
Github user nitin-maharana commented on the issue: https://github.com/apache/cloudstack/pull/1771 @rhtyd : Changed PR's base branch to 4.9 --- 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 issue #1799: CLOUDSTACK-9632: Upgrade bouncy castle to version 1....

2016-12-06 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1799 code change 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 enabled and wishe

[GitHub] cloudstack issue #1579: CLOUDSTACK-9403 : Support for shared networks in Nua...

2016-12-06 Thread krissterckx
Github user krissterckx commented on the issue: https://github.com/apache/cloudstack/pull/1579 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 enabled and wishes so, or if t

[GitHub] cloudstack issue #1792: CLOUDSTACK-9623: Deploying virtual machine fails due...

2016-12-06 Thread nitin-maharana
Github user nitin-maharana commented on the issue: https://github.com/apache/cloudstack/pull/1792 @jburwell : Switched the base branch to 4.9 --- 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 t

[GitHub] cloudstack issue #1659: CLOUDSTACK-9339 Virtual Routers don't handle Multipl...

2016-12-06 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1659 Yes I think test failures does not seem to be caused by this PR, LGTM based on code review. --- If your project is set up for it, you can reply to this email and have your reply appear o

[GitHub] cloudstack issue #1804: CLOUDSTACK-9639: Unable to create shared network wit...

2016-12-06 Thread nitin-maharana
Github user nitin-maharana commented on the issue: https://github.com/apache/cloudstack/pull/1804 @rhtyd : Changed the base branch to 4.9 --- 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