[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

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

[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

[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

[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

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: > >

[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

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

[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

[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