[GitHub] cloudstack pull request #1619: Add the Transifex config for next version of ...

2016-09-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1619 --- 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 #1690: Update L10N resource files with 4.10 strings ...

2016-09-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1690 --- 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 #1684: CLOUDSTACK-9489: the new config vars that are added ...

2016-09-29 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1684 @pdion891 based on the behavior reported, I believe we can make this experience much smoother if not completely consistent. Do you have time to grab a quick GTM to discuss tomorrow (29 Sept

[GitHub] cloudstack issue #872: Strongswan vpn feature

2016-09-29 Thread swill
Github user swill commented on the issue: https://github.com/apache/cloudstack/pull/872 Here is the template (for xen anyway): https://objects-east.cloud.ca/v1/5ef827605f884961b94881e928e7a250/swill/systemvm64template-master-4.6.0-xen.vhd.bz2 So far I have not been able to

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

2016-09-29 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1676 @jburwell This is already in master. @nvazquez opened this to target 4.9 and forward merge. It is completely fine for fix to be 4.10 only since it only affects rather special circumstances.

[GitHub] cloudstack issue #1684: CLOUDSTACK-9489: the new config vars that are added ...

2016-09-29 Thread pdion891
Github user pdion891 commented on the issue: https://github.com/apache/cloudstack/pull/1684 Thanks @abhinandanprateek. So I guest it's an LGTM for this PR but it's not solving the whole issue for CLOUDSTACK-9489 which is configuration inconsistency between fresh install and upgrade.

[GitHub] cloudstack issue #1696: [4.9/lts] CLOUDSTACK-9364: Add Ubuntu 16.04 as a gue...

2016-09-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1696 I've manually kicked a trillian env (tid-84) to get an env against xenserver 6.5sp1, as soon as it comes up I'll perform the test to launch a Ubuntu 16.04 VM using templates from

[GitHub] cloudstack issue #1619: Add the Transifex config for next version of CS (4.1...

2016-09-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1619 LGTM. Since basic checks are all green and we've enough votes, I'll merge this. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] cloudstack issue #1690: Update L10N resource files with 4.10 strings from Tr...

2016-09-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1690 LGTM. Since it's a translation change, we've enough votes to merge this. I'll proceed with merging this on master. --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack issue #872: Strongswan vpn feature

2016-09-29 Thread pdion891
Github user pdion891 commented on the issue: https://github.com/apache/cloudstack/pull/872 in case of HTTPs issue registering the template http://objects-east.cloud.ca/v1/5ef827605f884961b94881e928e7a250/swill/systemvm64template-master-4.6.0-xen.vhd.bz2 can be use. --- If your

Re: is this a bug

2016-09-29 Thread Linas Žilinskas
If it's similar to what happened to me, it's caused by ssl. By default the SSVM listens on https for template POST's. Also by default it doesn't have a valid certificate (listening on ip address, without domain etc). Most browsers disallow posting to insecure url's. As a workaround, you can

[GitHub] cloudstack issue #1673: CLOUDSTACK-9071: Properly parse stats.output.uri in ...

2016-09-29 Thread wido
Github user wido commented on the issue: https://github.com/apache/cloudstack/pull/1673 @jburwell Ah, that went wrong with the rebase. Fixed! --- 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

Re: is this a bug

2016-09-29 Thread Will Stevens
When I just tried a couple days ago it was not working. It looked like an ssl issue, but I didn't investigate. On Sep 29, 2016 9:16 AM, "Jeroen Baten" wrote: > > > So, I could upload templates using an url after setting up a webserver. > > Am I correct in assuming it

is this a bug

