[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

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

[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

[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

[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 +

[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

[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

[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

[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?

[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.

[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

[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

[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

[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

[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

[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.

[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 ;)

[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

[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.

[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

[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

[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

[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

[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.

[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

[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

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

[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.

[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

[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

[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

[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

[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

[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

[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

[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

[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,

[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

[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

[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

[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?

[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?

[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),

[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

[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?

[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

[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

[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?

[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

[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

[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

[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