[GitHub] blueorangutan commented on issue #2476: CLOUDSTACK-10317: Fix SNAT rules for additional public nics

2018-03-05 Thread GitBox
blueorangutan commented on issue #2476: CLOUDSTACK-10317: Fix SNAT rules for 
additional public nics
URL: https://github.com/apache/cloudstack/pull/2476#issuecomment-370646501
 
 
   Trillian test result (tid-2315)
   Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 35308 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2476-t2315-kvm-centos7.zip
   Intermitten failure detected: 
/marvin/tests/smoke/test_deploy_virtio_scsi_vm.py
   Intermitten failure detected: /marvin/tests/smoke/test_internal_lb.py
   Intermitten failure detected: /marvin/tests/smoke/test_public_ip_range.py
   Intermitten failure detected: /marvin/tests/smoke/test_templates.py
   Intermitten failure detected: /marvin/tests/smoke/test_usage.py
   Intermitten failure detected: /marvin/tests/smoke/test_volumes.py
   Intermitten failure detected: /marvin/tests/smoke/test_vpc_redundant.py
   Intermitten failure detected: /marvin/tests/smoke/test_hostha_kvm.py
   Smoke tests completed. 64 look OK, 3 have error(s)
   Only failed tests results shown below:
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_04_extract_template | `Failure` | 128.34 | test_templates.py
   ContextSuite context=TestISOUsage>:setup | `Error` | 0.00 | test_usage.py
   test_06_download_detached_volume | `Failure` | 138.71 | test_volumes.py
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] khos2ow commented on issue #2433: CLOUDSTACK-10268: Fix and enhance package script

2018-03-05 Thread GitBox
khos2ow commented on issue #2433: CLOUDSTACK-10268: Fix and enhance package 
script
URL: https://github.com/apache/cloudstack/pull/2433#issuecomment-370598448
 
 
   I'm +1 on testing (may be individually on different environments). I don't 
think automated test would give us much needed information anyway (it helps 
regardless anyway).
   What I would do is:
   
   - build with `--use-timestamp`
 - once RPMs
 - once DEBs
   - build systemvm. preferably with
   ```bash
   VERSION=$(mvn -q -Dexec.executable="echo" -Dexec.args='${project.version}' 
--non-recursive org.codehaus.mojo:exec-maven-plugin:1.3.1:exec)
   TIMESTAMP=$(date +%s)
   VERSION=$(echo $VERSION | sed 's/\-SNAPSHOT/\-'${TIMESTAMP}'/g')
   ./build.sh systemvmtemplate $VERSION $BUILD_NUMBER
   ```
   - manually deploy the whole thing
   
   What to look for:
   
   - .rpm or .deb file to have a name like `cloudstack-xxx-.[deb|rpm]`
   - when logged in the UI > About Cloudstack should have the full version 
`4.12.0.0-`
   - when ssh in the systemvm `cat /etc/cloudstack-release` should have the 
full version `4.12.0.0-`
   - from UI > Infrastructure > VR should detect correctly to upgrase
 - when upgraded should show the full version in the UI (`4.12.0.0-`)


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rafaelweingartner commented on issue #2478: [CLOUDSTACK-10318] Bug on sorting ACL rules list in chrome

2018-03-05 Thread GitBox
rafaelweingartner commented on issue #2478: [CLOUDSTACK-10318] Bug on sorting 
ACL rules list in chrome
URL: https://github.com/apache/cloudstack/pull/2478#issuecomment-370587137
 
 
   Thanks @DaanHoogland 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rafaelweingartner commented on issue #2433: CLOUDSTACK-10268: Fix and enhance package script

2018-03-05 Thread GitBox
rafaelweingartner commented on issue #2433: CLOUDSTACK-10268: Fix and enhance 
package script
URL: https://github.com/apache/cloudstack/pull/2433#issuecomment-370586983
 
 
   Well.. that version.java was changed, and I think that affects ACS rode when 
it deals with system VMs. What do you think?


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2448: CLOUDSTACK-10274: L2 network refused to be designed on VXLAN physical network

