Re: [PR] API: Fix missing keys in listZonesMetrics response [cloudstack]

2024-07-28 Thread via GitHub
rohityadavcloud merged PR #9419: URL: https://github.com/apache/cloudstack/pull/9419 -- 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...

Re: [PR] API: Fix missing keys in listZonesMetrics response [cloudstack]

2024-07-24 Thread via GitHub
blueorangutan commented on PR #9419: URL: https://github.com/apache/cloudstack/pull/9419#issuecomment-2249118902 [SF] Trillian test result (tid-10955) Environment: kvm-rocky8 (x2), Advanced Networking with Mgmt server r8 Total time taken: 50027 seconds Marvin logs: https://github.co

Re: [PR] API: Fix missing keys in listZonesMetrics response [cloudstack]

2024-07-24 Thread via GitHub
blueorangutan commented on PR #9419: URL: https://github.com/apache/cloudstack/pull/9419#issuecomment-2247485431 @vishesh92 a [SL] Trillian-Jenkins test job (rocky8 mgmt + kvm-rocky8) has been kicked to run smoke tests -- This is an automated message from the Apache Git Service. To respon

Re: [PR] API: Fix missing keys in listZonesMetrics response [cloudstack]

2024-07-24 Thread via GitHub
vishesh92 commented on PR #9419: URL: https://github.com/apache/cloudstack/pull/9419#issuecomment-2247478437 @blueorangutan test rocky8 kvm-rocky8 -- 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

Re: [PR] API: Fix missing keys in listZonesMetrics response [cloudstack]

2024-07-23 Thread via GitHub
blueorangutan commented on PR #9419: URL: https://github.com/apache/cloudstack/pull/9419#issuecomment-2246286105 Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 10449 -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [PR] API: Fix missing keys in listZonesMetrics response [cloudstack]

2024-07-23 Thread via GitHub
blueorangutan commented on PR #9419: URL: https://github.com/apache/cloudstack/pull/9419#issuecomment-2246151347 @vishesh92 a [SL] 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.

Re: [PR] API: Fix missing keys in listZonesMetrics response [cloudstack]

2024-07-23 Thread via GitHub
vishesh92 commented on PR #9419: URL: https://github.com/apache/cloudstack/pull/9419#issuecomment-2246150176 @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 co

Re: [PR] API: Fix missing keys in listZonesMetrics response [cloudstack]

2024-07-22 Thread via GitHub
blueorangutan commented on PR #9419: URL: https://github.com/apache/cloudstack/pull/9419#issuecomment-2243979381 [SF] Trillian test result (tid-10930) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 54045 seconds Marvin logs: https://github.co

Re: [PR] API: Fix missing keys in listZonesMetrics response [cloudstack]

2024-07-22 Thread via GitHub
blueorangutan commented on PR #9419: URL: https://github.com/apache/cloudstack/pull/9419#issuecomment-2242336663 @rohityadavcloud a [SL] 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. T

Re: [PR] API: Fix missing keys in listZonesMetrics response [cloudstack]

2024-07-22 Thread via GitHub
rohityadavcloud commented on PR #9419: URL: https://github.com/apache/cloudstack/pull/9419#issuecomment-2242335184 @vishesh92 also worth checking for list APIs & list metrics APIs for cluster and host? (and may VM and volumes)? @blueorangutan test -- This is an automated message fr

Re: [PR] API: Fix missing keys in listZonesMetrics response [cloudstack]

2024-07-19 Thread via GitHub
blueorangutan commented on PR #9419: URL: https://github.com/apache/cloudstack/pull/9419#issuecomment-2239084537 Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 10412 -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [PR] API: Fix missing keys in listZonesMetrics response [cloudstack]

2024-07-19 Thread via GitHub
blueorangutan commented on PR #9419: URL: https://github.com/apache/cloudstack/pull/9419#issuecomment-2238996550 @vishesh92 a [SL] 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.

Re: [PR] API: Fix missing keys in listZonesMetrics response [cloudstack]

2024-07-19 Thread via GitHub
vishesh92 commented on PR #9419: URL: https://github.com/apache/cloudstack/pull/9419#issuecomment-2238993956 @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 co

