[GitHub] [cloudstack] DaanHoogland closed pull request #7622: Fix scaleio tests

2023-06-12 Thread via GitHub
DaanHoogland closed pull request #7622: Fix scaleio tests URL: https://github.com/apache/cloudstack/pull/7622 -- 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-

[GitHub] [cloudstack] blueorangutan commented on pull request #7489: Flexible tags for hosts and storage pools

2023-06-12 Thread via GitHub
blueorangutan commented on PR #7489: URL: https://github.com/apache/cloudstack/pull/7489#issuecomment-1588646308 Packaging result [LL]: :heavy_multiplication_x: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 6136 -- This is a

[GitHub] [cloudstack] DaanHoogland commented on pull request #7621: pom.xml: correct url of libvirt maven2

2023-06-12 Thread via GitHub
DaanHoogland commented on PR #7621: URL: https://github.com/apache/cloudstack/pull/7621#issuecomment-1588646138 @weizhouapache the old one is still not reachable and the new one is, do we merge this? -- This is an automated message from the Apache Git Service. To respond to the message, p

[GitHub] [cloudstack] codecov[bot] commented on pull request #7622: Fix scaleio tests

2023-06-12 Thread via GitHub
codecov[bot] commented on PR #7622: URL: https://github.com/apache/cloudstack/pull/7622#issuecomment-1588639004 ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/7622?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)

[GitHub] [cloudstack] blueorangutan commented on pull request #7489: Flexible tags for hosts and storage pools

2023-06-12 Thread via GitHub
blueorangutan commented on PR #7489: URL: https://github.com/apache/cloudstack/pull/7489#issuecomment-1588590547 @DaanHoogland a [LL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress

[GitHub] [cloudstack] rajujith opened a new issue, #7623: Deleted physical network leaves stale entries

2023-06-12 Thread via GitHub
rajujith opened a new issue, #7623: URL: https://github.com/apache/cloudstack/issues/7623 # ISSUE TYPE * Bug Report # COMPONENT NAME ~~~ VR ~~~ # CLOUDSTACK VERSION ~~~ 4.18.0.0 4.17.2 ~~~ # CONFIGURAT

[GitHub] [cloudstack] blueorangutan commented on pull request #7408: Allow volume migrations in ScaleIO within and across ScaleIO storage clusters

2023-06-12 Thread via GitHub
blueorangutan commented on PR #7408: URL: https://github.com/apache/cloudstack/pull/7408#issuecomment-1588589180 Packaging result [LL]: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 6135 -- This is an auto

[GitHub] [cloudstack] DaanHoogland commented on pull request #7489: Flexible tags for hosts and storage pools

2023-06-12 Thread via GitHub
DaanHoogland commented on PR #7489: URL: https://github.com/apache/cloudstack/pull/7489#issuecomment-1588589096 @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 specific

[GitHub] [cloudstack] vishesh92 commented on pull request #7622: Fix scaleio tests

2023-06-12 Thread via GitHub
vishesh92 commented on PR #7622: URL: https://github.com/apache/cloudstack/pull/7622#issuecomment-1588576559 ![image](https://github.com/apache/cloudstack/assets/8760112/a22fe506-62f9-4fbf-8112-cfe575994366) -- This is an automated message from the Apache Git Service. To respond to th

[GitHub] [cloudstack] vishesh92 commented on a diff in pull request #7622: Fix scaleio tests

2023-06-12 Thread via GitHub
vishesh92 commented on code in PR #7622: URL: https://github.com/apache/cloudstack/pull/7622#discussion_r1227546880 ## plugins/storage/volume/scaleio/src/test/java/org/apache/cloudstack/storage/datastore/client/ScaleIOGatewayClientImplTest.java: ## @@ -48,7 +48,7 @@ @RunWith(

[GitHub] [cloudstack] vishesh92 commented on pull request #7612: [WIP] coverage: fix scaleio plugin code coverage

2023-06-12 Thread via GitHub
vishesh92 commented on PR #7612: URL: https://github.com/apache/cloudstack/pull/7612#issuecomment-1588569725 @harikrishna-patnala @DaanHoogland @weizhouapache I have fixed the tests and created a new PR here: https://github.com/apache/cloudstack/pull/7622 -- This is an automated message f

[GitHub] [cloudstack] vishesh92 opened a new pull request, #7622: Fix scaleio tests

2023-06-12 Thread via GitHub
vishesh92 opened a new pull request, #7622: URL: https://github.com/apache/cloudstack/pull/7622 ### Description This PR... ### Types of changes - [ ] Breaking change (fix or feature that would cause existing functionali

[GitHub] [cloudstack] blueorangutan commented on pull request #7408: Allow volume migrations in ScaleIO within and across ScaleIO storage clusters

