[GitHub] cloudstack issue #1936: [CLOUDSTACK-9773] API: don't break API output with n...

2017-02-12 Thread marcaurele
Github user marcaurele commented on the issue: https://github.com/apache/cloudstack/pull/1936 Forced push to trigger another build. Looks like the failed test is not stable. --- 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 #1934: [CLOUDSTACK-9772] Template: perform a HEAD re...

2017-02-12 Thread remibergsma
Github user remibergsma commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1934#discussion_r100702832 --- Diff: utils/src/main/java/com/cloud/utils/UriUtils.java --- @@ -200,37 +198,27 @@ public static String getUpdateUri(String url, boolean encrypt)

Sanity Check: Router order of operations

2017-02-12 Thread Matthew Smart
Hey guys, Can I get a confirmation that the following statements are true for routing in ACS? 1. When processing incoming packets the ingress firewall rules are applied before the port forwarding rules. 2. Adding a port forwarding rule that does not have a corresponding ingress firewall rule wil

[GitHub] cloudstack issue #1938: CLOUDSTACK-9780: Fixed the default JAVA_HOME value t...

2017-02-12 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1938 @borisstoyanov can you post ubuntu test results? --- 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

Re: [GitHub] cloudstack issue #1938: CLOUDSTACK-9780: Fixed the default JAVA_HOME value t...

2017-02-12 Thread Will Stevens
I doubt the test environment will do a test without java_home set, so it is unlikely this fix will to tested with a run of that test. On Feb 12, 2017 11:36 PM, "karuturi" wrote: > Github user karuturi commented on the issue: > > https://github.com/apache/cloudstack/pull/1938 > > @borisst

[GitHub] cloudstack issue #1928: Update hypervisor capabilities for xenserver 7

2017-02-12 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1928 @syed since 4.9.2.0 is already released, please created 4.9.2.0->4.9.3.0 upgrade path and move the changes to the suggested sql file. This needs to go into master/4.10 as well /cc @karuturi ---

[GitHub] cloudstack issue #1914: CLOUDSTACK-9753 - Update L10N resource files with 4....

2017-02-12 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1914 LGTM, /cc @karuturi --- 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 s

[GitHub] cloudstack issue #1884: CLOUDSTACK-9699: Add global setting for enable/disab...

2017-02-12 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1884 I'm working on improving the metrics view feature by implementing the logic as backend APIs, will post a PR this/next week. Thanks. --- If your project is set up for it, you can reply to this ema

Re: apidocs build failure

2017-02-12 Thread Rohit Yadav
Jenkins need to have jdk8 available, someone need to setup jenv on it as well. (The first job in Travis does apidocs/marvin/rat related checks to validate changes and apidocs build). Regards. From: Rajani Karuturi Sent: 09 February 2017 12:21:40 To: dev@cloud

[GitHub] cloudstack issue #1934: [CLOUDSTACK-9772] Template: perform a HEAD request t...

2017-02-12 Thread marcaurele
Github user marcaurele commented on the issue: https://github.com/apache/cloudstack/pull/1934 @borisstoyanov Dunno what's wrong with the failed tests. Its' failing at https://github.com/apache/cloudstack/blob/master/plugins/storage/image/default/src/org/apache/cloudstack/storage/datast