[GitHub] [cloudstack] blueorangutan commented on pull request #5410: CloudStack fails to migrate VM with volume when there are datadisks attatched

2021-10-04 Thread GitBox
blueorangutan commented on pull request #5410: URL: https://github.com/apache/cloudstack/pull/5410#issuecomment-934108724 @sureshanaparti 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.

[GitHub] [cloudstack] sureshanaparti commented on pull request #5410: CloudStack fails to migrate VM with volume when there are datadisks attatched

2021-10-04 Thread GitBox
sureshanaparti commented on pull request #5410: URL: https://github.com/apache/cloudstack/pull/5410#issuecomment-934108614 @blueorangutan package -- 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 to

[GitHub] [cloudstack] blueorangutan commented on pull request #5542: Report the PowerFlex/ScaleIO disk copy failure during volume migration and fail the migration

2021-10-04 Thread GitBox
blueorangutan commented on pull request #5542: URL: https://github.com/apache/cloudstack/pull/5542#issuecomment-934108216 @sureshanaparti 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.

[GitHub] [cloudstack] sureshanaparti commented on pull request #5542: Report the PowerFlex/ScaleIO disk copy failure during volume migration and fail the migration

2021-10-04 Thread GitBox
sureshanaparti commented on pull request #5542: URL: https://github.com/apache/cloudstack/pull/5542#issuecomment-934107680 @blueorangutan package -- 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 to

[GitHub] [cloudstack] blueorangutan commented on pull request #5410: CloudStack fails to migrate VM with volume when there are datadisks attatched

2021-10-04 Thread GitBox
blueorangutan commented on pull request #5410: URL: https://github.com/apache/cloudstack/pull/5410#issuecomment-934101508 Packaging result: :heavy_multiplication_x: el7 :heavy_check_mark: el8 :heavy_multiplication_x: debian :heavy_check_mark: suse15. SL-JID 1503 -- This is an automated m

[GitHub] [cloudstack] blueorangutan commented on pull request #5201: [HEALTH] 4.16 Health Checks - Please don't merge

2021-10-04 Thread GitBox
blueorangutan commented on pull request #5201: URL: https://github.com/apache/cloudstack/pull/5201#issuecomment-934101506 Packaging result: :heavy_multiplication_x: el7 :heavy_multiplication_x: el8 :heavy_multiplication_x: debian :heavy_multiplication_x: suse15. SL-JID 1502 -- This is an

[GitHub] [cloudstack] blueorangutan commented on pull request #5543: xcp-ng: fix vm boot options

2021-10-04 Thread GitBox
blueorangutan commented on pull request #5543: URL: https://github.com/apache/cloudstack/pull/5543#issuecomment-934100371 Packaging result: :heavy_multiplication_x: el7 :heavy_multiplication_x: el8 :heavy_multiplication_x: debian :heavy_multiplication_x: suse15. SL-JID 1501 -- This is an

[GitHub] [cloudstack] sureshanaparti commented on pull request #5542: Report the PowerFlex/ScaleIO disk copy failure during volume migration and fail the migration

2021-10-04 Thread GitBox
sureshanaparti commented on pull request #5542: URL: https://github.com/apache/cloudstack/pull/5542#issuecomment-934098648 > @sureshanaparti does this need any manual testing, or was it tested and is this good to merge? @rhtyd I've tested this. @borisstoyanov will double check and co

[GitHub] [cloudstack] rhtyd commented on pull request #5542: Report the PowerFlex/ScaleIO disk copy failure during volume migration and fail the migration

2021-10-04 Thread GitBox
rhtyd commented on pull request #5542: URL: https://github.com/apache/cloudstack/pull/5542#issuecomment-934093871 @sureshanaparti does this need any manual testing, or was it tested and is this good to merge? -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [cloudstack] rhtyd commented on pull request #5544: Fix of revert RBD snapshots

2021-10-04 Thread GitBox
rhtyd commented on pull request #5544: URL: https://github.com/apache/cloudstack/pull/5544#issuecomment-934090560 @sureshanaparti @nvazquez @DaanHoogland what's the concensus here, is this good to merge? -- This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] [cloudstack] blueorangutan commented on pull request #5549: UI - Add clear all notification button

2021-10-04 Thread GitBox
blueorangutan commented on pull request #5549: URL: https://github.com/apache/cloudstack/pull/5549#issuecomment-934090507 UI build: :heavy_check_mark: Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/5549 (SL-JID-732) -- This is an automated message from the Apache Git Service. T

[GitHub] [cloudstack] blueorangutan commented on pull request #5549: UI - Add clear all notification button

2021-10-04 Thread GitBox
blueorangutan commented on pull request #5549: URL: https://github.com/apache/cloudstack/pull/5549#issuecomment-934088118 @utchoang a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress. -- This is an automated message from the Apache Git Service. To r

[GitHub] [cloudstack] utchoang commented on pull request #5549: UI - Add clear all notification button