2018-03-05 Thread GitBox
blueorangutan commented on issue #2448: CLOUDSTACK-10274: L2 network refused to 
be designed on VXLAN physical network
URL: https://github.com/apache/cloudstack/pull/2448#issuecomment-370580552
 
 
   Trillian test result (tid-2314)
   Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 29989 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2448-t2314-kvm-centos7.zip
   Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
   Intermitten failure detected: /marvin/tests/smoke/test_ssvm.py
   Intermitten failure detected: /marvin/tests/smoke/test_hostha_kvm.py
   Smoke tests completed. 66 look OK, 1 have error(s)
   Only failed tests results shown below:
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_hostha_enable_ha_when_host_in_maintenance | `Error` | 1.12 | 
test_hostha_kvm.py
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] khos2ow commented on issue #2433: CLOUDSTACK-10268: Fix and enhance package script

2018-03-05 Thread GitBox
khos2ow commented on issue #2433: CLOUDSTACK-10268: Fix and enhance package 
script
URL: https://github.com/apache/cloudstack/pull/2433#issuecomment-370572358
 
 
   @rafaelweingartner yes, that's correct.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] DaanHoogland commented on issue #2433: CLOUDSTACK-10268: Fix and enhance package script

2018-03-05 Thread GitBox
DaanHoogland commented on issue #2433: CLOUDSTACK-10268: Fix and enhance 
package script
URL: https://github.com/apache/cloudstack/pull/2433#issuecomment-370571231
 
 
   No, I don't think production code has been touched. maybe test installs?


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] DaanHoogland commented on issue #2470: [CLOUDSTACK-10197] Update DisplayText of XenServer tools ISO entry in the database when it already exists

2018-03-05 Thread GitBox
DaanHoogland commented on issue #2470: [CLOUDSTACK-10197] Update DisplayText of 
XenServer tools ISO entry in the database when it already exists
URL: https://github.com/apache/cloudstack/pull/2470#issuecomment-370570524
 
 
   Yes @rafaelweingartner , I've seen this one a lot and seen it pass as well. 
Your code here is not related to that test afaik.
   @rhtyd @PaulAngus can you give second and third opinions?


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rafaelweingartner commented on issue #2470: [CLOUDSTACK-10197] Update DisplayText of XenServer tools ISO entry in the database when it already exists

2018-03-05 Thread GitBox
rafaelweingartner commented on issue #2470: [CLOUDSTACK-10197] Update 
DisplayText of XenServer tools ISO entry in the database when it already exists
URL: https://github.com/apache/cloudstack/pull/2470#issuecomment-370551424
 
 
   @DaanHoogland is this "test_hostha_enable_ha_when_host_in_maintenance" an 
intermittent error? I have seen this on #2425, which is a very different PR.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2470: [CLOUDSTACK-10197] Update DisplayText of XenServer tools ISO entry in the database when it already exists

2018-03-05 Thread GitBox
blueorangutan commented on issue #2470: [CLOUDSTACK-10197] Update DisplayText 
of XenServer tools ISO entry in the database when it already exists
URL: https://github.com/apache/cloudstack/pull/2470#issuecomment-370542094
 
 
   Trillian test result (tid-2313)
   Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 29032 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2470-t2313-kvm-centos7.zip
   Intermitten failure detected: /marvin/tests/smoke/test_hostha_kvm.py
   Smoke tests completed. 66 look OK, 1 have error(s)
   Only failed tests results shown below:
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_hostha_enable_ha_when_host_in_maintenance | `Error` | 1.03 | 
test_hostha_kvm.py
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rafaelweingartner commented on issue #2433: CLOUDSTACK-10268: Fix and enhance package script

2018-03-05 Thread GitBox
rafaelweingartner commented on issue #2433: CLOUDSTACK-10268: Fix and enhance 
package script
URL: https://github.com/apache/cloudstack/pull/2433#issuecomment-370528832
 
 
   @DaanHoogland thanks for the update here. So, should we update for you to 
run the tests as well?


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rafaelweingartner commented on issue #2433: CLOUDSTACK-10268: Fix and enhance package script

2018-03-05 Thread GitBox
rafaelweingartner commented on issue #2433: CLOUDSTACK-10268: Fix and enhance 
package script
URL: https://github.com/apache/cloudstack/pull/2433#issuecomment-370528832
 
 
   @DaanHoogland thanks for the update here. So, should we wait for you to run 
the tests as well?


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rafaelweingartner opened a new pull request #2478: [CLOUDSTACK-10318] Bug on sorting ACL rules list in chrome