2016-09-29 Thread Jeroen Baten
So, I could upload templates using an url after setting up a webserver. Am I correct in assuming it is a bug that I can not use the webgui to upload a template? Using 4.9.0 Jeroen Baten Specialist ICT T +31(0)88335 7941 M+31(0)6 4691 4649 [Description: Description:

[GitHub] cloudstack issue #872: Strongswan vpn feature

2016-09-29 Thread swill
Github user swill commented on the issue: https://github.com/apache/cloudstack/pull/872 Right, I forgot to mention that. Thanks @pdion891. --- 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

[GitHub] cloudstack issue #1696: [4.9/lts] CLOUDSTACK-9364: Add Ubuntu 16.04 as a gue...

2016-09-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1696 With a manually deployed Trillian environment based on XenServer 6.5sp1, I was able to register Ubuntu 16.04 and launch a VM. Screenshots attached: ![screenshot from 2016-09-29

[GitHub] cloudstack issue #1692: fix ping tests to properly recognise successful ping...

2016-09-29 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1692 Trillian test result (tid-78) Environment: kvm-centos7 (x2), Advanced Networking Total time taken: 26135 seconds Marvin logs:

[GitHub] cloudstack issue #1684: CLOUDSTACK-9489: the new config vars that are added ...

2016-09-29 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1684 @rhtyd discussing this issue with @paul, there is a scenario where this wedged condition can occur. If you perform an upgrade by creating a new management server VM then no

[GitHub] cloudstack issue #1638: CLOUDSTACK-9456: Migrate master to Spring 4.x

2016-09-29 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1638 Trillian test result (tid-80) Environment: kvm-centos7 (x2), Advanced Networking Total time taken: 26599 seconds Marvin logs:

[GitHub] cloudstack issue #1696: [4.9/lts] CLOUDSTACK-9364: Add Ubuntu 16.04 as a gue...

2016-09-29 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1696 @rhtyd do you think we should include this update in 4.8.2.0 as well? --- 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 #1600: Support Backup of Snapshots for Managed Storage

2016-09-29 Thread syed
Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1600 @mike-tutkowski I've added your updated tests and rebased. --- 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 issue #1684: CLOUDSTACK-9489: the new config vars that are added ...

2016-09-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1684 @pdion891 @jburwell as I've already explained, there should be no need to enable dynamic apichecker if you're upgrading from an older system that has a commands.properties file. I think the

[GitHub] cloudstack issue #1692: Fix Smoke Test Failures

2016-09-29 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1692 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-41 --- 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

[GitHub] cloudstack issue #1695: CLOUDSTACK-9511: fix test_privategw_acl.py to handle...

2016-09-29 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1695 Trillian test result (tid-82) Environment: kvm-centos7 (x2), Advanced Networking Total time taken: 27946 seconds Marvin logs:

[GitHub] cloudstack issue #1684: CLOUDSTACK-9489: the new config vars that are added ...

2016-09-29 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1684 @rhtyd we have a case where two experienced users (and likely more to come) have gotten tripped up by this case. Therefore, we need to address it. As I mentioned previously, the solution to

[GitHub] cloudstack issue #1684: CLOUDSTACK-9489: the new config vars that are added ...

2016-09-29 Thread PaulAngus
Github user PaulAngus commented on the issue: https://github.com/apache/cloudstack/pull/1684 That isnt correct @bhaisaab. commands.properties can be different on every mgmt server. Cloudstack would not know which version to use.

[GitHub] cloudstack issue #1695: CLOUDSTACK-9511: fix test_privategw_acl.py to handle...

2016-09-29 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1695 I am closing this pull request, and pushed the changes as part of PR #1692, which will cover overall marvin test failures --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack issue #1600: Support Backup of Snapshots for Managed Storage

2016-09-29 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1600 @mike-tutkowski sounds good. I am putting together a test results email which should be out shortly. It outlines the issues we have found thus far. As such, I expect the testing period for

[GitHub] cloudstack issue #1696: [4.9/lts] CLOUDSTACK-9364: Add Ubuntu 16.04 as a gue...

2016-09-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1696 @jburwell As per our maintenance policy we only maintain n, n-1 branches i.e. master (upcoming 4.10) and 4.9. At no point we had ever maintained three branches, it's too much overhead. So, which

[GitHub] cloudstack issue #1600: Support Backup of Snapshots for Managed Storage

2016-09-29 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1600 Sounds good, @syed Since what you added is the same as what's currently in master, it looks like your list of changed files for this PR is down to 35 (only one of which is a Python file,

[GitHub] cloudstack issue #1545: CLOUDSTACK-8715: Add channel to Instances for Qemu G...

2016-09-29 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1545 @wido I misunderstood. I thought was fixing one or more bugs. Given that it is an enhancement, you are correct that it should only target master. As soon as we iron the smoke test issues in

[GitHub] cloudstack issue #1684: CLOUDSTACK-9489: the new config vars that are added ...

2016-09-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1684 @jburwell when a fresh mgmt server is deployed for upgrading an existing CloudStack environment, several files from `/etc/cloudstack/management` are copied to the new server, especially the

[GitHub] cloudstack issue #1600: Support Backup of Snapshots for Managed Storage

2016-09-29 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1600 @syed @mike-tutkowski should the tests be re-executed to make sure everything is super green? --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack issue #1696: [4.9/lts] CLOUDSTACK-9364: Add Ubuntu 16.04 as a gue...

2016-09-29 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1696 @rhtyd 4.8 is still supported. However, support is for bug fixes, and this change is not a bug fix. 4.9 is an LTS branch which can take changes such as these. So I am cool with it being

[GitHub] cloudstack issue #1684: CLOUDSTACK-9489: the new config vars that are added ...

2016-09-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1684 @PaulAngus but it's up to the admin and how they want to upgrade/setup a new mgmt server. Assuming you want to create a new mgmt server for each old mgmt server, one strategy could be to copy the

[GitHub] cloudstack issue #1692: Fix Smoke Test Failures

2016-09-29 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1692 @borisstoyanov could you please amend the first line of commit 62f4826 to be : ? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] cloudstack issue #1694: CLOUDSTACK-9509: Host Connects Without Storage

2016-09-29 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1694 Trillian test result (tid-79) Environment: vmware-55u3 (x2), Advanced Networking Total time taken: 30443 seconds Marvin logs:

[GitHub] cloudstack issue #1688: Marvin Tests: Fix VPC network offering selection

2016-09-29 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1688 Trillian test result (tid-81) Environment: kvm-centos7 (x2), Advanced Networking Total time taken: 25885 seconds Marvin logs:

[GitHub] cloudstack pull request #1695: CLOUDSTACK-9511: fix test_privategw_acl.py to...

2016-09-29 Thread murali-reddy
Github user murali-reddy closed the pull request at: https://github.com/apache/cloudstack/pull/1695 --- 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

[GitHub] cloudstack issue #1673: CLOUDSTACK-9071: Properly parse stats.output.uri in ...

2016-09-29 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1673 @wido thank you for re-targeting the PR. Do you think running the smoke tests against one of the hypervisors is sufficient to verify this defect? Should we update a Marvin test to cover it?

[GitHub] cloudstack issue #1600: Support Backup of Snapshots for Managed Storage

2016-09-29 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1600 @jburwell Yes, the managed-storage tests should be re-executed. I can do this over the weekend as my CloudStack systems in the lab here are being fully utilized for the time being with

[GitHub] cloudstack issue #1692: fix ping tests to properly recognise successful ping...

2016-09-29 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1692 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-37 --- 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

[GitHub] cloudstack issue #1695: CLOUDSTACK-9511: fix test_privategw_acl.py to handle...

2016-09-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1695 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 the

[GitHub] cloudstack issue #1695: CLOUDSTACK-9511: fix test_privategw_acl.py to handle...

2016-09-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1695 @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

[GitHub] cloudstack issue #1688: Marvin Tests: Fix VPC network offering selection

2016-09-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1688 @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

[GitHub] cloudstack issue #1688: Marvin Tests: Fix VPC network offering selection

2016-09-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1688 LGTM. Jenkins failure is not caused by change from the PR. @abhinandanprateek @murali-reddy a second review please? --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack issue #1688: Marvin Tests: Fix VPC network offering selection

2016-09-29 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1688 @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

[GitHub] cloudstack issue #1638: CLOUDSTACK-9456: Migrate master to Spring 4.x

2016-09-29 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1638 @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

[GitHub] cloudstack pull request #1695: CLOUDSTACK-9511: fix test_privategw_acl.py to...

2016-09-29 Thread murali-reddy
GitHub user murali-reddy opened a pull request: https://github.com/apache/cloudstack/pull/1695 CLOUDSTACK-9511: fix test_privategw_acl.py to handle multiple physical networks modified script to handle case where there can be multple physical networks in the zone

[GitHub] cloudstack issue #1692: fix ping tests to properly recognise successful ping...

2016-09-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1692 On investigation, it was found that Trillian was falling back on master branch's tests which is why the test failures were high. Some of the feature such as oobm don't exist in 4.8. I've rekicked

[GitHub] cloudstack issue #1638: CLOUDSTACK-9456: Migrate master to Spring 4.x

2016-09-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1638 @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

[GitHub] cloudstack issue #928: CLOUDSTACK-8950 Hypervisor Parameter check is not per...

2016-09-29 Thread cloudmonger
Github user cloudmonger commented on the issue: https://github.com/apache/cloudstack/pull/928 ### ACS CI BVT Run **Sumarry:** Build Number 106 Hypervisor xenserver NetworkType Advanced Passed=103 Failed=0 Skipped=4 _Link to logs Folder

[GitHub] cloudstack issue #1695: CLOUDSTACK-9511: fix test_privategw_acl.py to handle...

2016-09-29 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1695 LGTM based on code review --- 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 #1650: fix macchinina template specificied for vmware in Ma...

2016-09-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1650 LGTM. @abhinandanprateek is the changes okay, or we should refactor move them to the default test_data file? --- If your project is set up for it, you can reply to this email and have your reply

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

2016-09-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1669 @jburwell @milamberspace with the current changes, no explicit change in workflow is needed but may be change in future. Since the aim of the PR was to get rid of jsps, I would like to avoid too

[GitHub] cloudstack issue #1638: CLOUDSTACK-9456: Migrate master to Spring 4.x

2016-09-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1638 @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

[GitHub] cloudstack issue #1694: CLOUDSTACK-9509: Host Connects Without Storage

2016-09-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1694 The previous test run was incorrect, several tests were run from master the features that don't exist on 4.8. @blueorangutan test --- If your project is set up for it, you can reply to this

[GitHub] cloudstack issue #1695: CLOUDSTACK-9511: fix test_privategw_acl.py to handle...

2016-09-29 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1695 @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

[GitHub] cloudstack issue #1694: CLOUDSTACK-9509: Host Connects Without Storage

2016-09-29 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1694 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + vmware-55u3) 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