2021-10-04 Thread GitBox
utchoang commented on pull request #5549: URL: https://github.com/apache/cloudstack/pull/5549#issuecomment-934087656 @blueorangutan ui -- 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 to the specifi

[GitHub] [cloudstack] sureshanaparti removed a comment on pull request #5410: CloudStack fails to migrate VM with volume when there are datadisks attatched

2021-10-04 Thread GitBox
sureshanaparti removed a comment on pull request #5410: URL: https://github.com/apache/cloudstack/pull/5410#issuecomment-934082655 @blueorangutan package -- 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] blueorangutan commented on pull request #5410: CloudStack fails to migrate VM with volume when there are datadisks attatched

2021-10-04 Thread GitBox
blueorangutan commented on pull request #5410: URL: https://github.com/apache/cloudstack/pull/5410#issuecomment-934086129 @sureshanaparti 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.

[GitHub] [cloudstack] utchoang commented on pull request #5549: UI - Add clear all notification button

2021-10-04 Thread GitBox
utchoang commented on pull request #5549: URL: https://github.com/apache/cloudstack/pull/5549#issuecomment-934085817 @davidjumani Please refresh or clear cache, I have the same problem as you in QA server but after ctrl+f5 it works fine. ![image](https://user-images.githubusercontent.co

[GitHub] [cloudstack] sureshanaparti commented on pull request #5410: CloudStack fails to migrate VM with volume when there are datadisks attatched

2021-10-04 Thread GitBox
sureshanaparti commented on pull request #5410: URL: https://github.com/apache/cloudstack/pull/5410#issuecomment-934085400 @blueorangutan package -- 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 to

[GitHub] [cloudstack] sureshanaparti commented on pull request #4774: Added configuration and Integration test to restrict public template …

2021-10-04 Thread GitBox
sureshanaparti commented on pull request #4774: URL: https://github.com/apache/cloudstack/pull/4774#issuecomment-934083593 ping @soreana, can you address the outstanding comments. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitH

[GitHub] [cloudstack] blueorangutan commented on pull request #5201: [HEALTH] 4.16 Health Checks - Please don't merge

2021-10-04 Thread GitBox
blueorangutan commented on pull request #5201: URL: https://github.com/apache/cloudstack/pull/5201#issuecomment-934083338 @sureshanaparti 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.

[GitHub] [cloudstack] sureshanaparti commented on pull request #5201: [HEALTH] 4.16 Health Checks - Please don't merge

2021-10-04 Thread GitBox
sureshanaparti commented on pull request #5201: URL: https://github.com/apache/cloudstack/pull/5201#issuecomment-934082827 @blueorangutan package -- 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 to

[GitHub] [cloudstack] sureshanaparti commented on pull request #5410: CloudStack fails to migrate VM with volume when there are datadisks attatched

2021-10-04 Thread GitBox
sureshanaparti commented on pull request #5410: URL: https://github.com/apache/cloudstack/pull/5410#issuecomment-934082655 @blueorangutan package -- 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 to

[GitHub] [cloudstack] blueorangutan commented on pull request #5543: xcp-ng: fix vm boot options

2021-10-04 Thread GitBox
blueorangutan commented on pull request #5543: URL: https://github.com/apache/cloudstack/pull/5543#issuecomment-934082481 @sureshanaparti 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.

[GitHub] [cloudstack] sureshanaparti commented on pull request #5543: xcp-ng: fix vm boot options

2021-10-04 Thread GitBox
sureshanaparti commented on pull request #5543: URL: https://github.com/apache/cloudstack/pull/5543#issuecomment-934082332 @blueorangutan package -- 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 to

[GitHub] [cloudstack] Pearl1594 commented on pull request #4329: Adding AutoScaling for cks + CKS CoreOS EOL update + systemvmtemplate improvements

2021-10-04 Thread GitBox
Pearl1594 commented on pull request #4329: URL: https://github.com/apache/cloudstack/pull/4329#issuecomment-934073124 @rhtyd - @alexandremattioli has already provided their review - https://github.com/apache/cloudstack/pull/4329#issuecomment-933491250 -- This is an automated message from

[GitHub] [cloudstack] rhtyd commented on pull request #4329: Adding AutoScaling for cks + CKS CoreOS EOL update + systemvmtemplate improvements

2021-10-04 Thread GitBox
rhtyd commented on pull request #4329: URL: https://github.com/apache/cloudstack/pull/4329#issuecomment-934072111 Ping @alexandremattioli or @andrijapanicsb are you lgtm on this as well? -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[cloudstack-go] branch main updated: Update SDK documentation (#12)

