[GitHub] [cloudstack] blueorangutan commented on pull request #4068: Adding Centos8 Support

2020-05-19 Thread GitBox
blueorangutan commented on pull request #4068: URL: https://github.com/apache/cloudstack/pull/4068#issuecomment-631057352 Packaging result: ✖centos7 ✔debian. JID-1233 This is an automated message from the Apache Git Service.

[GitHub] [cloudstack] blueorangutan commented on pull request #4068: Adding Centos8 Support

2020-05-19 Thread GitBox
blueorangutan commented on pull request #4068: URL: https://github.com/apache/cloudstack/pull/4068#issuecomment-631044894 @shwstppr a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] [cloudstack] shwstppr commented on pull request #4068: Adding Centos8 Support

2020-05-19 Thread GitBox
shwstppr commented on pull request #4068: URL: https://github.com/apache/cloudstack/pull/4068#issuecomment-631044318 @blueorangutan package This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [cloudstack] blueorangutan commented on pull request #4004: Fixed null pointer and deployment issue on Xenserver with L2 Guest network with configDrive

2020-05-19 Thread GitBox
blueorangutan commented on pull request #4004: URL: https://github.com/apache/cloudstack/pull/4004#issuecomment-630978353 Trillian test result (tid-1537) Environment: xenserver-71 (x2), Advanced Networking with Mgmt server 7 Total time taken: 30299 seconds Marvin logs:

[GitHub] [cloudstack] svenvogel commented on a change in pull request #4070: Update cloud-set-guest-password.in

2020-05-19 Thread GitBox
svenvogel commented on a change in pull request #4070: URL: https://github.com/apache/cloudstack/pull/4070#discussion_r427342669 ## File path: setup/bindir/cloud-set-guest-password.in ## @@ -26,25 +26,58 @@ # Modify this line to specify the user (default is root) user=root

[GitHub] [cloudstack] svenvogel commented on a change in pull request #4070: Update cloud-set-guest-password.in

2020-05-19 Thread GitBox
svenvogel commented on a change in pull request #4070: URL: https://github.com/apache/cloudstack/pull/4070#discussion_r427342669 ## File path: setup/bindir/cloud-set-guest-password.in ## @@ -26,25 +26,58 @@ # Modify this line to specify the user (default is root) user=root

[GitHub] [cloudstack] Pearl1594 commented on a change in pull request #4053: Secondary Storage Usage Improvements

2020-05-19 Thread GitBox
Pearl1594 commented on a change in pull request #4053: URL: https://github.com/apache/cloudstack/pull/4053#discussion_r427322103 ## File path: engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/DataMigrationUtility.java ## @@ -0,0 +1,262 @@ +//

[GitHub] [cloudstack] GabrielBrascher commented on issue #4088: UEFI boot in Legacy mode corrupts VSphere connection.

2020-05-19 Thread GitBox
GabrielBrascher commented on issue #4088: URL: https://github.com/apache/cloudstack/issues/4088#issuecomment-630798212 Not related with UEFI "only". > register a faulty template meant for virtualbox as a vmware OVA. > deploy using boot type UEFI and boot mode LEGACY. Please,

[GitHub] [cloudstack] dimsijul commented on a change in pull request #4070: Update cloud-set-guest-password.in

2020-05-19 Thread GitBox
dimsijul commented on a change in pull request #4070: URL: https://github.com/apache/cloudstack/pull/4070#discussion_r427275638 ## File path: setup/bindir/cloud-set-guest-password.in ## @@ -26,25 +26,58 @@ # Modify this line to specify the user (default is root) user=root

[GitHub] [cloudstack] GabrielBrascher commented on issue #4088: UEFI boot in Legacy mode corrupts VSphere connection.

2020-05-19 Thread GitBox
GabrielBrascher commented on issue #4088: URL: https://github.com/apache/cloudstack/issues/4088#issuecomment-630788682 @DaanHoogland I assume that this issue happens only with VMware. The UEFI implementation #3638 addressed VMware and KVM. Do you know if it has been tested and

[GitHub] [cloudstack] DaanHoogland commented on a change in pull request #4053: Secondary Storage Usage Improvements

2020-05-19 Thread GitBox
DaanHoogland commented on a change in pull request #4053: URL: https://github.com/apache/cloudstack/pull/4053#discussion_r427257039 ## File path: engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/DataMigrationUtility.java ## @@ -0,0 +1,262 @@ +//

[GitHub] [cloudstack] andrijapanicsb merged pull request #3967: noVNC console integration

2020-05-19 Thread GitBox
andrijapanicsb merged pull request #3967: URL: https://github.com/apache/cloudstack/pull/3967 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [cloudstack] andrijapanicsb commented on pull request #3967: noVNC console integration