2023-06-12 Thread via GitHub
blueorangutan commented on PR #7408: URL: https://github.com/apache/cloudstack/pull/7408#issuecomment-1588562279 @harikrishna-patnala a [LL] Jenkins job has been kicked to build packages. It will be bundled with SystemVM template(s). I'll keep you posted as I make progress. -- Thi

[GitHub] [cloudstack] harikrishna-patnala commented on pull request #7408: Allow volume migrations in ScaleIO within and across ScaleIO storage clusters

2023-06-12 Thread via GitHub
harikrishna-patnala commented on PR #7408: URL: https://github.com/apache/cloudstack/pull/7408#issuecomment-1588561423 @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

[GitHub] [cloudstack] ilhamalrahm commented on pull request #7532: Updated the code to access the primary color from config.json.

2023-06-12 Thread via GitHub
ilhamalrahm commented on PR #7532: URL: https://github.com/apache/cloudstack/pull/7532#issuecomment-1588539970 Anything i can do? -- 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 comm

[GitHub] [cloudstack] harikrishna-patnala commented on pull request #7612: [WIP] coverage: fix scaleio plugin code coverage

2023-06-12 Thread via GitHub
harikrishna-patnala commented on PR #7612: URL: https://github.com/apache/cloudstack/pull/7612#issuecomment-1588513884 > the unit tests failures have been fixed by @harikrishna-patnala in PR #7408 Not all @weizhouapache there are a couple of other failures in ScaleIOGatewayClientImplT

[GitHub] [cloudstack] blueorangutan commented on pull request #7237: console proxy: use AeadBase64Encryptor instead of AES/CBC/PKCS5Padding

2023-06-12 Thread via GitHub
blueorangutan commented on PR #7237: URL: https://github.com/apache/cloudstack/pull/7237#issuecomment-1588494501 [SF] Trillian test result (tid-6730) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 41298 seconds Marvin logs: https://github.com

[GitHub] [cloudstack] blueorangutan commented on pull request #7344: [HEALTH] 4.19/main Health Check, please don't merge this!

2023-06-12 Thread via GitHub
blueorangutan commented on PR #7344: URL: https://github.com/apache/cloudstack/pull/7344#issuecomment-1588484754 [LL]Trillian test result (tid-6602) Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server r8 Total time taken: 69762 seconds Marvin logs: https://github.com

[GitHub] [cloudstack] blueorangutan commented on pull request #7344: [HEALTH] 4.19/main Health Check, please don't merge this!

2023-06-12 Thread via GitHub
blueorangutan commented on PR #7344: URL: https://github.com/apache/cloudstack/pull/7344#issuecomment-1588384047 [SF] Trillian test result (tid-6728) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 41332 seconds Marvin logs: https://github.com

[GitHub] [cloudstack] blueorangutan commented on pull request #7345: [HEALTH] 4.18 LTS Health Check, please don't merge this!

2023-06-12 Thread via GitHub
blueorangutan commented on PR #7345: URL: https://github.com/apache/cloudstack/pull/7345#issuecomment-1588381721 [SF] Trillian test result (tid-6729) Environment: vmware-70u3 (x2), Advanced Networking with Mgmt server r8 Total time taken: 32599 seconds Marvin logs: https://github.co

[GitHub] [cloudstack] blueorangutan commented on pull request #7345: [HEALTH] 4.18 LTS Health Check, please don't merge this!

2023-06-12 Thread via GitHub
blueorangutan commented on PR #7345: URL: https://github.com/apache/cloudstack/pull/7345#issuecomment-1588362616 [SF] Trillian test result (tid-6727) Environment: vmware-70u3 (x2), Advanced Networking with Mgmt server r8 Total time taken: 42104 seconds Marvin logs: https://github.co

[GitHub] [cloudstack] blueorangutan commented on pull request #7345: [HEALTH] 4.18 LTS Health Check, please don't merge this!

2023-06-12 Thread via GitHub
blueorangutan commented on PR #7345: URL: https://github.com/apache/cloudstack/pull/7345#issuecomment-1588221609 [LL]Trillian test result (tid-6601) Environment: xcpng82 (x2), Advanced Networking with Mgmt server 7 Total time taken: 52449 seconds Marvin logs: https://github.com/blue

[GitHub] [cloudstack] blueorangutan commented on pull request #7381: Fix foreign key constraints and the mysql procedure that is used

2023-06-12 Thread via GitHub
blueorangutan commented on PR #7381: URL: https://github.com/apache/cloudstack/pull/7381#issuecomment-1588217530 Packaging result [LL]: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 6134 -- This is an auto

[GitHub] [cloudstack] blueorangutan commented on pull request #7131: Normalize loggers and upgrade log4j 1.2 to log4j 2.19