2018-03-05 Thread GitBox
rafaelweingartner opened a new pull request #2478: [CLOUDSTACK-10318] Bug on 
sorting ACL rules list in chrome
URL: https://github.com/apache/cloudstack/pull/2478
 
 
   There is a bug on ACL rules list when a user creates more than 10 rules.
   
   Steps to reproduce this issue:
   - Create more than 10 ACL rules.
   - Access the ACL rules list page.
   - You will see that they are not sorted as they should by the ACL rule number
   
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2468: [4.11] VR: minor fixes

2018-03-05 Thread GitBox
blueorangutan commented on issue #2468: [4.11] VR: minor fixes
URL: https://github.com/apache/cloudstack/pull/2468#issuecomment-370497093
 
 
   Packaging result: ?centos6 ?centos7 ?debian. JID-1754


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] PaulAngus opened a new pull request #2477: Smaller systemvm images using sparse disks.

2018-03-05 Thread GitBox
PaulAngus opened a new pull request #2477: Smaller systemvm images using sparse 
disks.
URL: https://github.com/apache/cloudstack/pull/2477
 
 
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2425: [CLOUDSTACK-10240] ACS cannot migrate a local volume to shared storage

2018-03-05 Thread GitBox
blueorangutan commented on issue #2425: [CLOUDSTACK-10240] ACS cannot migrate a 
local volume to shared storage
URL: https://github.com/apache/cloudstack/pull/2425#issuecomment-370493732
 
 
   Trillian test result (tid-2312)
   Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 28936 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2425-t2312-kvm-centos7.zip
   Intermitten failure detected: 
/marvin/tests/smoke/test_deploy_virtio_scsi_vm.py
   Intermitten failure detected: /marvin/tests/smoke/test_outofbandmanagement.py
   Intermitten failure detected: /marvin/tests/smoke/test_vpc_redundant.py
   Intermitten failure detected: /marvin/tests/smoke/test_vpc_vpn.py
   Intermitten failure detected: /marvin/tests/smoke/test_hostha_kvm.py
   Smoke tests completed. 66 look OK, 1 have error(s)
   Only failed tests results shown below:
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_hostha_enable_ha_when_host_in_maintenance | `Error` | 1.86 | 
test_hostha_kvm.py
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rafaelweingartner commented on a change in pull request #2476: CLOUDSTACK-10317: Fix SNAT rules for additional public nics

2018-03-05 Thread GitBox
rafaelweingartner commented on a change in pull request #2476: 
CLOUDSTACK-10317: Fix SNAT rules for additional public nics
URL: https://github.com/apache/cloudstack/pull/2476#discussion_r172254964
 
 

 ##
 File path: systemvm/debian/opt/cloud/bin/cs/CsAddress.py
 ##
 @@ -388,7 +388,7 @@ def fw_router(self):
 self.fw.append(["mangle", "",
 "-A VPN_%s -j RETURN" % self.address['public_ip']])
 self.fw.append(["nat", "",
-"-A POSTROUTING -o eth2 -j SNAT --to-source %s" % 
self.address['public_ip']])
+"-A POSTROUTING -o %s -j SNAT --to-source %s" % 
(self.dev, self.address['public_ip'])])
 
 Review comment:
   Thanks for the explanation.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on a change in pull request #2476: CLOUDSTACK-10317: Fix SNAT rules for additional public nics

2018-03-05 Thread GitBox
rhtyd commented on a change in pull request #2476: CLOUDSTACK-10317: Fix SNAT 
rules for additional public nics
URL: https://github.com/apache/cloudstack/pull/2476#discussion_r172254521
 
 

 ##
 File path: systemvm/debian/opt/cloud/bin/cs/CsAddress.py
 ##
 @@ -388,7 +388,7 @@ def fw_router(self):
 self.fw.append(["mangle", "",
 "-A VPN_%s -j RETURN" % self.address['public_ip']])
 self.fw.append(["nat", "",
-"-A POSTROUTING -o eth2 -j SNAT --to-source %s" % 
self.address['public_ip']])
+"-A POSTROUTING -o %s -j SNAT --to-source %s" % 
(self.dev, self.address['public_ip'])])
 
 Review comment:
   Yes @rafaelweingartner, the logic is same what as in vpc routers, see - 