[GitHub] cloudstack issue #1696: [4.9/lts] CLOUDSTACK-9364: Add Ubuntu 16.04 as a gue...

2016-09-29 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1696 Hi @rhtyd there are some typo in my comment, it should be "Ubuntu 16.04" not "Ubuntu 14.04" in some lines. --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack issue #1696: [4.9/lts] CLOUDSTACK-9364: Add Ubuntu 16.04 as a gue...

2016-09-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1696 @ustcweizhou Hi, I think I used your changes here. VMware is sane to interpret Ubuntu guest vms as either ubuntuGuest or ubuntu64Guest (identifier used), while to XS and KVM full names are passed

[GitHub] cloudstack issue #1688: Marvin Tests: Fix VPC network offering selection

2016-09-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1688 @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

[GitHub] cloudstack issue #1695: CLOUDSTACK-9511: fix test_privategw_acl.py to handle...

2016-09-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1695 @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

Re: L10N Translation File Format

2016-09-29 Thread Rohit Yadav
I think switching to Transifex's json format should be okay. I don't know the downside of switching to json format as well, which is why at the moment we can continue to use the current workflow and format. I would, therefore, avoid doing too many changes so I'll proceed with merging the PR

[GitHub] cloudstack issue #927: secure and hidden config values are returned as plain...