2023-06-12 Thread via GitHub
blueorangutan commented on PR #7131: URL: https://github.com/apache/cloudstack/pull/7131#issuecomment-1588211887 Packaging result [LL]: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_multiplication_x: debian :heavy_check_mark: suse15. SL-JID 6133 -- This is a

[GitHub] [cloudstack] blueorangutan commented on pull request #7345: [HEALTH] 4.18 LTS Health Check, please don't merge this!

2023-06-12 Thread via GitHub
blueorangutan commented on PR #7345: URL: https://github.com/apache/cloudstack/pull/7345#issuecomment-1588195794 [LL]Trillian test result (tid-6603) Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server r8 Total time taken: 50335 seconds Marvin logs: https://github.com

[GitHub] [cloudstack] blueorangutan commented on pull request #7381: Fix foreign key constraints and the mysql procedure that is used

2023-06-12 Thread via GitHub
blueorangutan commented on PR #7381: URL: https://github.com/apache/cloudstack/pull/7381#issuecomment-1588155027 @vladimirpetrov a [LL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progre

[GitHub] [cloudstack] vladimirpetrov commented on pull request #7381: Fix foreign key constraints and the mysql procedure that is used

2023-06-12 Thread via GitHub
vladimirpetrov commented on PR #7381: URL: https://github.com/apache/cloudstack/pull/7381#issuecomment-1588153355 @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 specif

[GitHub] [cloudstack] blueorangutan commented on pull request #7131: Normalize loggers and upgrade log4j 1.2 to log4j 2.19

2023-06-12 Thread via GitHub
blueorangutan commented on PR #7131: URL: https://github.com/apache/cloudstack/pull/7131#issuecomment-1588153272 @vladimirpetrov a [LL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progre

[GitHub] [cloudstack] vladimirpetrov commented on pull request #7131: Normalize loggers and upgrade log4j 1.2 to log4j 2.19

2023-06-12 Thread via GitHub
vladimirpetrov commented on PR #7131: URL: https://github.com/apache/cloudstack/pull/7131#issuecomment-1588152641 @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 specif

[GitHub] [cloudstack] blueorangutan commented on pull request #7408: Allow volume migrations in ScaleIO within and across ScaleIO storage clusters

2023-06-12 Thread via GitHub
blueorangutan commented on PR #7408: URL: https://github.com/apache/cloudstack/pull/7408#issuecomment-1588142782 [SF] Trillian test result (tid-6726) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 45288 seconds Marvin logs: https://github.com

[GitHub] [cloudstack] blueorangutan commented on pull request #7345: [HEALTH] 4.18 LTS Health Check, please don't merge this!

2023-06-12 Thread via GitHub
blueorangutan commented on PR #7345: URL: https://github.com/apache/cloudstack/pull/7345#issuecomment-1588131599 [LL]Trillian test result (tid-6600) Environment: xenserver-71 (x2), Advanced Networking with Mgmt server 7 Total time taken: 46853 seconds Marvin logs: https://github.com

[GitHub] [cloudstack] DaanHoogland commented on pull request #7237: console proxy: use AeadBase64Encryptor instead of AES/CBC/PKCS5Padding

2023-06-12 Thread via GitHub
DaanHoogland commented on PR #7237: URL: https://github.com/apache/cloudstack/pull/7237#issuecomment-1587647536 @blueorangutan test keepEnv -- 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 spe

[GitHub] [cloudstack] DaanHoogland commented on a diff in pull request #7408: Allow volume migrations in ScaleIO within and across ScaleIO storage clusters

2023-06-12 Thread via GitHub
DaanHoogland commented on code in PR #7408: URL: https://github.com/apache/cloudstack/pull/7408#discussion_r1226903045 ## plugins/storage/volume/scaleio/src/test/java/org/apache/cloudstack/storage/datastore/driver/ScaleIOPrimaryDataStoreDriverTest.java: ## @@ -0,0 +1,527 @@ +//

[GitHub] [cloudstack] blueorangutan commented on pull request #7620: [AGENT] host os accepted if only the first word is defined for the cluster

2023-06-12 Thread via GitHub
blueorangutan commented on PR #7620: URL: https://github.com/apache/cloudstack/pull/7620#issuecomment-1587631638 Packaging result [LL]: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 6130 -- This is an auto

[GitHub] [cloudstack] blueorangutan commented on pull request #6979: Guest os mappings improvements

2023-06-12 Thread via GitHub
blueorangutan commented on PR #6979: URL: https://github.com/apache/cloudstack/pull/6979#issuecomment-1587631653 Packaging result [LL]: :heavy_multiplication_x: el7 :heavy_multiplication_x: el8 :heavy_multiplication_x: el9 :heavy_multiplication_x: debian :heavy_multiplication_x: suse15. SL-

[GitHub] [cloudstack] blueorangutan commented on pull request #7597: consider last host only for root-admin