2020-05-19 Thread GitBox
andrijapanicsb commented on pull request #3967: URL: https://github.com/apache/cloudstack/pull/3967#issuecomment-630778733 Travis log length exceeded - otherwise LGMT Merging based on 2 x LGTMs, manual testing by 2 persons (Vladimir and me) and the regression test results passing

[cloudstack-documentation] branch master updated: ref update (#124)

2020-05-19 Thread andrijapanic
This is an automated email from the ASF dual-hosted git repository. andrijapanic pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git The following commit(s) were added to refs/heads/master by this push: new 43462db ref update

[cloudstack-documentation] branch andrijapanicsb-patch-1 updated (5491c78 -> 680ac01)

2020-05-19 Thread andrijapanic
This is an automated email from the ASF dual-hosted git repository. andrijapanic pushed a change to branch andrijapanicsb-patch-1 in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git. from 5491c78 Update vsphere.rst add 680ac01 Update vsphere.rst No new

[cloudstack-documentation] branch andrijapanicsb-patch-1 updated (a053212 -> 5491c78)

2020-05-19 Thread andrijapanic
This is an automated email from the ASF dual-hosted git repository. andrijapanic pushed a change to branch andrijapanicsb-patch-1 in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git. from a053212 ref update2 add 5491c78 Update vsphere.rst No new revisions

[cloudstack-documentation] branch andrijapanicsb-patch-1 updated (16a66f1 -> a053212)

2020-05-19 Thread andrijapanic
This is an automated email from the ASF dual-hosted git repository. andrijapanic pushed a change to branch andrijapanicsb-patch-1 in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git. from 16a66f1 ref update add a053212 ref update2 No new revisions were

[cloudstack-documentation] branch andrijapanicsb-patch-1 created (now 16a66f1)

2020-05-19 Thread andrijapanic
This is an automated email from the ASF dual-hosted git repository. andrijapanic pushed a change to branch andrijapanicsb-patch-1 in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git. at 16a66f1 ref update This branch includes the following new commits:

[cloudstack-documentation] 01/01: ref update

2020-05-19 Thread andrijapanic
This is an automated email from the ASF dual-hosted git repository. andrijapanic pushed a commit to branch andrijapanicsb-patch-1 in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git commit 16a66f126f634472868555203d481938a5ee16eb Author: Andrija Panic

[GitHub] [cloudstack] DaanHoogland commented on pull request #4089: extra guard in vmwareresource

2020-05-19 Thread GitBox
DaanHoogland commented on pull request #4089: URL: https://github.com/apache/cloudstack/pull/4089#issuecomment-630742693 @pavanaravapalli unfortunately this change does not seem to give the system enough protection. I have been able to break it based on this PR :(

[GitHub] [cloudstack] DaanHoogland commented on pull request #4076: Support searching vm's by multiple id

2020-05-19 Thread GitBox
DaanHoogland commented on pull request #4076: URL: https://github.com/apache/cloudstack/pull/4076#issuecomment-630742220 @ravening, so did you find out what caused this to work this way? The chance seems so unrelated to your objectives that it reeks of an undesirable side effect. also,

[GitHub] [cloudstack] blueorangutan commented on pull request #4091: Revert: UEFI feature from #3638 and #3985

2020-05-19 Thread GitBox
blueorangutan commented on pull request #4091: URL: https://github.com/apache/cloudstack/pull/4091#issuecomment-630738917 Packaging result: ✔centos7 ✔debian. JID-1232 This is an automated message from the Apache Git Service.

[GitHub] [cloudstack] rhtyd commented on pull request #4068: Adding Centos8 Support

2020-05-19 Thread GitBox
rhtyd commented on pull request #4068: URL: https://github.com/apache/cloudstack/pull/4068#issuecomment-630732996 Notes on CentOS8: ``` network setup: nmcli conn show --active nmcli conn add type bridge autoconnect

[GitHub] [cloudstack] blueorangutan commented on pull request #4091: Revert: UEFI feature from #3638 and #3985

2020-05-19 Thread GitBox
blueorangutan commented on pull request #4091: URL: https://github.com/apache/cloudstack/pull/4091#issuecomment-630724836 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. This

[GitHub] [cloudstack] rhtyd commented on pull request #4091: Revert: UEFI feature from #3638 and #3985

2020-05-19 Thread GitBox
rhtyd commented on pull request #4091: URL: https://github.com/apache/cloudstack/pull/4091#issuecomment-630724538 @blueorangutan package This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [cloudstack] rhtyd opened a new pull request #4091: Revert: UEFI feature from #3638 and #3985

2020-05-19 Thread GitBox
rhtyd opened a new pull request #4091: URL: https://github.com/apache/cloudstack/pull/4091 This PR reverts #3638 and #3985 which we think is causing regression for Vmware wrt 4.14 release. If regression is confirmed to go away with the revert, we probably want to go ahead with 4.14

[GitHub] [cloudstack] svenvogel commented on a change in pull request #4070: Update cloud-set-guest-password.in

2020-05-19 Thread GitBox
svenvogel commented on a change in pull request #4070: URL: https://github.com/apache/cloudstack/pull/4070#discussion_r427165827 ## File path: setup/bindir/cloud-set-guest-password.in ## @@ -26,25 +26,58 @@ # Modify this line to specify the user (default is root) user=root

[GitHub] [cloudstack] davidjumani opened a new pull request #3967: noVNC console integration

2020-05-19 Thread GitBox
davidjumani opened a new pull request #3967: URL: https://github.com/apache/cloudstack/pull/3967 ## Description This feature provides novnc console for virtual machines. It is developed based on noVNC 1.1.0 It includes changes in cloudstack : (1) add a new websocket proxy to

[GitHub] [cloudstack] andrijapanicsb closed pull request #3967: noVNC console integration

2020-05-19 Thread GitBox
andrijapanicsb closed pull request #3967: URL: https://github.com/apache/cloudstack/pull/3967 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [cloudstack] DennisKonrad closed pull request #3316: Fix private gateway ACLs

2020-05-19 Thread GitBox
DennisKonrad closed pull request #3316: URL: https://github.com/apache/cloudstack/pull/3316 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [cloudstack] DennisKonrad commented on pull request #3316: Fix private gateway ACLs

2020-05-19 Thread GitBox
DennisKonrad commented on pull request #3316: URL: https://github.com/apache/cloudstack/pull/3316#issuecomment-630674210 Currently I can make no time to push this PR to a point where we could move on. I will open another PR when this is ready.

[GitHub] [cloudstack] blueorangutan commented on pull request #4004: Fixed null pointer and deployment issue on Xenserver with L2 Guest network with configDrive

2020-05-19 Thread GitBox
blueorangutan commented on pull request #4004: URL: https://github.com/apache/cloudstack/pull/4004#issuecomment-630663134 @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + xenserver-71) has been kicked to run smoke tests

[GitHub] [cloudstack] borisstoyanov commented on pull request #4004: Fixed null pointer and deployment issue on Xenserver with L2 Guest network with configDrive

2020-05-19 Thread GitBox
borisstoyanov commented on pull request #4004: URL: https://github.com/apache/cloudstack/pull/4004#issuecomment-630662783 @blueorangutan test centos7 xenserver-71 This is an automated message from the Apache Git Service. To

[GitHub] [cloudstack] blueorangutan commented on pull request #4004: Fixed null pointer and deployment issue on Xenserver with L2 Guest network with configDrive

2020-05-19 Thread GitBox
blueorangutan commented on pull request #4004: URL: https://github.com/apache/cloudstack/pull/4004#issuecomment-630662536 @borisstoyanov unsupported parameters provided. Supported mgmt server os are: `centos6, centos7, ubuntu`. Supported hypervisors are: `kvm-centos6, kvm-centos7,

[GitHub] [cloudstack] borisstoyanov commented on pull request #4004: Fixed null pointer and deployment issue on Xenserver with L2 Guest network with configDrive

2020-05-19 Thread GitBox
borisstoyanov commented on pull request #4004: URL: https://github.com/apache/cloudstack/pull/4004#issuecomment-630662250 @blueorangutan test centos7 xenserver70 This is an automated message from the Apache Git Service. To

[GitHub] [cloudstack] Pearl1594 commented on a change in pull request #4053: Secondary Storage Usage Improvements

2020-05-19 Thread GitBox
Pearl1594 commented on a change in pull request #4053: URL: https://github.com/apache/cloudstack/pull/4053#discussion_r427101218 ## File path: engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/DataMigrationUtility.java ## @@ -0,0 +1,262 @@ +//

[GitHub] [cloudstack] Pearl1594 commented on a change in pull request #4053: Secondary Storage Usage Improvements

2020-05-19 Thread GitBox
Pearl1594 commented on a change in pull request #4053: URL: https://github.com/apache/cloudstack/pull/4053#discussion_r427100775 ## File path: engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/DataMigrationUtility.java ## @@ -0,0 +1,262 @@ +//

[GitHub] [cloudstack] Pearl1594 commented on a change in pull request #4053: Secondary Storage Usage Improvements

2020-05-19 Thread GitBox
Pearl1594 commented on a change in pull request #4053: URL: https://github.com/apache/cloudstack/pull/4053#discussion_r427057420 ## File path: engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/DataMigrationUtility.java ## @@ -0,0 +1,262 @@ +//