2016-09-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/927 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 the

[GitHub] cloudstack issue #1695: CLOUDSTACK-9511: fix test_privategw_acl.py to handle...

2016-09-29 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1695 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-38 --- 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

[GitHub] cloudstack issue #1688: Marvin Tests: Fix VPC network offering selection

2016-09-29 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1688 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-39 --- 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

[GitHub] cloudstack issue #1695: CLOUDSTACK-9511: fix test_privategw_acl.py to handle...

2016-09-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1695 @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

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

2016-09-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1669 Thanks @milamberspace is that a 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 issue #1669: Make CloudStack JSP-free

2016-09-29 Thread milamberspace
Github user milamberspace commented on the issue: https://github.com/apache/cloudstack/pull/1669 @rhtyd Yes LGTM. Now (and in another PR), I would like work to try to use directly the json key/value files from Transifex. --- If your project is set up for it, you can reply to

[GitHub] cloudstack issue #927: secure and hidden config values are returned as plain...

2016-09-29 Thread cloudmonger
Github user cloudmonger commented on the issue: https://github.com/apache/cloudstack/pull/927 ### ACS CI BVT Run **Sumarry:** Build Number 107 Hypervisor xenserver NetworkType Advanced Passed=101 Failed=2 Skipped=4 _Link to logs Folder

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