2023-06-12 Thread via GitHub
blueorangutan commented on PR #7597: URL: https://github.com/apache/cloudstack/pull/7597#issuecomment-1587631641 Packaging result [LL]: :heavy_multiplication_x: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 6131 -- This is a

[GitHub] [cloudstack] codecov[bot] commented on pull request #7620: [AGENT] host os accepted if only the first word is defined for the cluster

2023-06-12 Thread via GitHub
codecov[bot] commented on PR #7620: URL: https://github.com/apache/cloudstack/pull/7620#issuecomment-1587583842 ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/7620?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)

[GitHub] [cloudstack] blueorangutan commented on pull request #7237: console proxy: use AeadBase64Encryptor instead of AES/CBC/PKCS5Padding

2023-06-12 Thread via GitHub
blueorangutan commented on PR #7237: URL: https://github.com/apache/cloudstack/pull/7237#issuecomment-1587552700 Packaging result [LL]: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 6129 -- This is an auto

[GitHub] [cloudstack] rohityadavcloud commented on pull request #7408: Allow volume migrations in ScaleIO within and across ScaleIO storage clusters

2023-06-12 Thread via GitHub
rohityadavcloud commented on PR #7408: URL: https://github.com/apache/cloudstack/pull/7408#issuecomment-1587519304 @harikrishna-patnala @borisstoyanov should this bugfix/improvement be targeted for 4.18 since there are no DB changes and essentially you're addressing a limitation of the scal

[GitHub] [cloudstack] DaanHoogland closed pull request #7597: consider last host only for root-admin

2023-06-12 Thread via GitHub
DaanHoogland closed pull request #7597: consider last host only for root-admin URL: https://github.com/apache/cloudstack/pull/7597 -- 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

[GitHub] [cloudstack] blueorangutan commented on pull request #6979: Guest os mappings improvements

2023-06-12 Thread via GitHub
blueorangutan commented on PR #6979: URL: https://github.com/apache/cloudstack/pull/6979#issuecomment-1587474182 @DaanHoogland a [LL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress

[GitHub] [cloudstack] DaanHoogland commented on pull request #6979: Guest os mappings improvements

2023-06-12 Thread via GitHub
DaanHoogland commented on PR #6979: URL: https://github.com/apache/cloudstack/pull/6979#issuecomment-1587472020 @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 specific

[GitHub] [cloudstack] DaanHoogland closed pull request #7588: Zone wizzard fix

2023-06-12 Thread via GitHub
DaanHoogland closed pull request #7588: Zone wizzard fix URL: https://github.com/apache/cloudstack/pull/7588 -- 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-m

[GitHub] [cloudstack] DaanHoogland commented on pull request #7588: Zone wizzard fix

2023-06-12 Thread via GitHub
DaanHoogland commented on PR #7588: URL: https://github.com/apache/cloudstack/pull/7588#issuecomment-1587466686 @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 specific

[GitHub] [cloudstack] blueorangutan commented on pull request #7597: consider last host only for root-admin

2023-06-12 Thread via GitHub
blueorangutan commented on PR #7597: URL: https://github.com/apache/cloudstack/pull/7597#issuecomment-1587465401 @DaanHoogland a [LL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress

[GitHub] [cloudstack] DaanHoogland commented on pull request #7597: consider last host only for root-admin

2023-06-12 Thread via GitHub
DaanHoogland commented on PR #7597: URL: https://github.com/apache/cloudstack/pull/7597#issuecomment-1587462143 @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 specific

[GitHub] [cloudstack] codecov[bot] commented on pull request #7621: pom.xml: correct url of libvirt maven2

2023-06-12 Thread via GitHub
codecov[bot] commented on PR #7621: URL: https://github.com/apache/cloudstack/pull/7621#issuecomment-1587459566 ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/7621?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)

[GitHub] [cloudstack] blueorangutan commented on pull request #7620: [AGENT] host os accepted if only the first word is defined for the cluster

2023-06-12 Thread via GitHub
blueorangutan commented on PR #7620: URL: https://github.com/apache/cloudstack/pull/7620#issuecomment-1587451207 @DaanHoogland a [LL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress

[GitHub] [cloudstack] DaanHoogland commented on pull request #7620: [AGENT] host os accepted if only the first word is defined for the cluster

2023-06-12 Thread via GitHub
DaanHoogland commented on PR #7620: URL: https://github.com/apache/cloudstack/pull/7620#issuecomment-1587448093 @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 specific

[GitHub] [cloudstack] DaanHoogland closed pull request #7620: [AGENT] host os accepted if only the first word is defined for the cluster

2023-06-12 Thread via GitHub
DaanHoogland closed pull request #7620: [AGENT] host os accepted if only the first word is defined for the cluster URL: https://github.com/apache/cloudstack/pull/7620 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