https://github.com/apache/cloudstack/blob/4.11/systemvm/debian/opt/cloud/bin/cs/CsAddress.py#L584


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on a change in pull request #2476: CLOUDSTACK-10317: Fix SNAT rules for additional public nics

2018-03-05 Thread GitBox
rhtyd commented on a change in pull request #2476: CLOUDSTACK-10317: Fix SNAT 
rules for additional public nics
URL: https://github.com/apache/cloudstack/pull/2476#discussion_r172254521
 
 

 ##
 File path: systemvm/debian/opt/cloud/bin/cs/CsAddress.py
 ##
 @@ -388,7 +388,7 @@ def fw_router(self):
 self.fw.append(["mangle", "",
 "-A VPN_%s -j RETURN" % self.address['public_ip']])
 self.fw.append(["nat", "",
-"-A POSTROUTING -o eth2 -j SNAT --to-source %s" % 
self.address['public_ip']])
+"-A POSTROUTING -o %s -j SNAT --to-source %s" % 
(self.dev, self.address['public_ip'])])
 
 Review comment:
   Yes @rafaelweingartner, the logic is same what is being used in vpc routers, 
see - 
https://github.com/apache/cloudstack/blob/4.11/systemvm/debian/opt/cloud/bin/cs/CsAddress.py#L584


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2468: [4.11] VR: minor fixes

2018-03-05 Thread GitBox
blueorangutan commented on issue #2468: [4.11] VR: minor fixes
URL: https://github.com/apache/cloudstack/pull/2468#issuecomment-370485370
 
 
   @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted 
as I make progress.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on issue #2468: [4.11] VR: minor fixes

2018-03-05 Thread GitBox
rhtyd commented on issue #2468: [4.11] VR: minor fixes
URL: https://github.com/apache/cloudstack/pull/2468#issuecomment-370485137
 
 
   @blueorangutan package
   
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rafaelweingartner commented on a change in pull request #2476: CLOUDSTACK-10317: Fix SNAT rules for additional public nics

2018-03-05 Thread GitBox
rafaelweingartner commented on a change in pull request #2476: 
CLOUDSTACK-10317: Fix SNAT rules for additional public nics
URL: https://github.com/apache/cloudstack/pull/2476#discussion_r172252984
 
 

 ##
 File path: systemvm/debian/opt/cloud/bin/cs/CsAddress.py
 ##
 @@ -388,7 +388,7 @@ def fw_router(self):
 self.fw.append(["mangle", "",
 "-A VPN_%s -j RETURN" % self.address['public_ip']])
 self.fw.append(["nat", "",
-"-A POSTROUTING -o eth2 -j SNAT --to-source %s" % 
self.address['public_ip']])
+"-A POSTROUTING -o %s -j SNAT --to-source %s" % 
(self.dev, self.address['public_ip'])])
 
 Review comment:
   The attribute `self.dev` is already populated with the correct device name?


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2476: CLOUDSTACK-10317: Fix SNAT rules for additional public nics

2018-03-05 Thread GitBox
blueorangutan commented on issue #2476: CLOUDSTACK-10317: Fix SNAT rules for 
additional public nics
URL: https://github.com/apache/cloudstack/pull/2476#issuecomment-370483999
 
 
   @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been 
kicked to run smoke tests


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on issue #2476: CLOUDSTACK-10317: Fix SNAT rules for additional public nics

2018-03-05 Thread GitBox
rhtyd commented on issue #2476: CLOUDSTACK-10317: Fix SNAT rules for additional 
public nics
URL: https://github.com/apache/cloudstack/pull/2476#issuecomment-370483846
 
 
   @blueorangutan test


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2476: CLOUDSTACK-10317: Fix SNAT rules for additional public nics

2018-03-05 Thread GitBox
blueorangutan commented on issue #2476: CLOUDSTACK-10317: Fix SNAT rules for 
additional public nics
URL: https://github.com/apache/cloudstack/pull/2476#issuecomment-370482352
 
 
   Packaging result: ?centos6 ?centos7 ?debian. JID-1753


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2476: CLOUDSTACK-10317: Fix SNAT rules for additional public nics

2018-03-05 Thread GitBox
blueorangutan commented on issue #2476: CLOUDSTACK-10317: Fix SNAT rules for 
additional public nics
URL: https://github.com/apache/cloudstack/pull/2476#issuecomment-370468577
 
 
   @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted 