2016-09-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1669 With the tests and lgtms, I'll go with merging this. --- 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

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

2016-09-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1669 --- 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 #1696: [4.9/lts] CLOUDSTACK-9364: Add Ubuntu 16.04 a...

2016-09-29 Thread rhtyd
GitHub user rhtyd opened a pull request: https://github.com/apache/cloudstack/pull/1696 [4.9/lts] CLOUDSTACK-9364: Add Ubuntu 16.04 as a guest OS Adds Ubuntu 16.04 as a supported guest os. This allows users to select the OS when creating a template etc. SQL queries

[GitHub] cloudstack issue #1696: [4.9/lts] CLOUDSTACK-9364: Add Ubuntu 16.04 as a gue...

2016-09-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1696 @abhinandanprateek @murali-reddy please review, 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 issue #1688: Marvin Tests: Fix VPC network offering selection

2016-09-29 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1688 @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

[GitHub] cloudstack issue #1695: CLOUDSTACK-9511: fix test_privategw_acl.py to handle...

2016-09-29 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1695 @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

[GitHub] cloudstack issue #1688: Marvin Tests: Fix VPC network offering selection

2016-09-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1688 @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

[GitHub] cloudstack issue #1685: IGNORE this PR

2016-09-29 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1685 @rhtyd I understand these words: "help", "hello", "thanks", "package", "test" Test run usage: test (mgmt server, one of: centos6, centos7, ubuntu) (hypervisor, one of: kvm-centos6,

[GitHub] cloudstack issue #1685: IGNORE this PR

2016-09-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1685 @blueorangutan help --- 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 issue #1669: Make CloudStack JSP-free

2016-09-29 Thread milamberspace
Github user milamberspace commented on the issue: https://github.com/apache/cloudstack/pull/1669 @rhtyd I've just tests successfully the last version of the PR. Now if the value isn't translate, the default language is display. --- If your project is set up for it, you can reply

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

2016-09-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1669 @milamberspace thanks, yes please proceed with changing transifex format following merge of this PR. I lack experience with Transifex usage, but I can help with transforming the current

[GitHub] cloudstack issue #1696: [4.9/lts] CLOUDSTACK-9364: Add Ubuntu 16.04 as a gue...

2016-09-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1696 @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

[GitHub] cloudstack issue #1696: [4.9/lts] CLOUDSTACK-9364: Add Ubuntu 16.04 as a gue...

2016-09-29 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1696 @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

[GitHub] cloudstack issue #1696: [4.9/lts] CLOUDSTACK-9364: Add Ubuntu 16.04 as a gue...

2016-09-29 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1696 @rhtyd I think it is not an issue in KVM/LXC. the guest pv is checked by this line in plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java

[GitHub] cloudstack issue #1644: Honors the snapshot.backup.rightafter configuration ...

2016-09-29 Thread nathanejohnson
Github user nathanejohnson commented on the issue: https://github.com/apache/cloudstack/pull/1644 @jburwell @kiwiflyer @bvbharatk I've gotten this to a place where I'm feeling more comfortable with how it works, would you all like me to close this PR, reopen it against 4.9 then?

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

2016-09-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1669 @milamberspace I've started a new ticket to track progress on migration of transifex to json format: https://issues.apache.org/jira/browse/CLOUDSTACK-9513 --- If your project is set up for it,

[GitHub] cloudstack issue #1696: [4.9/lts] CLOUDSTACK-9364: Add Ubuntu 16.04 as a gue...

2016-09-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1696 Thanks @ustcweizhou I'll fix as suggested. --- 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 #1696: [4.9/lts] CLOUDSTACK-9364: Add Ubuntu 16.04 as a gue...

2016-09-29 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1696 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-40 --- 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

[GitHub] cloudstack issue #1673: CLOUDSTACK-9071: Properly parse stats.output.uri in ...

2016-09-29 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1673 @wido the Travis and Jenkins are failing due a compilation error. It appears to be that `StringUtils` has not been imported into `StatsCollector`. --- If your project is set up for it, you