2021-10-04 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/cloudstack-go.git The following commit(s) were added to refs/heads/main by this push: new b72a0ae Update SDK documentation (#12) b72a0a

[GitHub] [cloudstack] davidjumani commented on pull request #5506: kvm: Use lscpu to get cpu max speed

2021-10-04 Thread GitBox
davidjumani commented on pull request #5506: URL: https://github.com/apache/cloudstack/pull/5506#issuecomment-934062148 Agree with the order @weizhouapache proposed -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [cloudstack] davidjumani commented on pull request #5549: UI - Add clear all notification button

2021-10-04 Thread GitBox
davidjumani commented on pull request #5549: URL: https://github.com/apache/cloudstack/pull/5549#issuecomment-934061521 @utchoang The notifications cover the usermenu and the clear all icon ![Screenshot from 2021-10-05 10-25-38](https://user-images.githubusercontent.com/8244774/135962531

[GitHub] [cloudstack] sureshanaparti commented on a change in pull request #5544: Fix of revert RBD snapshots

2021-10-04 Thread GitBox
sureshanaparti commented on a change in pull request #5544: URL: https://github.com/apache/cloudstack/pull/5544#discussion_r721883767 ## File path: server/src/main/java/com/cloud/storage/snapshot/SnapshotManagerImpl.java ## @@ -304,7 +304,10 @@ public Snapshot revertSnapshot(L

[GitHub] [cloudstack] blueorangutan commented on pull request #5201: [HEALTH] 4.16 Health Checks - Please don't merge

2021-10-04 Thread GitBox
blueorangutan commented on pull request #5201: URL: https://github.com/apache/cloudstack/pull/5201#issuecomment-934056562 Trillian test result (tid-2316) Environment: vmware-70u1 (x2), Advanced Networking with Mgmt server 7 Total time taken: 45548 seconds Marvin logs: https://githu

[GitHub] [cloudstack] blueorangutan removed a comment on pull request #5549: UI - Add clear all notification button

2021-10-04 Thread GitBox
blueorangutan removed a comment on pull request #5549: URL: https://github.com/apache/cloudstack/pull/5549#issuecomment-934005644 UI build: :heavy_multiplication_x: (SL-JID-729) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

[GitHub] [cloudstack] blueorangutan commented on pull request #5549: UI - Add clear all notification button

2021-10-04 Thread GitBox
blueorangutan commented on pull request #5549: URL: https://github.com/apache/cloudstack/pull/5549#issuecomment-934005644 -- 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 to the specific comment. To

[GitHub] [cloudstack] utchoang removed a comment on pull request #5549: UI - Add clear all notification button

2021-10-04 Thread GitBox
utchoang removed a comment on pull request #5549: URL: https://github.com/apache/cloudstack/pull/5549#issuecomment-934002581 @blueorangutan ui -- 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 to the

[GitHub] [cloudstack] blueorangutan removed a comment on pull request #5549: UI - Add clear all notification button

2021-10-04 Thread GitBox
blueorangutan removed a comment on pull request #5549: URL: https://github.com/apache/cloudstack/pull/5549#issuecomment-934003366 @utchoang a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress. -- This is an automated message from the Apache Git Servi

[GitHub] [cloudstack] blueorangutan commented on pull request #5549: UI - Add clear all notification button

2021-10-04 Thread GitBox
blueorangutan commented on pull request #5549: URL: https://github.com/apache/cloudstack/pull/5549#issuecomment-934003366 @utchoang a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress. -- This is an automated message from the Apache Git Service. To r

[GitHub] [cloudstack] utchoang commented on pull request #5549: UI - Add clear all notification button

2021-10-04 Thread GitBox
utchoang commented on pull request #5549: URL: https://github.com/apache/cloudstack/pull/5549#issuecomment-934002581 @blueorangutan ui -- 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 to the specifi

[GitHub] [cloudstack] utchoang opened a new pull request #5549: UI - Add clear all notification button

2021-10-04 Thread GitBox
utchoang opened a new pull request #5549: URL: https://github.com/apache/cloudstack/pull/5549 ### Description Fixes https://github.com/apache/cloudstack/issues/5538#issuecomment-933154768 ### Types of changes - [ ] Brea

[GitHub] [cloudstack] blueorangutan commented on pull request #5201: [HEALTH] 4.16 Health Checks - Please don't merge

2021-10-04 Thread GitBox
blueorangutan commented on pull request #5201: URL: https://github.com/apache/cloudstack/pull/5201#issuecomment-933988364 Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 1498 -- This is an automated message from

[GitHub] [cloudstack] blueorangutan commented on pull request #5201: [HEALTH] 4.16 Health Checks - Please don't merge

2021-10-04 Thread GitBox
blueorangutan commented on pull request #5201: URL: https://github.com/apache/cloudstack/pull/5201#issuecomment-933975076 @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 re

[GitHub] [cloudstack] nvazquez commented on pull request #5201: [HEALTH] 4.16 Health Checks - Please don't merge

2021-10-04 Thread GitBox
nvazquez commented on pull request #5201: URL: https://github.com/apache/cloudstack/pull/5201#issuecomment-933974762 @blueorangutan package -- 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 to the s

[cloudstack] branch main updated: Fix resize volume and migrate volume to update volume path if DRS is applied on volume in datastore cluster (#5539)

2021-10-04 Thread nvazquez
This is an automated email from the ASF dual-hosted git repository. nvazquez pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/main by this push: new cd4e7e0 Fix resize volume and migrate volume

[GitHub] [cloudstack] nvazquez closed issue #5484: Resize volume fails after DRS migrates volume in datastore cluster

2021-10-04 Thread GitBox
nvazquez closed issue #5484: URL: https://github.com/apache/cloudstack/issues/5484 -- 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 to the specific comment. To unsubscribe, e-mail: commits-unsubsc

[GitHub] [cloudstack] nvazquez merged pull request #5539: Fix resize volume and migrate volume to update volume path if DRS is applied on volume in datastore cluster

2021-10-04 Thread GitBox
nvazquez merged pull request #5539: URL: https://github.com/apache/cloudstack/pull/5539 -- 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 to the specific comment. To unsubscribe, e-mail: commits-un

[GitHub] [cloudstack] Hudratronium commented on issue #4238: uefi vm creation fail

2021-10-04 Thread GitBox
Hudratronium commented on issue #4238: URL: https://github.com/apache/cloudstack/issues/4238#issuecomment-933944893 That would be really appreaciated. Currently the docs as well as the wiki are quiet ... empty regarding this whole topic. -- This is an automated message from the Apache

[GitHub] [cloudstack] nvazquez commented on pull request #5539: Fix resize volume and migrate volume to update volume path if DRS is applied on volume in datastore cluster

2021-10-04 Thread GitBox
nvazquez commented on pull request #5539: URL: https://github.com/apache/cloudstack/pull/5539#issuecomment-933944703 Smoke test failure was fixed by PR #5547, merging -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use th

[GitHub] [cloudstack] blueorangutan commented on pull request #5539: Fix resize volume and migrate volume to update volume path if DRS is applied on volume in datastore cluster

2021-10-04 Thread GitBox
blueorangutan commented on pull request #5539: URL: https://github.com/apache/cloudstack/pull/5539#issuecomment-933943009 Trillian test result (tid-2315) Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server 7 Total time taken: 36377 seconds Marvin logs: https://githu

[cloudstack] branch main updated: vmware, network: add maclearning option (#5471)

2021-10-04 Thread nvazquez
This is an automated email from the ASF dual-hosted git repository. nvazquez pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/main by this push: new 6e216dd vmware, network: add maclearning opti

[GitHub] [cloudstack] nvazquez merged pull request #5471: vmware, network: add maclearning option

2021-10-04 Thread GitBox
nvazquez merged pull request #5471: URL: https://github.com/apache/cloudstack/pull/5471 -- 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 to the specific comment. To unsubscribe, e-mail: commits-un

[GitHub] [cloudstack] nvazquez commented on pull request #5471: vmware, network: add maclearning option

2021-10-04 Thread GitBox
nvazquez commented on pull request #5471: URL: https://github.com/apache/cloudstack/pull/5471#issuecomment-933917563 The test failure has been fixed by PR #5547, merging -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

[GitHub] [cloudstack] blueorangutan commented on pull request #5471: vmware, network: add maclearning option

2021-10-04 Thread GitBox
blueorangutan commented on pull request #5471: URL: https://github.com/apache/cloudstack/pull/5471#issuecomment-933894963 Trillian test result (tid-2314) Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server 7 Total time taken: 36552 seconds Marvin logs: https://githu

[GitHub] [cloudstack] blueorangutan commented on pull request #5201: [HEALTH] 4.16 Health Checks - Please don't merge

2021-10-04 Thread GitBox
blueorangutan commented on pull request #5201: URL: https://github.com/apache/cloudstack/pull/5201#issuecomment-933801977 Packaging result: :heavy_check_mark: el7 :heavy_multiplication_x: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 1497 -- This is an automated message

[GitHub] [cloudstack] blueorangutan commented on pull request #5540: kvm available memory calculation optimization

2021-10-04 Thread GitBox
blueorangutan commented on pull request #5540: URL: https://github.com/apache/cloudstack/pull/5540#issuecomment-933784323 Trillian test result (tid-2312) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 36962 seconds Marvin logs: https://githu

[GitHub] [cloudstack] blueorangutan commented on pull request #5201: [HEALTH] 4.16 Health Checks - Please don't merge

2021-10-04 Thread GitBox
blueorangutan commented on pull request #5201: URL: https://github.com/apache/cloudstack/pull/5201#issuecomment-933783574 @weizhouapache 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.

[cloudstack] branch main updated (3574d8d -> ad4def5)

2021-10-04 Thread weizhou
This is an automated email from the ASF dual-hosted git repository. weizhou pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/cloudstack.git. from 3574d8d parallel nic adding (#5541) add ad4def5 an inject annotation short (#5547) No new revisions were a

[GitHub] [cloudstack] weizhouapache commented on pull request #5201: [HEALTH] 4.16 Health Checks - Please don't merge

2021-10-04 Thread GitBox
weizhouapache commented on pull request #5201: URL: https://github.com/apache/cloudstack/pull/5201#issuecomment-933782895 @blueorangutan package -- 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 to t

[GitHub] [cloudstack] weizhouapache merged pull request #5547: an inject annotation short

2021-10-04 Thread GitBox
weizhouapache merged pull request #5547: URL: https://github.com/apache/cloudstack/pull/5547 -- 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 to the specific comment. To unsubscribe, e-mail: commi

[GitHub] [cloudstack] kohrar opened a new issue #5548: SAML2 authentication with MS Azure AD intermittently fails with error AADSTS7500529

2021-10-04 Thread GitBox
kohrar opened a new issue #5548: URL: https://github.com/apache/cloudstack/issues/5548 # ISSUE TYPE * Bug Report # COMPONENT NAME ~~~ SAML/SSO ~~~ # CLOUDSTACK VERSION ~~~ 4.15.2 ~~~ # CONFIGURATION Enabled

[GitHub] [cloudstack] blueorangutan commented on pull request #5547: an inject annotation short

2021-10-04 Thread GitBox
blueorangutan commented on pull request #5547: URL: https://github.com/apache/cloudstack/pull/5547#issuecomment-933771043 Trillian test result (tid-2311) Environment: vmware-65u2 (x2), Advanced Networking with Mgmt server 7 Total time taken: 36349 seconds Marvin logs: https://githu

[GitHub] [cloudstack] slavkap commented on a change in pull request #5544: Fix of revert RBD snapshots

2021-10-04 Thread GitBox
slavkap commented on a change in pull request #5544: URL: https://github.com/apache/cloudstack/pull/5544#discussion_r721600443 ## File path: server/src/main/java/com/cloud/storage/snapshot/SnapshotManagerImpl.java ## @@ -304,7 +304,10 @@ public Snapshot revertSnapshot(Long sna

[GitHub] [cloudstack] blueorangutan commented on pull request #5547: an inject annotation short

2021-10-04 Thread GitBox
blueorangutan commented on pull request #5547: URL: https://github.com/apache/cloudstack/pull/5547#issuecomment-933718924 Trillian test result (tid-2310) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 31955 seconds Marvin logs: https://githu

[GitHub] [cloudstack] blueorangutan commented on pull request #5547: an inject annotation short

2021-10-04 Thread GitBox
blueorangutan commented on pull request #5547: URL: https://github.com/apache/cloudstack/pull/5547#issuecomment-933718010 Trillian test result (tid-2309) Environment: xenserver-71 (x2), Advanced Networking with Mgmt server 7 Total time taken: 31999 seconds Marvin logs: https://gith

[GitHub] [cloudstack] GabrielBrascher edited a comment on pull request #5410: CloudStack fails to migrate VM with volume when there are datadisks attatched

2021-10-04 Thread GitBox
GabrielBrascher edited a comment on pull request #5410: URL: https://github.com/apache/cloudstack/pull/5410#issuecomment-933534842 @sureshanaparti sorry for the delay, I have been debugging and running some tests: I am also pushing a commit (tested recently on it as well) which I am

[GitHub] [cloudstack] GabrielBrascher commented on a change in pull request #5410: CloudStack fails to migrate VM with volume when there are datadisks attatched

2021-10-04 Thread GitBox
GabrielBrascher commented on a change in pull request #5410: URL: https://github.com/apache/cloudstack/pull/5410#discussion_r721509160 ## File path: engine/storage/datamotion/src/main/java/org/apache/cloudstack/storage/motion/KvmNonManagedStorageDataMotionStrategy.java ## @@ -

[GitHub] [cloudstack] GabrielBrascher commented on a change in pull request #5410: CloudStack fails to migrate VM with volume when there are datadisks attatched

2021-10-04 Thread GitBox
GabrielBrascher commented on a change in pull request #5410: URL: https://github.com/apache/cloudstack/pull/5410#discussion_r721509160 ## File path: engine/storage/datamotion/src/main/java/org/apache/cloudstack/storage/motion/KvmNonManagedStorageDataMotionStrategy.java ## @@ -

[GitHub] [cloudstack] DaanHoogland commented on issue #5499: Failed to add NIC to VM concurrently

2021-10-04 Thread GitBox
DaanHoogland commented on issue #5499: URL: https://github.com/apache/cloudstack/issues/5499#issuecomment-933622504 @div8cn , this ticket autoclosed as the PR was merged that claimed to solve this issue (and did so for me and Wei. Feel free to reopen or create a new one though, if this doe

[GitHub] [cloudstack] DaanHoogland commented on a change in pull request #5542: Report the PowerFlex/ScaleIO disk copy failure during volume migration and fail the migration

2021-10-04 Thread GitBox
DaanHoogland commented on a change in pull request #5542: URL: https://github.com/apache/cloudstack/pull/5542#discussion_r721491107 ## File path: plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/storage/KVMStorageProcessor.java ## @@ -1988,14 +1988,20 @@ public A

[GitHub] [cloudstack] blueorangutan commented on pull request #5201: [HEALTH] 4.16 Health Checks - Please don't merge

2021-10-04 Thread GitBox
blueorangutan commented on pull request #5201: URL: https://github.com/apache/cloudstack/pull/5201#issuecomment-933609734 @DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + vmware-70u1) has been kicked to run smoke tests -- This is an automated message from the Apache Git Service.

[GitHub] [cloudstack] DaanHoogland commented on pull request #5201: [HEALTH] 4.16 Health Checks - Please don't merge

2021-10-04 Thread GitBox
DaanHoogland commented on pull request #5201: URL: https://github.com/apache/cloudstack/pull/5201#issuecomment-933609064 @blueorangutan test centos7 vmware-70u1 keepEnv -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[cloudstack] branch main updated: parallel nic adding (#5541)

2021-10-04 Thread nvazquez
This is an automated email from the ASF dual-hosted git repository. nvazquez pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/main by this push: new 3574d8d parallel nic adding (#5541) 3574d8d i

[GitHub] [cloudstack] nvazquez merged pull request #5541: parallel nic adding

2021-10-04 Thread GitBox
nvazquez merged pull request #5541: URL: https://github.com/apache/cloudstack/pull/5541 -- 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 to the specific comment. To unsubscribe, e-mail: commits-un

[GitHub] [cloudstack] nvazquez closed issue #5499: Failed to add NIC to VM concurrently

2021-10-04 Thread GitBox
nvazquez closed issue #5499: URL: https://github.com/apache/cloudstack/issues/5499 -- 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 to the specific comment. To unsubscribe, e-mail: commits-unsubsc

[cloudstack] branch main updated (9f5ac89 -> 93c0b60)

2021-10-04 Thread nvazquez
This is an automated email from the ASF dual-hosted git repository. nvazquez pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/cloudstack.git. from 9f5ac89 VR: fix data-server if shared network has multiple ip ranges (#5530) add 93c0b60 [UI] Edit backup

[GitHub] [cloudstack] nvazquez merged pull request #5546: [UI] Edit backup offering

2021-10-04 Thread GitBox
nvazquez merged pull request #5546: URL: https://github.com/apache/cloudstack/pull/5546 -- 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 to the specific comment. To unsubscribe, e-mail: commits-un

[GitHub] [cloudstack] GabrielBrascher commented on pull request #5410: CloudStack fails to migrate VM with volume when there are datadisks attatched

2021-10-04 Thread GitBox
GabrielBrascher commented on pull request #5410: URL: https://github.com/apache/cloudstack/pull/5410#issuecomment-933534842 @sureshanaparti sorry for the delay, I have been debugging and running some tests: I am also pushing a commit (tested recently on it as well) which I am addres

[GitHub] [cloudstack] blueorangutan commented on pull request #5546: [UI] Edit backup offering

2021-10-04 Thread GitBox
blueorangutan commented on pull request #5546: URL: https://github.com/apache/cloudstack/pull/5546#issuecomment-933516473 UI build: :heavy_check_mark: Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/5546 (SL-JID-727) -- This is an automated message from the Apache Git Service. T

[GitHub] [cloudstack] blueorangutan commented on pull request #5546: [UI] Edit backup offering

2021-10-04 Thread GitBox
blueorangutan commented on pull request #5546: URL: https://github.com/apache/cloudstack/pull/5546#issuecomment-933512401 @nvazquez a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress. -- This is an automated message from the Apache Git Service. To r

[GitHub] [cloudstack] nvazquez commented on pull request #5546: [UI] Edit backup offering

2021-10-04 Thread GitBox
nvazquez commented on pull request #5546: URL: https://github.com/apache/cloudstack/pull/5546#issuecomment-933511501 @blueorangutan ui -- 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 to the specifi

[GitHub] [cloudstack] nvazquez commented on issue #5535: After installing 4.15.2.0 instances created from template with uefi cannot boot for second time (xcp-ng 8.2)

2021-10-04 Thread GitBox
nvazquez commented on issue #5535: URL: https://github.com/apache/cloudstack/issues/5535#issuecomment-933507778 Thanks @rhtyd @borisstoyanov @vladimirpetrov please check Rohit's comment above - there is a PR which might solve this issue: https://github.com/apache/cloudstack/pull/554

[GitHub] [cloudstack] PaulAngus commented on pull request #4978: KVM High Availability regardless of storage

2021-10-04 Thread GitBox
PaulAngus commented on pull request #4978: URL: https://github.com/apache/cloudstack/pull/4978#issuecomment-933496393 Correct me if I'm wrong, but this looks like its relying on HTTP over 8080? This will absolutely have to be HTTPS with strong auth at both ends. -- This is an a

[GitHub] [cloudstack] alexandremattioli commented on pull request #4329: Adding AutoScaling for cks + CKS CoreOS EOL update + systemvmtemplate improvements

2021-10-04 Thread GitBox
alexandremattioli commented on pull request #4329: URL: https://github.com/apache/cloudstack/pull/4329#issuecomment-933491250 LGTM -- 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 to the specific co

[GitHub] [cloudstack] blueorangutan commented on pull request #5539: Fix resize volume and migrate volume to update volume path if DRS is applied on volume in datastore cluster

2021-10-04 Thread GitBox
blueorangutan commented on pull request #5539: URL: https://github.com/apache/cloudstack/pull/5539#issuecomment-933488628 @nvazquez a Trillian-Jenkins test job (centos7 mgmt + vmware-67u3) has been kicked to run smoke tests -- This is an automated message from the Apache Git Service. To

[GitHub] [cloudstack] nvazquez commented on pull request #5539: Fix resize volume and migrate volume to update volume path if DRS is applied on volume in datastore cluster

2021-10-04 Thread GitBox
nvazquez commented on pull request #5539: URL: https://github.com/apache/cloudstack/pull/5539#issuecomment-933487496 @blueorangutan test centos7 vmware-67u3 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abov

[GitHub] [cloudstack] GabrielBrascher commented on pull request #4978: KVM High Availability regardless of storage

2021-10-04 Thread GitBox
GabrielBrascher commented on pull request #4978: URL: https://github.com/apache/cloudstack/pull/4978#issuecomment-933484526 @rhtyd thanks for the review. I hope that I can address all your comments here: > we should explore options to implement this without introducing a new service

[GitHub] [cloudstack] blueorangutan commented on pull request #5539: Fix resize volume and migrate volume to update volume path if DRS is applied on volume in datastore cluster

2021-10-04 Thread GitBox
blueorangutan commented on pull request #5539: URL: https://github.com/apache/cloudstack/pull/5539#issuecomment-933450511 Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 1496 -- This is an automated message from

[GitHub] [cloudstack] DaanHoogland commented on a change in pull request #5539: Fix resize volume and migrate volume to update volume path if DRS is applied on volume in datastore cluster

2021-10-04 Thread GitBox
DaanHoogland commented on a change in pull request #5539: URL: https://github.com/apache/cloudstack/pull/5539#discussion_r721327247 ## File path: plugins/hypervisors/vmware/src/main/java/com/cloud/hypervisor/vmware/resource/VmwareResource.java ## @@ -922,7 +937,17 @@ private A

[GitHub] [cloudstack] slavkap commented on issue #4238: uefi vm creation fail

2021-10-04 Thread GitBox
slavkap commented on issue #4238: URL: https://github.com/apache/cloudstack/issues/4238#issuecomment-933433161 @Hudratronium, I don't remember. I will test and let you know -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [cloudstack] blueorangutan commented on pull request #5539: Fix resize volume and migrate volume to update volume path if DRS is applied on volume in datastore cluster

2021-10-04 Thread GitBox
blueorangutan commented on pull request #5539: URL: https://github.com/apache/cloudstack/pull/5539#issuecomment-933432371 @harikrishna-patnala 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 Ser

[GitHub] [cloudstack] harikrishna-patnala commented on pull request #5539: Fix resize volume and migrate volume to update volume path if DRS is applied on volume in datastore cluster

2021-10-04 Thread GitBox
harikrishna-patnala commented on pull request #5539: URL: https://github.com/apache/cloudstack/pull/5539#issuecomment-933431817 @blueorangutan package -- 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 g

[GitHub] [cloudstack] harikrishna-patnala commented on a change in pull request #5539: Fix resize volume and migrate volume to update volume path if DRS is applied on volume in datastore cluster

2021-10-04 Thread GitBox
harikrishna-patnala commented on a change in pull request #5539: URL: https://github.com/apache/cloudstack/pull/5539#discussion_r721311031 ## File path: plugins/hypervisors/vmware/src/main/java/com/cloud/hypervisor/vmware/resource/VmwareResource.java ## @@ -922,7 +937,17 @@ pr

[GitHub] [cloudstack] harikrishna-patnala commented on pull request #5539: Fix resize volume and migrate volume to update volume path if DRS is applied on volume in datastore cluster

2021-10-04 Thread GitBox
harikrishna-patnala commented on pull request #5539: URL: https://github.com/apache/cloudstack/pull/5539#issuecomment-933427030 Addressed all review comments and tested the issue again. -- This is an automated message from the Apache Git Service. To respond to the message, please log on t

[cloudstack] branch FixResizeVolumeMigrateVolumeInCaseOfDatastoreCluster updated (e51a689 -> 5c26f0a)

2021-10-04 Thread harikrishna
This is an automated email from the ASF dual-hosted git repository. harikrishna pushed a change to branch FixResizeVolumeMigrateVolumeInCaseOfDatastoreCluster in repository https://gitbox.apache.org/repos/asf/cloudstack.git. from e51a689 Refactor code for more readability add 5c26f0a

[GitHub] [cloudstack] rhtyd commented on issue #5535: After installing 4.15.2.0 instances created from template with uefi cannot boot for second time (xcp-ng 8.2)

2021-10-04 Thread GitBox
rhtyd commented on issue #5535: URL: https://github.com/apache/cloudstack/issues/5535#issuecomment-933422857 Thanks @AlexanderKgr @nvazquez @sureshanaparti @weizhouapache I think our test for the PR should involve creating an existing VM on XenServer/XCP-ng (say 4.15.0 or 4.15.1) and t

[GitHub] [cloudstack] harikrishna-patnala commented on a change in pull request #5539: Fix resize volume and migrate volume to update volume path if DRS is applied on volume in datastore cluster

2021-10-04 Thread GitBox
harikrishna-patnala commented on a change in pull request #5539: URL: https://github.com/apache/cloudstack/pull/5539#discussion_r721301734 ## File path: plugins/hypervisors/vmware/src/main/java/com/cloud/hypervisor/vmware/resource/VmwareResource.java ## @@ -892,6 +860,53 @@ pr

[GitHub] [cloudstack] harikrishna-patnala commented on a change in pull request #5539: Fix resize volume and migrate volume to update volume path if DRS is applied on volume in datastore cluster

2021-10-04 Thread GitBox
harikrishna-patnala commented on a change in pull request #5539: URL: https://github.com/apache/cloudstack/pull/5539#discussion_r721301467 ## File path: plugins/hypervisors/vmware/src/main/java/com/cloud/hypervisor/vmware/resource/VmwareResource.java ## @@ -892,6 +860,53 @@ pr

[cloudstack] branch FixResizeVolumeMigrateVolumeInCaseOfDatastoreCluster updated (7da5a8c -> e51a689)

2021-10-04 Thread harikrishna
This is an automated email from the ASF dual-hosted git repository. harikrishna pushed a change to branch FixResizeVolumeMigrateVolumeInCaseOfDatastoreCluster in repository https://gitbox.apache.org/repos/asf/cloudstack.git. from 7da5a8c Remove commented code add e51a689 Refactor cod

[GitHub] [cloudstack] AlexanderKgr edited a comment on issue #5535: After installing 4.15.2.0 instances created from template with uefi cannot boot for second time (xcp-ng 8.2)

2021-10-04 Thread GitBox
AlexanderKgr edited a comment on issue #5535: URL: https://github.com/apache/cloudstack/issues/5535#issuecomment-933418313 > @AlexanderKgr does it work for newly deployed VMs with the fix, i.e. after you upgrade to the packages and deploy a VM, stop it, are able to start it second time?

[GitHub] [cloudstack] AlexanderKgr commented on issue #5535: After installing 4.15.2.0 instances created from template with uefi cannot boot for second time (xcp-ng 8.2)

2021-10-04 Thread GitBox
AlexanderKgr commented on issue #5535: URL: https://github.com/apache/cloudstack/issues/5535#issuecomment-933418313 > @AlexanderKgr does it work for newly deployed VMs with the fix, i.e. after you upgrade to the packages and deploy a VM, stop it, are able to start it second time? It

[GitHub] [cloudstack] weizhouapache commented on pull request #4774: Added configuration and Integration test to restrict public template …

2021-10-04 Thread GitBox
weizhouapache commented on pull request #4774: URL: https://github.com/apache/cloudstack/pull/4774#issuecomment-933410522 > @nvazquez the config scope is Domain https://github.com/apache/cloudstack/pull/4774/files#diff-a7bfc6f26c3369d9ee68fec67d2c4fcdd228dbc3d67fb56ee3b1590c4a24c4f4R117

[GitHub] [cloudstack] SadiJr commented on a change in pull request #5546: [UI] Edit backup offering

2021-10-04 Thread GitBox
SadiJr commented on a change in pull request #5546: URL: https://github.com/apache/cloudstack/pull/5546#discussion_r721288385 ## File path: ui/src/config/section/offering.js ## @@ -215,6 +215,15 @@ export default { listView: true, popup: true, compone

[GitHub] [cloudstack] blueorangutan commented on pull request #5471: vmware, network: add maclearning option

2021-10-04 Thread GitBox
blueorangutan commented on pull request #5471: URL: https://github.com/apache/cloudstack/pull/5471#issuecomment-933402034 @sureshanaparti a Trillian-Jenkins test job (centos7 mgmt + vmware-67u3) has been kicked to run smoke tests -- This is an automated message from the Apache Git Servic

  1   2   >