as I make progress.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on issue #2476: CLOUDSTACK-10317: Fix SNAT rules for additional public nics

2018-03-05 Thread GitBox
rhtyd commented on issue #2476: CLOUDSTACK-10317: Fix SNAT rules for additional 
public nics
URL: https://github.com/apache/cloudstack/pull/2476#issuecomment-370468520
 
 
   @blueorangutan package


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on issue #2476: CLOUDSTACK-10317: Fix SNAT rules for additional public nics

2018-03-05 Thread GitBox
rhtyd commented on issue #2476: CLOUDSTACK-10317: Fix SNAT rules for additional 
public nics
URL: https://github.com/apache/cloudstack/pull/2476#issuecomment-370467906
 
 
   Same logic in vpc:
   
https://github.com/apache/cloudstack/blob/4.11/systemvm/debian/opt/cloud/bin/cs/CsAddress.py#L584


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on issue #2476: CLOUDSTACK-10317: Fix SNAT rules for additional public nics

2018-03-05 Thread GitBox
rhtyd commented on issue #2476: CLOUDSTACK-10317: Fix SNAT rules for additional 
public nics
URL: https://github.com/apache/cloudstack/pull/2476#issuecomment-370467906
 
 
   Same logic in vpc: 
https://github.com/apache/cloudstack/blob/4.11/systemvm/debian/opt/cloud/bin/cs/CsAddress.py#L480


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd opened a new pull request #2476: CLOUDSTACK-10317: Fix SNAT rules for additional public nics

2018-03-05 Thread GitBox
rhtyd opened a new pull request #2476: CLOUDSTACK-10317: Fix SNAT rules for 
additional public nics
URL: https://github.com/apache/cloudstack/pull/2476
 
 
   This allows networks with additional public nics to have correct SNAT 
iptables rules applied on configuration.
   
   Pinging for review - @ustcweizhou @PaulAngus @DagSonstebo and others.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2448: CLOUDSTACK-10274: L2 network refused to be designed on VXLAN physical network

2018-03-05 Thread GitBox
blueorangutan commented on issue #2448: CLOUDSTACK-10274: L2 network refused to 
be designed on VXLAN physical network
URL: https://github.com/apache/cloudstack/pull/2448#issuecomment-370412972
 
 
   @nvazquez a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been 
kicked to run smoke tests


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] nvazquez commented on issue #2448: CLOUDSTACK-10274: L2 network refused to be designed on VXLAN physical network

2018-03-05 Thread GitBox
nvazquez commented on issue #2448: CLOUDSTACK-10274: L2 network refused to be 
designed on VXLAN physical network
URL: https://github.com/apache/cloudstack/pull/2448#issuecomment-370412798
 
 
   @blueorangutan test


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2448: CLOUDSTACK-10274: L2 network refused to be designed on VXLAN physical network

2018-03-05 Thread GitBox
blueorangutan commented on issue #2448: CLOUDSTACK-10274: L2 network refused to 
be designed on VXLAN physical network
URL: https://github.com/apache/cloudstack/pull/2448#issuecomment-370411609
 
 
   Packaging result: ?centos6 ?centos7 ?debian. JID-1752


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rafaelweingartner commented on issue #2475: [CLOUDSTACK-10314] Add Text-Field to each ACL Rule

2018-03-05 Thread GitBox
rafaelweingartner commented on issue #2475: [CLOUDSTACK-10314] Add Text-Field 
to each ACL Rule
URL: https://github.com/apache/cloudstack/pull/2475#issuecomment-370405145
 
 
   I was checking that tag thing there, and it is a bit too generic in my 
opinion. I could change the UI to use those fields (I would also need to 
increase the size of those fields), but I do prefer a not so generic solution 
as this one, where the description/history field becomes part of the ACL rule 
API.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2448: CLOUDSTACK-10274: L2 network refused to be designed on VXLAN physical network

2018-03-05 Thread GitBox
blueorangutan commented on issue #2448: CLOUDSTACK-10274: L2 network refused to 
be designed on VXLAN physical network
URL: https://github.com/apache/cloudstack/pull/2448#issuecomment-370402817
 
 
   @nvazquez a Jenkins job has been kicked to build packages. I'll keep you 
