[GitHub] [cloudstack] DaanHoogland commented on a change in pull request #4467: vpc: fix ips on wrong interfaces after rebooting vpc vrs

2020-11-13 Thread GitBox
DaanHoogland commented on a change in pull request #4467: URL: https://github.com/apache/cloudstack/pull/4467#discussion_r522763710 ## File path: server/src/main/java/com/cloud/network/router/VpcVirtualNetworkApplianceManagerImpl.java ## @@ -834,4 +837,11 @@ public boolean pos

[GitHub] [cloudstack] DaanHoogland merged pull request #4464: Fix IndexOutOfBoundsException when creating basic network

2020-11-13 Thread GitBox
DaanHoogland merged pull request #4464: URL: https://github.com/apache/cloudstack/pull/4464 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 t

[GitHub] [cloudstack] DaanHoogland commented on pull request #4464: Fix IndexOutOfBoundsException when creating basic network

2020-11-13 Thread GitBox
DaanHoogland commented on pull request #4464: URL: https://github.com/apache/cloudstack/pull/4464#issuecomment-726591427 taking a leap here as it only adds a guard extra. we'll have to merge forward and test the result. This

[cloudstack] branch 4.13 updated: Fix IndexOutOfBoundsException when creating basic network (#4464)

2020-11-13 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a commit to branch 4.13 in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/4.13 by this push: new cc8bee7 Fix IndexOutOfBoundsException when creati

[GitHub] [cloudstack] borisstoyanov commented on issue #4462: restore vm does not attach back data disks

2020-11-13 Thread GitBox
borisstoyanov commented on issue #4462: URL: https://github.com/apache/cloudstack/issues/4462#issuecomment-726602977 Sounds to me as a bug to the restore functionality. Further if @weizhouapache cannot reproduce in 4.14 maybe it's a regerssion, since I've tested with latest Master.

[GitHub] [cloudstack] DaanHoogland closed issue #3784: systemVM stuck in starting state

2020-11-13 Thread GitBox
DaanHoogland closed issue #3784: URL: https://github.com/apache/cloudstack/issues/3784 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 to go

[GitHub] [cloudstack] DaanHoogland commented on issue #3784: systemVM stuck in starting state

2020-11-13 Thread GitBox
DaanHoogland commented on issue #3784: URL: https://github.com/apache/cloudstack/issues/3784#issuecomment-726615825 tested on latest healthcheck build, stopped and started the host that contained cpvm out of bound. no problem found -

[GitHub] [cloudstack] Pearl1594 opened a new pull request #4469: Regression fix

2020-11-13 Thread GitBox
Pearl1594 opened a new pull request #4469: URL: https://github.com/apache/cloudstack/pull/4469 ## Description A few changes made to `server/src/main/java/com/cloud/api/query/QueryManagerImpl.java` were reverted by commit: `fc05d3168f14584431a1a6b3c1f88cf04999cfaa` ## Types of

[GitHub] [cloudstack] Pearl1594 commented on pull request #4469: Regression fix

2020-11-13 Thread GitBox
Pearl1594 commented on pull request #4469: URL: https://github.com/apache/cloudstack/pull/4469#issuecomment-726622529 @blueorangutan package This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [cloudstack] blueorangutan commented on pull request #4469: Regression fix

2020-11-13 Thread GitBox
blueorangutan commented on pull request #4469: URL: https://github.com/apache/cloudstack/pull/4469#issuecomment-726623355 @Pearl1594 a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. T

[GitHub] [cloudstack] DaanHoogland commented on issue #4346: SQLIntegrityConstraintViolationException: Duplicate entry 's-190-VM-PremiumSecondaryStorageResource' for key 'guid'

2020-11-13 Thread GitBox
DaanHoogland commented on issue #4346: URL: https://github.com/apache/cloudstack/issues/4346#issuecomment-726629592 @borisstoyanov i ran this test but no new ssvm was automatically started and as I registered a template while the ost was down, as soon as it came up the ssvm was functional

[GitHub] [cloudstack] blueorangutan commented on pull request #4469: Fix: Listing projects comprising of only the user's on listAll=true

2020-11-13 Thread GitBox
blueorangutan commented on pull request #4469: URL: https://github.com/apache/cloudstack/pull/4469#issuecomment-726649028 Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2379 This is an automated message from the Apache Git

[GitHub] [cloudstack] Pearl1594 commented on pull request #4469: Fix: Listing projects comprising of only the user's on listAll=true

2020-11-13 Thread GitBox
Pearl1594 commented on pull request #4469: URL: https://github.com/apache/cloudstack/pull/4469#issuecomment-726650156 @blueorangutan package This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [cloudstack] blueorangutan commented on pull request #4469: Fix: Listing projects comprising of only the user's on listAll=true

2020-11-13 Thread GitBox
blueorangutan commented on pull request #4469: URL: https://github.com/apache/cloudstack/pull/4469#issuecomment-726651485 @Pearl1594 a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. T

[GitHub] [cloudstack] blueorangutan commented on pull request #4469: Fix: Listing projects comprising of only the user's on listAll=true

2020-11-13 Thread GitBox
blueorangutan commented on pull request #4469: URL: https://github.com/apache/cloudstack/pull/4469#issuecomment-726668919 Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2380 This is an automated message from the Apache Git

[GitHub] [cloudstack] DaanHoogland commented on a change in pull request #4469: Fix: Listing projects comprising of only the user's on listAll=true

2020-11-13 Thread GitBox
DaanHoogland commented on a change in pull request #4469: URL: https://github.com/apache/cloudstack/pull/4469#discussion_r522848665 ## File path: api/src/main/java/org/apache/cloudstack/api/command/user/project/CreateProjectCmd.java ## @@ -116,7 +116,7 @@ public long getEntity

[GitHub] [cloudstack] DaanHoogland commented on a change in pull request #4469: Fix: Listing projects comprising of only the user's on listAll=true

2020-11-13 Thread GitBox
DaanHoogland commented on a change in pull request #4469: URL: https://github.com/apache/cloudstack/pull/4469#discussion_r522848532 ## File path: api/src/main/java/org/apache/cloudstack/api/command/user/project/CreateProjectCmd.java ## @@ -116,7 +116,7 @@ public long getEntity

[GitHub] [cloudstack] Pearl1594 commented on a change in pull request #4469: Fix: Listing projects comprising of only the user's on listAll=true

2020-11-13 Thread GitBox
Pearl1594 commented on a change in pull request #4469: URL: https://github.com/apache/cloudstack/pull/4469#discussion_r522854520 ## File path: api/src/main/java/org/apache/cloudstack/api/command/user/project/CreateProjectCmd.java ## @@ -116,7 +116,7 @@ public long getEntityOwn

[GitHub] [cloudstack] Pearl1594 commented on pull request #4469: Fix: Listing projects comprising of only the user's on listAll=true

2020-11-13 Thread GitBox
Pearl1594 commented on pull request #4469: URL: https://github.com/apache/cloudstack/pull/4469#issuecomment-726677824 @blueorangutan test This is an automated message from the Apache Git Service. To respond to the message, pl

[GitHub] [cloudstack] blueorangutan commented on pull request #4469: Fix: Listing projects comprising of only the user's on listAll=true

2020-11-13 Thread GitBox
blueorangutan commented on pull request #4469: URL: https://github.com/apache/cloudstack/pull/4469#issuecomment-726678226 @Pearl1594 a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] [cloudstack] Pearl1594 commented on a change in pull request #4469: Fix: Listing projects comprising of only the user's on listAll=true

2020-11-13 Thread GitBox
Pearl1594 commented on a change in pull request #4469: URL: https://github.com/apache/cloudstack/pull/4469#discussion_r522854520 ## File path: api/src/main/java/org/apache/cloudstack/api/command/user/project/CreateProjectCmd.java ## @@ -116,7 +116,7 @@ public long getEntityOwn

[GitHub] [cloudstack] DaanHoogland commented on a change in pull request #4469: Fix: Listing projects comprising of only the user's on listAll=true

2020-11-13 Thread GitBox
DaanHoogland commented on a change in pull request #4469: URL: https://github.com/apache/cloudstack/pull/4469#discussion_r522866128 ## File path: api/src/main/java/org/apache/cloudstack/api/command/user/project/CreateProjectCmd.java ## @@ -116,7 +116,7 @@ public long getEntity

[GitHub] [cloudstack] DaanHoogland commented on pull request #4428: Moved dedicated hosts to the end of the resultset when selecting an e…

2020-11-13 Thread GitBox
DaanHoogland commented on pull request #4428: URL: https://github.com/apache/cloudstack/pull/4428#issuecomment-726712458 @blueorangutan package This is an automated message from the Apache Git Service. To respond to the messa

[GitHub] [cloudstack] blueorangutan commented on pull request #4428: Moved dedicated hosts to the end of the resultset when selecting an e…

2020-11-13 Thread GitBox
blueorangutan commented on pull request #4428: URL: https://github.com/apache/cloudstack/pull/4428#issuecomment-726712859 @DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. ---

[GitHub] [cloudstack] blueorangutan commented on pull request #4428: Moved dedicated hosts to the end of the resultset when selecting an e…

2020-11-13 Thread GitBox
blueorangutan commented on pull request #4428: URL: https://github.com/apache/cloudstack/pull/4428#issuecomment-726726974 Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2381 This is an automated message from the Apache Git

[GitHub] [cloudstack] weizhouapache commented on a change in pull request #4467: vpc: fix ips on wrong interfaces after rebooting vpc vrs

2020-11-13 Thread GitBox
weizhouapache commented on a change in pull request #4467: URL: https://github.com/apache/cloudstack/pull/4467#discussion_r522930767 ## File path: server/src/main/java/com/cloud/network/router/VpcVirtualNetworkApplianceManagerImpl.java ## @@ -834,4 +837,11 @@ public boolean po

[GitHub] [cloudstack] DaanHoogland commented on pull request #4467: vpc: fix ips on wrong interfaces after rebooting vpc vrs

2020-11-13 Thread GitBox
DaanHoogland commented on pull request #4467: URL: https://github.com/apache/cloudstack/pull/4467#issuecomment-726760378 @blueorangutan package This is an automated message from the Apache Git Service. To respond to the messa

[GitHub] [cloudstack] blueorangutan commented on pull request #4467: vpc: fix ips on wrong interfaces after rebooting vpc vrs

2020-11-13 Thread GitBox
blueorangutan commented on pull request #4467: URL: https://github.com/apache/cloudstack/pull/4467#issuecomment-726760743 @DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. ---

[cloudstack] branch 4.14 updated (9a253c4 -> db3c12d)

2020-11-13 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a change to branch 4.14 in repository https://gitbox.apache.org/repos/asf/cloudstack.git. from 9a253c4 Preventing port 53 being added as lb rule when dns service is availab… (#4442) add cc8bee7 Fix IndexO

[cloudstack] branch master updated (a348afe -> 79e46f5)

2020-11-13 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack.git. from a348afe default teardown methods with reversed() handling (#4289) add cc8bee7 Fix IndexOutOfBoundsException

[cloudstack] 01/01: Merge branch '4.14'

2020-11-13 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit 79e46f50cabf71c056cbbede5a37f847e1a8 Merge: a348afe db3c12d Author: Daan Hoogland AuthorDate: Fri Nov 13 13:25:07

[GitHub] [cloudstack] DaanHoogland commented on pull request #4428: Moved dedicated hosts to the end of the resultset when selecting an e…

2020-11-13 Thread GitBox
DaanHoogland commented on pull request #4428: URL: https://github.com/apache/cloudstack/pull/4428#issuecomment-726763425 @sureshanaparti can you have look again? @blueorangutan test This is an automated message from the Ap

[GitHub] [cloudstack] blueorangutan commented on pull request #4428: Moved dedicated hosts to the end of the resultset when selecting an e…

2020-11-13 Thread GitBox
blueorangutan commented on pull request #4428: URL: https://github.com/apache/cloudstack/pull/4428#issuecomment-726764230 @DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests -

[GitHub] [cloudstack] blueorangutan commented on pull request #4467: vpc: fix ips on wrong interfaces after rebooting vpc vrs

2020-11-13 Thread GitBox
blueorangutan commented on pull request #4467: URL: https://github.com/apache/cloudstack/pull/4467#issuecomment-726773724 Packaging result: ✔centos7 ✖centos8 ✔debian. JID-2382 This is an automated message from the Apache Git

[GitHub] [cloudstack] DaanHoogland commented on pull request #4467: vpc: fix ips on wrong interfaces after rebooting vpc vrs

2020-11-13 Thread GitBox
DaanHoogland commented on pull request #4467: URL: https://github.com/apache/cloudstack/pull/4467#issuecomment-726774175 @blueorangutan test This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [cloudstack] blueorangutan commented on pull request #4467: vpc: fix ips on wrong interfaces after rebooting vpc vrs

2020-11-13 Thread GitBox
blueorangutan commented on pull request #4467: URL: https://github.com/apache/cloudstack/pull/4467#issuecomment-726774710 @DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests -

[GitHub] [cloudstack] DaanHoogland commented on pull request #4150: [HEALTH][4.14] Health Check Run

2020-11-13 Thread GitBox
DaanHoogland commented on pull request #4150: URL: https://github.com/apache/cloudstack/pull/4150#issuecomment-726790027 @blueorangutan package This is an automated message from the Apache Git Service. To respond to the messa

[GitHub] [cloudstack] blueorangutan commented on pull request #4150: [HEALTH][4.14] Health Check Run

2020-11-13 Thread GitBox
blueorangutan commented on pull request #4150: URL: https://github.com/apache/cloudstack/pull/4150#issuecomment-726790543 @DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. ---

[GitHub] [cloudstack] blueorangutan commented on pull request #4150: [HEALTH][4.14] Health Check Run

2020-11-13 Thread GitBox
blueorangutan commented on pull request #4150: URL: https://github.com/apache/cloudstack/pull/4150#issuecomment-726805834 Packaging result: ✔centos7 ✖centos8 ✔debian. JID-2383 This is an automated message from the Apache Git

[GitHub] [cloudstack] DaanHoogland commented on pull request #4150: [HEALTH][4.14] Health Check Run

2020-11-13 Thread GitBox
DaanHoogland commented on pull request #4150: URL: https://github.com/apache/cloudstack/pull/4150#issuecomment-726834262 @blueoranutan test This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [cloudstack] blueorangutan commented on pull request #4131: [HEALTH] Master/4.15 Health Check please don't merge this

2020-11-13 Thread GitBox
blueorangutan commented on pull request #4131: URL: https://github.com/apache/cloudstack/pull/4131#issuecomment-726853134 Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2384 This is an automated message from the Apache Git

[cloudstack] branch master updated: Update PULL_REQUEST_TEMPLATE.md

2020-11-13 Thread paul_a
This is an automated email from the ASF dual-hosted git repository. paul_a pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/master by this push: new deff74c Update PULL_REQUEST_TEMPLATE.md def

[GitHub] [cloudstack] lmhetz commented on issue #4347: CS simulator docker hub image issues

2020-11-13 Thread GitBox
lmhetz commented on issue #4347: URL: https://github.com/apache/cloudstack/issues/4347#issuecomment-726888517 I have a question for ravening, I have been trying to get the advanced zone docker working, and it tells me the repository does not exist. What is the appropriate mailing list?

[GitHub] [cloudstack] PaulAngus commented on pull request #4200: Allow domain admins to create offering without mentioning domainid

2020-11-13 Thread GitBox
PaulAngus commented on pull request #4200: URL: https://github.com/apache/cloudstack/pull/4200#issuecomment-726890927 @ravening I am absolutely -1 on allowing admins to screw-up by not requiring the domain id, there is no need. Requiring it does not stop admins from doing anything

[GitHub] [cloudstack] ravening commented on pull request #4200: Allow domain admins to create offering without mentioning domainid

2020-11-13 Thread GitBox
ravening commented on pull request #4200: URL: https://github.com/apache/cloudstack/pull/4200#issuecomment-726894671 > @ravening I am absolutely -1 on allowing admins to screw-up by not requiring the domain id, there is no need. Requiring it does not stop admins from doing anything >

[GitHub] [cloudstack] GabrielBrascher commented on pull request #4200: Allow domain admins to create offering without mentioning domainid

2020-11-13 Thread GitBox
GabrielBrascher commented on pull request #4200: URL: https://github.com/apache/cloudstack/pull/4200#issuecomment-726933313 @PaulAngus can you please elaborate more on the risks involved on domain admin deploying VMs on its own domain by default? --

[GitHub] [cloudstack] blueorangutan commented on pull request #4469: Fix: Listing projects comprising of only the user's on listAll=true

2020-11-13 Thread GitBox
blueorangutan commented on pull request #4469: URL: https://github.com/apache/cloudstack/pull/4469#issuecomment-726999641 Trillian test result (tid-3167) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 32636 seconds Marvin logs: https://githu

[GitHub] [cloudstack] blueorangutan commented on pull request #4428: Moved dedicated hosts to the end of the resultset when selecting an e…

2020-11-13 Thread GitBox
blueorangutan commented on pull request #4428: URL: https://github.com/apache/cloudstack/pull/4428#issuecomment-727074166 Trillian test result (tid-3168) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 31848 seconds Marvin logs: https://githu

[GitHub] [cloudstack] blueorangutan commented on pull request #4467: vpc: fix ips on wrong interfaces after rebooting vpc vrs

2020-11-13 Thread GitBox
blueorangutan commented on pull request #4467: URL: https://github.com/apache/cloudstack/pull/4467#issuecomment-727078318 Trillian test result (tid-3169) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 31470 seconds Marvin logs: https://githu