[GitHub] [cloudstack] DaanHoogland commented on pull request #7620: [AGENT] host os accepted if only the first word is defined for the cluster

2023-06-12 Thread via GitHub
DaanHoogland commented on PR #7620: URL: https://github.com/apache/cloudstack/pull/7620#issuecomment-1587447000 > I will test it thanks > another option might be updating existing Host.OS in host_details (mostly rocky/rhel). is alma impacted? I was thinking of that a

[GitHub] [cloudstack] DaanHoogland closed pull request #7617: Retrieve only resource count on user dashboard

2023-06-12 Thread via GitHub
DaanHoogland closed pull request #7617: Retrieve only resource count on user dashboard URL: https://github.com/apache/cloudstack/pull/7617 -- 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] blueorangutan commented on pull request #7237: console proxy: use AeadBase64Encryptor instead of AES/CBC/PKCS5Padding

2023-06-12 Thread via GitHub
blueorangutan commented on PR #7237: URL: https://github.com/apache/cloudstack/pull/7237#issuecomment-1587376082 @DaanHoogland a [LL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress

[GitHub] [cloudstack] DaanHoogland closed pull request #7612: [WIP] coverage: fix scaleio plugin code coverage

2023-06-12 Thread via GitHub
DaanHoogland closed pull request #7612: [WIP] coverage: fix scaleio plugin code coverage URL: https://github.com/apache/cloudstack/pull/7612 -- 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 speci

[GitHub] [cloudstack] DaanHoogland commented on pull request #7237: console proxy: use AeadBase64Encryptor instead of AES/CBC/PKCS5Padding

2023-06-12 Thread via GitHub
DaanHoogland commented on PR #7237: URL: https://github.com/apache/cloudstack/pull/7237#issuecomment-1587372707 @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 specific

[GitHub] [cloudstack] DaanHoogland closed pull request #7621: pom.xml: correct url of libvirt maven2

2023-06-12 Thread via GitHub
DaanHoogland closed pull request #7621: pom.xml: correct url of libvirt maven2 URL: https://github.com/apache/cloudstack/pull/7621 -- 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

[GitHub] [cloudstack] weizhouapache commented on pull request #7237: console proxy: use AeadBase64Encryptor instead of AES/CBC/PKCS5Padding

2023-06-12 Thread via GitHub
weizhouapache commented on PR #7237: URL: https://github.com/apache/cloudstack/pull/7237#issuecomment-1587256605 > > we need to test cpvm patch/recreation manually > > Can you add what you expect to the description of this PR, @weizhouapache ? @DaanHoogland done, thanks -- Th

[GitHub] [cloudstack] weizhouapache commented on pull request #7621: pom.xml: correct url of libvirt maven2

2023-06-12 Thread via GitHub
weizhouapache commented on PR #7621: URL: https://github.com/apache/cloudstack/pull/7621#issuecomment-1587254613 > clgrtm, and I can reach https://download.libvirt.org/maven2 but build still failed. this does not help -- This is an automated message from the Apache Git Service. To r