Re: [PR] API: Fix missing keys in listZonesMetrics response [cloudstack]

2024-07-19 Thread via GitHub
github-actions[bot] commented on PR #9419: URL: https://github.com/apache/cloudstack/pull/9419#issuecomment-2238986244 This pull request has merge conflicts. Dear author, please fix the conflicts and sync your branch with the base branch. -- This is an automated message from the Apache Gi

Re: [PR] API: Fix missing keys in listZonesMetrics response [cloudstack]

2024-07-19 Thread via GitHub
DaanHoogland commented on code in PR #9419: URL: https://github.com/apache/cloudstack/pull/9419#discussion_r1684237382 ## api/src/main/java/org/apache/cloudstack/api/response/ZoneResponse.java: ## @@ -344,6 +336,30 @@ public Map getResourceDetails() { return resourceDet

Re: [PR] API: Fix missing keys in listZonesMetrics response [cloudstack]

2024-07-19 Thread via GitHub
rohityadavcloud commented on PR #9419: URL: https://github.com/apache/cloudstack/pull/9419#issuecomment-2238945066 @vishesh92 LGTM didn't test it, can you raise this for 4.19 branch? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

Re: [PR] API: Fix missing keys in listZonesMetrics response [cloudstack]

2024-07-19 Thread via GitHub
vishesh92 commented on code in PR #9419: URL: https://github.com/apache/cloudstack/pull/9419#discussion_r1684137783 ## api/src/main/java/org/apache/cloudstack/api/response/ZoneResponse.java: ## @@ -344,6 +336,30 @@ public Map getResourceDetails() { return resourceDetail

Re: [PR] API: Fix missing keys in listZonesMetrics response [cloudstack]

2024-07-19 Thread via GitHub
vishesh92 commented on code in PR #9419: URL: https://github.com/apache/cloudstack/pull/9419#discussion_r1684128515 ## api/src/main/java/org/apache/cloudstack/api/response/ZoneResponse.java: ## @@ -344,6 +336,30 @@ public Map getResourceDetails() { return resourceDetail

Re: [PR] API: Fix missing keys in listZonesMetrics response [cloudstack]

2024-07-19 Thread via GitHub
DaanHoogland commented on code in PR #9419: URL: https://github.com/apache/cloudstack/pull/9419#discussion_r1684116321 ## api/src/main/java/org/apache/cloudstack/api/response/ZoneResponse.java: ## @@ -344,6 +336,30 @@ public Map getResourceDetails() { return resourceDet

Re: [PR] API: Fix missing keys in listZonesMetrics response [cloudstack]

2024-07-19 Thread via GitHub
blueorangutan commented on PR #9419: URL: https://github.com/apache/cloudstack/pull/9419#issuecomment-2238730311 Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 10408 -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [PR] API: Fix missing keys in listZonesMetrics response [cloudstack]

2024-07-19 Thread via GitHub
codecov[bot] commented on PR #9419: URL: https://github.com/apache/cloudstack/pull/9419#issuecomment-2238610442 ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/9419?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments

[PR] API: Fix missing keys in listZonesMetrics response [cloudstack]

2024-07-19 Thread via GitHub
vishesh92 opened a new pull request, #9419: URL: https://github.com/apache/cloudstack/pull/9419 ### Description This PR fixes https://github.com/apache/cloudstack/issues/9379 Fix missing keys in `listZonesMetrics`. We use `BeanUtils.copyProperties` to copy the properties of `

Re: [PR] API: Fix missing keys in listZonesMetrics response [cloudstack]

2024-07-19 Thread via GitHub
blueorangutan commented on PR #9419: URL: https://github.com/apache/cloudstack/pull/9419#issuecomment-2238596174 @vishesh92 a [SL] 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.

Re: [PR] API: Fix missing keys in listZonesMetrics response [cloudstack]

2024-07-19 Thread via GitHub
vishesh92 commented on PR #9419: URL: https://github.com/apache/cloudstack/pull/9419#issuecomment-2238593783 @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 co