posted as I make progress.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] nvazquez commented on issue #2448: CLOUDSTACK-10274: L2 network refused to be designed on VXLAN physical network

2018-03-05 Thread GitBox
nvazquez commented on issue #2448: CLOUDSTACK-10274: L2 network refused to be 
designed on VXLAN physical network
URL: https://github.com/apache/cloudstack/pull/2448#issuecomment-370402683
 
 
   @blueorangutan package


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rafaelweingartner commented on issue #2475: [CLOUDSTACK-10314] Add Text-Field to each ACL Rule

2018-03-05 Thread GitBox
rafaelweingartner commented on issue #2475: [CLOUDSTACK-10314] Add Text-Field 
to each ACL Rule
URL: https://github.com/apache/cloudstack/pull/2475#issuecomment-370391836
 
 
   So, that option has nothing to do with ACS tagging system?
   I meant those tags used in hosts and storage.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rafaelweingartner commented on issue #2475: [CLOUDSTACK-10314] Add Text-Field to each ACL Rule

2018-03-05 Thread GitBox
rafaelweingartner commented on issue #2475: [CLOUDSTACK-10314] Add Text-Field 
to each ACL Rule
URL: https://github.com/apache/cloudstack/pull/2475#issuecomment-370391836
 
 
   So, that option has nothing to do with ACS tagging system?
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] DaanHoogland commented on issue #2475: [CLOUDSTACK-10314] Add Text-Field to each ACL Rule

2018-03-05 Thread GitBox
DaanHoogland commented on issue #2475: [CLOUDSTACK-10314] Add Text-Field to 
each ACL Rule
URL: https://github.com/apache/cloudstack/pull/2475#issuecomment-370391319
 
 
   ok, so there is a UI issue there. It is meant to be a an audit trail on 
updates.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2470: [CLOUDSTACK-10197] Update DisplayText of XenServer tools ISO entry in the database when it already exists

2018-03-05 Thread GitBox
blueorangutan commented on issue #2470: [CLOUDSTACK-10197] Update DisplayText 
of XenServer tools ISO entry in the database when it already exists
URL: https://github.com/apache/cloudstack/pull/2470#issuecomment-370388632
 
 
   @DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has 
been kicked to run smoke tests


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] DaanHoogland commented on issue #2470: [CLOUDSTACK-10197] Update DisplayText of XenServer tools ISO entry in the database when it already exists

2018-03-05 Thread GitBox
DaanHoogland commented on issue #2470: [CLOUDSTACK-10197] Update DisplayText of 
XenServer tools ISO entry in the database when it already exists
URL: https://github.com/apache/cloudstack/pull/2470#issuecomment-370388456
 
 
   @blueorangutan test


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] nvazquez commented on issue #2448: CLOUDSTACK-10274: L2 network refused to be designed on VXLAN physical network

2018-03-05 Thread GitBox
nvazquez commented on issue #2448: CLOUDSTACK-10274: L2 network refused to be 
designed on VXLAN physical network
URL: https://github.com/apache/cloudstack/pull/2448#issuecomment-370386912
 
 
   Yes @NuxRo, it can be ignored as gurus are iterated, asking each one of them 
if they can design the network. Before reaching VxlanGuestNetworkGuru and 
successfully design it, it tried BigSwitchBcfGuestNetworkGuru (and others) and 
refused


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] nvazquez commented on issue #2448: CLOUDSTACK-10274: L2 network refused to be designed on VXLAN physical network

2018-03-05 Thread GitBox
nvazquez commented on issue #2448: CLOUDSTACK-10274: L2 network refused to be 
designed on VXLAN physical network
URL: https://github.com/apache/cloudstack/pull/2448#issuecomment-370386912
 
 
   Yes @NuxRo, it can be ignored as gurus are iterated, asking each one of them 
if they can design the network. Before reaching VxlanGuestNetworkGuru and 
successfully design it, it tried BigSwitchBcfGuestNetworkGuru and refused


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] NuxRo commented on issue #2448: CLOUDSTACK-10274: L2 network refused to be designed on VXLAN physical network