[cloudstack-documentation] branch main updated: Fix spelling gatway to gateway (#323)

2023-06-12 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git The following commit(s) were added to refs/heads/main by this push: new 623a86c Fix spelling gatway to gate

[GitHub] [cloudstack] weizhouapache opened a new pull request, #7621: pom.xml: correct url of libvirt maven2

2023-06-12 Thread via GitHub
weizhouapache opened a new pull request, #7621: URL: https://github.com/apache/cloudstack/pull/7621 ### Description This PR... ### Types of changes - [ ] Breaking change (fix or feature that would cause existing functio

[cloudstack] branch main updated (5fda9c36873 -> 5fa9b8f6834)

2023-06-12 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/cloudstack.git from 5fda9c36873 Upgrade mockito (#7577) add 5fa9b8f6834 cks: enable by default (#7509) No new revisions were added b

[GitHub] [cloudstack] DaanHoogland merged pull request #7509: cks: enable by default

2023-06-12 Thread via GitHub
DaanHoogland merged PR #7509: URL: https://github.com/apache/cloudstack/pull/7509 -- 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-unsubscr...@cl

[GitHub] [cloudstack] DaanHoogland commented on pull request #7237: console proxy: use AeadBase64Encryptor instead of AES/CBC/PKCS5Padding

2023-06-12 Thread via GitHub
DaanHoogland commented on PR #7237: URL: https://github.com/apache/cloudstack/pull/7237#issuecomment-1587206113 > we need to test cpvm patch/recreation manually Can you add what you expect to the description of this PR, @weizhouapache ? -- This is an automated message from the Apach

[GitHub] [cloudstack] weizhouapache commented on pull request #7509: cks: enable by default

2023-06-12 Thread via GitHub
weizhouapache commented on PR #7509: URL: https://github.com/apache/cloudstack/pull/7509#issuecomment-1587198706 > @weizhouapache @DaanHoogland @rohityadavcloud @andrijapanicsb are we okay to merge this? @shwstppr yes, lgtm -- This is an automated message from the Apache Git Servic

[GitHub] [cloudstack] soreana commented on issue #7608: use.system.public.ips does nothing when domain doesn't have a range

2023-06-12 Thread via GitHub
soreana commented on issue #7608: URL: https://github.com/apache/cloudstack/issues/7608#issuecomment-1587180618 @weizhouapache Honestly, Before double checking the global setting definition I wasn't agree with you. But now it makes sense not to touch the global settings behaviour. Sh

[GitHub] [cloudstack] shwstppr commented on pull request #7509: cks: enable by default

2023-06-12 Thread via GitHub
shwstppr commented on PR #7509: URL: https://github.com/apache/cloudstack/pull/7509#issuecomment-1587109802 @weizhouapache @DaanHoogland @rohityadavcloud @andrijapanicsb are we okay to merge this? -- This is an automated message from the Apache Git Service. To respond to the message, plea

[GitHub] [cloudstack] blueorangutan commented on pull request #6979: Guest os mappings improvements

2023-06-12 Thread via GitHub
blueorangutan commented on PR #6979: URL: https://github.com/apache/cloudstack/pull/6979#issuecomment-1587098597 Packaging result [LL]: :heavy_multiplication_x: el7 :heavy_multiplication_x: el8 :heavy_multiplication_x: el9 :heavy_multiplication_x: debian :heavy_multiplication_x: suse15. SL-

[GitHub] [cloudstack] blueorangutan commented on pull request #6979: Guest os mappings improvements

2023-06-12 Thread via GitHub
blueorangutan commented on PR #6979: URL: https://github.com/apache/cloudstack/pull/6979#issuecomment-1587082434 @harikrishna-patnala a [LL] Jenkins job has been kicked to build packages. It will be bundled with SystemVM template(s). I'll keep you posted as I make progress. -- Thi

[GitHub] [cloudstack] harikrishna-patnala commented on pull request #6979: Guest os mappings improvements

2023-06-12 Thread via GitHub
harikrishna-patnala commented on PR #6979: URL: https://github.com/apache/cloudstack/pull/6979#issuecomment-1587081816 @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

[cloudstack] branch main updated: Upgrade mockito (#7577)

2023-06-12 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn 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 5fda9c36873 Upgrade mockito (#7577) 5fda9c36873 is

[GitHub] [cloudstack] DaanHoogland merged pull request #7577: Upgrade mockito

2023-06-12 Thread via GitHub
DaanHoogland merged PR #7577: URL: https://github.com/apache/cloudstack/pull/7577 -- 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-unsubscr...@cl

[GitHub] [cloudstack] weizhouapache commented on pull request #7237: console proxy: use AeadBase64Encryptor instead of AES/CBC/PKCS5Padding

2023-06-12 Thread via GitHub
weizhouapache commented on PR #7237: URL: https://github.com/apache/cloudstack/pull/7237#issuecomment-1587021630 @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 specifi

[GitHub] [cloudstack] kiranchavala commented on issue #7618: Get All Access Info Of a Cluster Kubernetes Through API Key's User

2023-06-12 Thread via GitHub
kiranchavala commented on issue #7618: URL: https://github.com/apache/cloudstack/issues/7618#issuecomment-1587008699 @VanDuy91 I am not able to reproduce this issue, The steps that i followed are Could you please cross check again if they are correct ``` 1. Crea

[GitHub] [cloudstack] soreana commented on issue #7608: use.system.public.ips does nothing when domain doesn't have a range

2023-06-12 Thread via GitHub
soreana commented on issue #7608: URL: https://github.com/apache/cloudstack/issues/7608#issuecomment-1586991837 Hey @DaanHoogland Yes, I'm going to create a PR for this issue. Any suggestion appreciated :) -- This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] [cloudstack] DaanHoogland closed pull request #7617: Retrieve only resource count on user dashboard

2023-06-12 Thread via GitHub
DaanHoogland closed pull request #7617: Retrieve only resource count on user dashboard URL: https://github.com/apache/cloudstack/pull/7617 -- 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] blueorangutan commented on pull request #7237: console proxy: use AeadBase64Encryptor instead of AES/CBC/PKCS5Padding

2023-06-12 Thread via GitHub
blueorangutan commented on PR #7237: URL: https://github.com/apache/cloudstack/pull/7237#issuecomment-1586957948 Packaging result [LL]: :heavy_multiplication_x: el7 :heavy_multiplication_x: el8 :heavy_multiplication_x: el9 :heavy_multiplication_x: debian :heavy_multiplication_x: suse15. SL-

[GitHub] [cloudstack] blueorangutan commented on pull request #7588: Zone wizzard fix

2023-06-12 Thread via GitHub
blueorangutan commented on PR #7588: URL: https://github.com/apache/cloudstack/pull/7588#issuecomment-1586954733 Packaging result [LL]: :heavy_multiplication_x: el7 :heavy_multiplication_x: el8 :heavy_multiplication_x: el9 :heavy_multiplication_x: debian :heavy_multiplication_x: suse15. SL-

[GitHub] [cloudstack] blueorangutan commented on pull request #7237: console proxy: use AeadBase64Encryptor instead of AES/CBC/PKCS5Padding

2023-06-12 Thread via GitHub
blueorangutan commented on PR #7237: URL: https://github.com/apache/cloudstack/pull/7237#issuecomment-1586934587 @rohityadavcloud a [LL] Jenkins job has been kicked to build packages. It will be bundled with SystemVM template(s). I'll keep you posted as I make progress. -- This is

[GitHub] [cloudstack] blueorangutan commented on pull request #7588: Zone wizzard fix

2023-06-12 Thread via GitHub
blueorangutan commented on PR #7588: URL: https://github.com/apache/cloudstack/pull/7588#issuecomment-1586931573 @rohityadavcloud a [LL] Jenkins job has been kicked to build packages. It will be bundled with SystemVM template(s). I'll keep you posted as I make progress. -- This is

[GitHub] [cloudstack] rohityadavcloud commented on pull request #7237: console proxy: use AeadBase64Encryptor instead of AES/CBC/PKCS5Padding

2023-06-12 Thread via GitHub
rohityadavcloud commented on PR #7237: URL: https://github.com/apache/cloudstack/pull/7237#issuecomment-1586931266 @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

[GitHub] [cloudstack] rohityadavcloud commented on pull request #7588: Zone wizzard fix

2023-06-12 Thread via GitHub
rohityadavcloud commented on PR #7588: URL: https://github.com/apache/cloudstack/pull/7588#issuecomment-1586930873 @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

[GitHub] [cloudstack] rohityadavcloud closed issue #7547: CloudStack agent trims Host.OS in the response to management server

2023-06-12 Thread via GitHub
rohityadavcloud closed issue #7547: CloudStack agent trims Host.OS in the response to management server URL: https://github.com/apache/cloudstack/issues/7547 -- 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] rohityadavcloud commented on issue #7547: CloudStack agent trims Host.OS in the response to management server

2023-06-12 Thread via GitHub
rohityadavcloud commented on issue #7547: URL: https://github.com/apache/cloudstack/issues/7547#issuecomment-1586925558 Fixed in https://github.com/apache/cloudstack/pull/7570 I see a new https://github.com/apache/cloudstack/issues/7616 so this issue can be closed -- This is an automated

[GitHub] [cloudstack] blueorangutan commented on pull request #7597: consider last host only for root-admin

2023-06-12 Thread via GitHub
blueorangutan commented on PR #7597: URL: https://github.com/apache/cloudstack/pull/7597#issuecomment-1586883551 Packaging result [LL]: :heavy_multiplication_x: el7 :heavy_multiplication_x: el8 :heavy_multiplication_x: el9 :heavy_multiplication_x: debian :heavy_multiplication_x: suse15. SL-

[GitHub] [cloudstack] MariPS commented on issue #7614: new feat: instantiate a new VM without NIC

2023-06-12 Thread via GitHub
MariPS commented on issue #7614: URL: https://github.com/apache/cloudstack/issues/7614#issuecomment-1586867341 Thank you for the quick reply. As for answering to your questions: > @MariPS , if a VM without NIC is still connecting automatically to the net,.. How would that work?

[GitHub] [cloudstack] blueorangutan commented on pull request #7597: consider last host only for root-admin

2023-06-12 Thread via GitHub
blueorangutan commented on PR #7597: URL: https://github.com/apache/cloudstack/pull/7597#issuecomment-1586863041 @DaanHoogland a [LL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress

[GitHub] [cloudstack] DaanHoogland commented on pull request #7597: consider last host only for root-admin

2023-06-12 Thread via GitHub
DaanHoogland commented on PR #7597: URL: https://github.com/apache/cloudstack/pull/7597#issuecomment-1586859659 @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 specific

[GitHub] [cloudstack] blueorangutan commented on pull request #7408: Allow volume migrations in ScaleIO within and across ScaleIO storage clusters

2023-06-12 Thread via GitHub
blueorangutan commented on PR #7408: URL: https://github.com/apache/cloudstack/pull/7408#issuecomment-1586859121 @borisstoyanov a [SF] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests -- This is an automated message from the Apache Git Service. To

[GitHub] [cloudstack] borisstoyanov commented on pull request #7408: Allow volume migrations in ScaleIO within and across ScaleIO storage clusters

2023-06-12 Thread via GitHub
borisstoyanov commented on PR #7408: URL: https://github.com/apache/cloudstack/pull/7408#issuecomment-1586857941 @blueorangutan test -- 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 c

[GitHub] [cloudstack] shwstppr commented on a diff in pull request #7588: Zone wizzard fix

2023-06-12 Thread via GitHub
shwstppr commented on code in PR #7588: URL: https://github.com/apache/cloudstack/pull/7588#discussion_r1226296645 ## plugins/hypervisors/vmware/src/main/java/com/cloud/hypervisor/vmware/manager/VmwareManagerImpl.java: ## @@ -1243,6 +1237,46 @@ public VmwareDatacenterVO addVmwa

[GitHub] [cloudstack] DaanHoogland commented on a diff in pull request #7588: Zone wizzard fix

2023-06-12 Thread via GitHub
DaanHoogland commented on code in PR #7588: URL: https://github.com/apache/cloudstack/pull/7588#discussion_r1226290359 ## plugins/hypervisors/vmware/src/main/java/com/cloud/hypervisor/vmware/manager/VmwareManagerImpl.java: ## @@ -1243,6 +1237,46 @@ public VmwareDatacenterVO add

[GitHub] [cloudstack] weizhouapache commented on pull request #7620: [AGENT] host os accepted if only the first word is defined for the cluster

2023-06-12 Thread via GitHub
weizhouapache commented on PR #7620: URL: https://github.com/apache/cloudstack/pull/7620#issuecomment-1586836125 > @weizhouapache , architecturally I don't whink this is the best solution but it will allow running environments to keep running. I don't know of an example where two different

[GitHub] [cloudstack] blueorangutan commented on pull request #7620: [AGENT] host os accepted if only the first word is defined for the cluster

2023-06-12 Thread via GitHub
blueorangutan commented on PR #7620: URL: https://github.com/apache/cloudstack/pull/7620#issuecomment-1586820793 Packaging result [LL]: :heavy_multiplication_x: el7 :heavy_multiplication_x: el8 :heavy_multiplication_x: el9 :heavy_multiplication_x: debian :heavy_multiplication_x: suse15. SL-

[GitHub] [cloudstack] blueorangutan commented on pull request #7620: [AGENT] host os accepted if only the first word is defined for the cluster

2023-06-12 Thread via GitHub
blueorangutan commented on PR #7620: URL: https://github.com/apache/cloudstack/pull/7620#issuecomment-1586788642 @DaanHoogland a [LL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress

[GitHub] [cloudstack] DaanHoogland commented on pull request #7620: [AGENT] host os accepted if only the first word is defined for the cluster

2023-06-12 Thread via GitHub
DaanHoogland commented on PR #7620: URL: https://github.com/apache/cloudstack/pull/7620#issuecomment-1586783501 @weizhouapache , architecturally I don't whink this is the best solution but it will allow running environments to keep running. I don't know of an example where two different OSs

[GitHub] [cloudstack] DaanHoogland commented on pull request #7620: [AGENT] host os accepted if only the first word is defined for the cluster

2023-06-12 Thread via GitHub
DaanHoogland commented on PR #7620: URL: https://github.com/apache/cloudstack/pull/7620#issuecomment-1586784168 @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 specific

[GitHub] [cloudstack] DaanHoogland opened a new pull request, #7620: [AGENT] host os accepted if only the first word is defined for the cluster

2023-06-12 Thread via GitHub
DaanHoogland opened a new pull request, #7620: URL: https://github.com/apache/cloudstack/pull/7620 ### Description This PR... Fixes: #7616 ### Types of changes - [ ] Breaking change (fix or feature that would cause existi

[GitHub] [cloudstack] weizhouapache commented on pull request #7617: Retrieve only resource count on user dashboard

2023-06-12 Thread via GitHub
weizhouapache commented on PR #7617: URL: https://github.com/apache/cloudstack/pull/7617#issuecomment-1586719080 @GutoVeronezi I think an API is better. back to your pr, I think it would be better to create an abstract class . -- This is an automated message from the Apache Git

[GitHub] [cloudstack] weizhouapache closed issue #7162: Pass the addresses of all image stores to the SSVM

2023-06-12 Thread via GitHub
weizhouapache closed issue #7162: Pass the addresses of all image stores to the SSVM URL: https://github.com/apache/cloudstack/issues/7162 -- 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 specif

[GitHub] [cloudstack] weizhouapache commented on issue #7162: Pass the addresses of all image stores to the SSVM

2023-06-12 Thread via GitHub
weizhouapache commented on issue #7162: URL: https://github.com/apache/cloudstack/issues/7162#issuecomment-1586707937 fixed via #7410 thanks @shwstppr -- 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

  1   2   >