[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/datastore/driver/CloudStackImageStoreDriverImpl.java#L83.
 The log says that the ssvm endpoint was not ready.
Could you try to launch another bueorangutan test suite?

We're running that fix in production since a week and haven't hit a problem 
yet.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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@cloudstack.apache.org
Subject: apidocs build failure

Hi all,

All the apidocs builds[1] are failing after the recent java 8
change. Can anyone having access fix it? Or should we talk to
INFRA about it?

Error message:

[INFO]
-
[ERROR] COMPILATION ERROR : [INFO]
-
[ERROR] javac: invalid target release: 1.8 Usage: javac use -help
for a list of possible options

[1] https://builds.apache.org/job/cloudstack-apidocs-master/

Thanks

~ Rajani

http://cloudplatform.accelerite.com/

rohit.ya...@shapeblue.com 
www.shapeblue.com
53 Chandos Place, Covent Garden, London  WC2N 4HSUK
@shapeblue

[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 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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[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 so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[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 


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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
>
> @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 this feature
> enabled and wishes so, or if the feature is enabled but not working, please
> contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
> with INFRA.
> ---
>

[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 this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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 will result in the packets matching the PF rule
being dropped.

3. Adding an ingress firewall rule without a corresponding port forwarding
rule will result in the packets matching the FW rule being dropped.

If any of these statements are incorrect can you please explain how the
router will actually behave in those circumstances?

Thanks!

-- 
Matthew Smart
President
Smart Software Solutions Inc.
108 S Pierre St.
Pierre, SD 57501

Phone: (605) 280-0383
Skype: msmart13
Email: msm...@smartsoftwareinc.com

[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) {
 }
 
 // Get the size of a file from URL response header.
-public static Long getRemoteSize(String url) {
-Long remoteSize = (long)0;
+public static long getRemoteSize(String url) {
--- End diff --

@marcaurele I think this needs to be changed back to Long, or else the 
method cannot be found as it is called with type Long.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[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. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---