2018-03-05 Thread GitBox
NuxRo commented on issue #2448: CLOUDSTACK-10274: L2 network refused to be 
designed on VXLAN physical network
URL: https://github.com/apache/cloudstack/pull/2448#issuecomment-370381293
 
 
   I have tested this and can confirm I can now create VXLAN based L2 and high 
ID numbers, thanks!
   
   I still see complaints like this, but I guess it's safe to ignore since I 
don't use BigSwitch:
   
   2018-03-05 10:47:43,096 DEBUG [c.c.n.g.BigSwitchBcfGuestNetworkGuru] 
(qtp788117692-288:ctx-822f75f3 ctx-8f100d92) (logid:ac2301da) Refusing to 
design this network, the physical isolation type is not BCF_SEGMENT
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rafaelweingartner commented on issue #2475: [CLOUDSTACK-10314] Add Text-Field to each ACL Rule

2018-03-05 Thread GitBox
rafaelweingartner commented on issue #2475: [CLOUDSTACK-10314] Add Text-Field 
to each ACL Rule
URL: https://github.com/apache/cloudstack/pull/2475#issuecomment-370372894
 
 
   @DaanHoogland yes I did, but I did not quite understand their purpose. I was 
not sure if the goal was to be used as the tag system we have in ACS for host 
and storage. Also, they are quite limited in the UI if you need to write 
something more than a few words.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rafaelweingartner commented on issue #2470: [CLOUDSTACK-10197] Update DisplayText of XenServer tools ISO entry in the database when it already exists

2018-03-05 Thread GitBox
rafaelweingartner commented on issue #2470: [CLOUDSTACK-10197] Update 
DisplayText of XenServer tools ISO entry in the database when it already exists
URL: https://github.com/apache/cloudstack/pull/2470#issuecomment-370372305
 
 
   @DaanHoogland no it does not ;)


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2448: CLOUDSTACK-10274: L2 network refused to be designed on VXLAN physical network

2018-03-05 Thread GitBox
blueorangutan commented on issue #2448: CLOUDSTACK-10274: L2 network refused to 
be designed on VXLAN physical network
URL: https://github.com/apache/cloudstack/pull/2448#issuecomment-370363253
 
 
   Packaging result: ?centos6 ?centos7 ?debian. JID-1751


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2448: CLOUDSTACK-10274: L2 network refused to be designed on VXLAN physical network

2018-03-05 Thread GitBox
blueorangutan commented on issue #2448: CLOUDSTACK-10274: L2 network refused to 
be designed on VXLAN physical network
URL: https://github.com/apache/cloudstack/pull/2448#issuecomment-370353620
 
 
   @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted 
as I make progress.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on issue #2448: CLOUDSTACK-10274: L2 network refused to be designed on VXLAN physical network

2018-03-05 Thread GitBox
rhtyd commented on issue #2448: CLOUDSTACK-10274: L2 network refused to be 
designed on VXLAN physical network
URL: https://github.com/apache/cloudstack/pull/2448#issuecomment-370353531
 
 
   @blueorangutan package


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2425: [CLOUDSTACK-10240] ACS cannot migrate a local volume to shared storage

2018-03-05 Thread GitBox
blueorangutan commented on issue #2425: [CLOUDSTACK-10240] ACS cannot migrate a 
local volume to shared storage
URL: https://github.com/apache/cloudstack/pull/2425#issuecomment-370348709
 
 
   @DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has 
been kicked to run smoke tests


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] DaanHoogland commented on issue #2425: [CLOUDSTACK-10240] ACS cannot migrate a local volume to shared storage

2018-03-05 Thread GitBox
DaanHoogland commented on issue #2425: [CLOUDSTACK-10240] ACS cannot migrate a 
local volume to shared storage
URL: https://github.com/apache/cloudstack/pull/2425#issuecomment-370348656
 
 
   :( jenkins had crashed due to disk full, should not be related and should 
not be. acting insane:
   @blueorangutan test


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2470: [CLOUDSTACK-10197] Update DisplayText of XenServer tools ISO entry in the database when it already exists

2018-03-05 Thread GitBox
blueorangutan commented on issue #2470: [CLOUDSTACK-10197] Update DisplayText 
of XenServer tools ISO entry in the database when it already exists
URL: https://github.com/apache/cloudstack/pull/2470#issuecomment-370338595
 
 
   Packaging result: ?centos6 ?centos7 ?debian. JID-1750


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services