Re: [PR] Move views into separate files for easier modification [cloudstack]

2024-02-16 Thread via GitHub
blueorangutan commented on PR #8647: URL: https://github.com/apache/cloudstack/pull/8647#issuecomment-1949486290 [SF] Trillian test result (tid-9253) Environment: xenserver-71 (x2), Advanced Networking with Mgmt server 7 Total time taken: 44956 seconds Marvin logs:

Re: [PR] Allow altering only either CPU or memory during VM live scale [cloudstack]

2024-02-16 Thread via GitHub
weizhouapache commented on code in PR #8234: URL: https://github.com/apache/cloudstack/pull/8234#discussion_r1493037323 ## server/src/main/java/com/cloud/vm/UserVmManagerImpl.java: ## @@ -1892,7 +1925,11 @@ public UserVm upgradeVirtualMachine(ScaleVMCmd cmd) throws

Re: [PR] Allow altering only either CPU or memory during VM live scale [cloudstack]

2024-02-16 Thread via GitHub
GutoVeronezi commented on code in PR #8234: URL: https://github.com/apache/cloudstack/pull/8234#discussion_r1492984147 ## server/src/main/java/com/cloud/vm/UserVmManagerImpl.java: ## @@ -1892,7 +1925,11 @@ public UserVm upgradeVirtualMachine(ScaleVMCmd cmd) throws

Re: [PR] Storage plugin support to check if volume on datastore requires access for migration [cloudstack]

2024-02-16 Thread via GitHub
blueorangutan commented on PR #8655: URL: https://github.com/apache/cloudstack/pull/8655#issuecomment-1949327252 [SF] Trillian test result (tid-9251) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 41762 seconds Marvin logs:

Re: [PR] Allow altering only either CPU or memory during VM live scale [cloudstack]

2024-02-16 Thread via GitHub
weizhouapache commented on code in PR #8234: URL: https://github.com/apache/cloudstack/pull/8234#discussion_r1492977792 ## server/src/main/java/com/cloud/vm/UserVmManagerImpl.java: ## @@ -1892,7 +1925,11 @@ public UserVm upgradeVirtualMachine(ScaleVMCmd cmd) throws

Re: [PR] Allow altering only either CPU or memory during VM live scale [cloudstack]

2024-02-16 Thread via GitHub
GutoVeronezi commented on code in PR #8234: URL: https://github.com/apache/cloudstack/pull/8234#discussion_r1492965095 ## server/src/main/java/com/cloud/vm/UserVmManagerImpl.java: ## @@ -1892,7 +1925,11 @@ public UserVm upgradeVirtualMachine(ScaleVMCmd cmd) throws

Re: [PR] Allow altering only either CPU or memory during VM live scale [cloudstack]

2024-02-16 Thread via GitHub
weizhouapache commented on code in PR #8234: URL: https://github.com/apache/cloudstack/pull/8234#discussion_r1492949837 ## server/src/main/java/com/cloud/vm/UserVmManagerImpl.java: ## @@ -1892,7 +1925,11 @@ public UserVm upgradeVirtualMachine(ScaleVMCmd cmd) throws

Re: [I] Trying to register a template with a long URL results in error [cloudstack]

2024-02-16 Thread via GitHub
weizhouapache commented on issue #8669: URL: https://github.com/apache/cloudstack/issues/8669#issuecomment-1949259798 > yeah @fabiomatavelli-f3 , I got the problem and the fix is easy but the question is in which version we could fix this. a dirty hack for you would just be `ALTER TABLE

Re: [PR] Quota tariff order [cloudstack]

2024-02-16 Thread via GitHub
JoaoJandre commented on code in PR #8347: URL: https://github.com/apache/cloudstack/pull/8347#discussion_r1492904222 ## plugins/database/quota/src/main/java/org/apache/cloudstack/api/command/QuotaTariffCreateCmd.java: ## @@ -66,7 +66,7 @@ public class QuotaTariffCreateCmd

Re: [PR] Fixup response code on incorrect credentials [cloudstack]

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

Re: [PR] Changes error message when using invalid `endpoint.url` [cloudstack]

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

Re: [PR] test: fix test_guest_os.py failure on xcpng82 [cloudstack]

2024-02-16 Thread via GitHub
weizhouapache commented on PR #8659: URL: https://github.com/apache/cloudstack/pull/8659#issuecomment-1949113125 > Therre are many failures on xcpng82 still. Are these related, @weizhouapache ? > Should we create separate issues/PRs for those? Hi @DaanHoogland Please check

Re: [PR] Fix 404 when moving a VM to out of a project [cloudstack]

2024-02-16 Thread via GitHub
github-actions[bot] commented on PR #8650: URL: https://github.com/apache/cloudstack/pull/8650#issuecomment-1949099479 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

Re: [I] Restarting management server causes multiple entries for the same node [cloudstack]

2024-02-16 Thread via GitHub
weizhouapache commented on issue #8174: URL: https://github.com/apache/cloudstack/issues/8174#issuecomment-1949100037 > > If a new record each time when management server is restarted, it might be because the management server id is calculated from a random mac address. > >

Re: [I] Restarting management server causes multiple entries for the same node [cloudstack]

2024-02-16 Thread via GitHub
poddm commented on issue #8174: URL: https://github.com/apache/cloudstack/issues/8174#issuecomment-1949089310 > If a new record each time when management server is restarted, it might be because the management server id is calculated from a random mac address. > >

Re: [PR] test: fix test_guest_os.py failure on xcpng82 [cloudstack]

2024-02-16 Thread via GitHub
DaanHoogland commented on PR #8659: URL: https://github.com/apache/cloudstack/pull/8659#issuecomment-1949049164 Therre are many failures on xcpng82 still. Are these related, @weizhouapache ? Should we create separate issues/PRs for those? -- This is an automated message from the

Re: [PR] Fixup response code on incorrect credentials [cloudstack]

2024-02-16 Thread via GitHub
blueorangutan commented on PR #8671: URL: https://github.com/apache/cloudstack/pull/8671#issuecomment-1949035653 @DaanHoogland 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

Re: [PR] Improve logs when adding components to avoid set [cloudstack]

2024-02-16 Thread via GitHub
blueorangutan commented on PR #7214: URL: https://github.com/apache/cloudstack/pull/7214#issuecomment-1949035568 @DaanHoogland 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. To

Re: [PR] Improve logs when adding components to avoid set [cloudstack]

2024-02-16 Thread via GitHub
DaanHoogland commented on PR #7214: URL: https://github.com/apache/cloudstack/pull/7214#issuecomment-1949034389 @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

Re: [PR] Fixup response code on incorrect credentials [cloudstack]

2024-02-16 Thread via GitHub
DaanHoogland commented on PR #8671: URL: https://github.com/apache/cloudstack/pull/8671#issuecomment-1949030397 @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

Re: [PR] Quota tariff order [cloudstack]

2024-02-16 Thread via GitHub
DaanHoogland commented on code in PR #8347: URL: https://github.com/apache/cloudstack/pull/8347#discussion_r1492827492 ## plugins/database/quota/src/main/java/org/apache/cloudstack/api/command/QuotaTariffUpdateCmd.java: ## @@ -67,7 +67,7 @@ public class QuotaTariffUpdateCmd

Re: [PR] test: fix test_guest_os.py failure on xcpng82 [cloudstack]

2024-02-16 Thread via GitHub
blueorangutan commented on PR #8659: URL: https://github.com/apache/cloudstack/pull/8659#issuecomment-1949016087 [SF] Trillian test result (tid-9242) Environment: xcpng82 (x2), Advanced Networking with Mgmt server r8 Total time taken: 65899 seconds Marvin logs:

Re: [PR] Changes error message when using invalid `endpoint.url` [cloudstack]

2024-02-16 Thread via GitHub
blueorangutan commented on PR #8603: URL: https://github.com/apache/cloudstack/pull/8603#issuecomment-1949006281 @weizhouapache 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

Re: [PR] Changes error message when using invalid `endpoint.url` [cloudstack]

2024-02-16 Thread via GitHub
weizhouapache commented on PR #8603: URL: https://github.com/apache/cloudstack/pull/8603#issuecomment-1949000751 @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

Re: [PR] prevent duplicate ip table rules in SSVM [cloudstack]

2024-02-16 Thread via GitHub
blueorangutan commented on PR #8530: URL: https://github.com/apache/cloudstack/pull/8530#issuecomment-1948960429 @weizhouapache 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. To

Re: [PR] prevent duplicate ip table rules in SSVM [cloudstack]

2024-02-16 Thread via GitHub
weizhouapache commented on PR #8530: URL: https://github.com/apache/cloudstack/pull/8530#issuecomment-1948959646 @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

Re: [PR] api,server,ui: granular resource limit management [cloudstack]

2024-02-16 Thread via GitHub
GutoVeronezi commented on PR #8362: URL: https://github.com/apache/cloudstack/pull/8362#issuecomment-1948864377 > @JoaoJandre @GutoVeronezi , this has been reviewed by several and has gone through our QA procedure (manual and automated testing. Do you guys hace any concerns with merging

Re: [PR] Improve logs when adding components to avoid set [cloudstack]

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

Re: [PR] Add logs to `LibvirtComputingResource`'s metrics collection process [cloudstack]

2024-02-16 Thread via GitHub
winterhazel commented on PR #8511: URL: https://github.com/apache/cloudstack/pull/8511#issuecomment-1948704455 @DaanHoogland yes, some users have already testes these changes. I can try asking someone to post their results here. Though, looking back at this PR, I think I may have

Re: [PR] prevent duplicate ip table rules in SSVM [cloudstack]

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

Re: [I] Trying to register a template with a long URL results in error [cloudstack]

2024-02-16 Thread via GitHub
DaanHoogland commented on issue #8669: URL: https://github.com/apache/cloudstack/issues/8669#issuecomment-1948573793 yeah @fabiomatavelli-f3 , I got the problem and the fix is easy but the question is in which version we could fix this. a dirty hack for you would just be `ALTER TABLE

Re: [PR] Polish UI instance schedule modal [cloudstack]

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

Re: [PR] Polish UI instance schedule modal [cloudstack]

2024-02-16 Thread via GitHub
DaanHoogland commented on PR #8620: URL: https://github.com/apache/cloudstack/pull/8620#issuecomment-1948551395 @blueorangutan test alma9 kvm-alma9 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

Re: [PR] Improve logs when adding components to avoid set [cloudstack]

2024-02-16 Thread via GitHub
blueorangutan commented on PR #7214: URL: https://github.com/apache/cloudstack/pull/7214#issuecomment-1948540229 @DaanHoogland 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

Re: [I] Trying to register a template with a long URL results in error [cloudstack]

2024-02-16 Thread via GitHub
fabiomatavelli-f3 commented on issue #8669: URL: https://github.com/apache/cloudstack/issues/8669#issuecomment-1948539004 indeed @DaanHoogland , I think even 1024 should be enough. Problem with S3 signed URLs is that they have the tokens/signature in the URL. -- This is an automated

Re: [PR] Improve logs when adding components to avoid set [cloudstack]

2024-02-16 Thread via GitHub
DaanHoogland commented on PR #7214: URL: https://github.com/apache/cloudstack/pull/7214#issuecomment-1948537625 @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

Re: [PR] server: apply rules when VR of Domain VPC is recreated [cloudstack]

2024-02-16 Thread via GitHub
blueorangutan commented on PR #8354: URL: https://github.com/apache/cloudstack/pull/8354#issuecomment-1948533468 @DaanHoogland 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. To

Re: [PR] Allow deletion of system VM templates [cloudstack]

2024-02-16 Thread via GitHub
blueorangutan commented on PR #8556: URL: https://github.com/apache/cloudstack/pull/8556#issuecomment-1948533412 @DaanHoogland 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. To

Re: [PR] Allow deletion of system VM templates [cloudstack]

2024-02-16 Thread via GitHub
DaanHoogland commented on PR #8556: URL: https://github.com/apache/cloudstack/pull/8556#issuecomment-1948532722 @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

Re: [PR] server: apply rules when VR of Domain VPC is recreated [cloudstack]

2024-02-16 Thread via GitHub
DaanHoogland commented on PR #8354: URL: https://github.com/apache/cloudstack/pull/8354#issuecomment-1948531531 @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

Re: [I] Trying to register a template with a long URL results in error [cloudstack]

2024-02-16 Thread via GitHub
DaanHoogland commented on issue #8669: URL: https://github.com/apache/cloudstack/issues/8669#issuecomment-1948525108 so would changing this to 2048 be enough? an url should not be a blob or such, but some beyond 255 seems reasonable. @JoaoJandre @sureshanaparti and all others, as

Re: [PR] prevent duplicate ip table rules in SSVM [cloudstack]

2024-02-16 Thread via GitHub
blueorangutan commented on PR #8530: URL: https://github.com/apache/cloudstack/pull/8530#issuecomment-1948491191 @DaanHoogland 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

Re: [PR] prevent duplicate ip table rules in SSVM [cloudstack]

2024-02-16 Thread via GitHub
DaanHoogland commented on PR #8530: URL: https://github.com/apache/cloudstack/pull/8530#issuecomment-1948488552 @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

Re: [PR] Fix false positives in Usage sanity checks of templates and network offerings [cloudstack]

2024-02-16 Thread via GitHub
DaanHoogland commented on PR #8136: URL: https://github.com/apache/cloudstack/pull/8136#issuecomment-1948475804 I guess it kind of worked because ``` 14:17:18.487 [main] ERROR com.cloud.usage.UsageSanityChecker - Error: Found 919 usage records with raw_usage > 24 ``` after

Re: [PR] Fixup response code on incorrect credentials [cloudstack]

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

Re: [PR] CKS: add ConfigDrive to cloud-init datasource_list in systemvm template [cloudstack]

2024-02-16 Thread via GitHub
blueorangutan commented on PR #7650: URL: https://github.com/apache/cloudstack/pull/7650#issuecomment-1948462592 [SF] Trillian test result (tid-9244) Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server r8 Total time taken: 51124 seconds Marvin logs:

Re: [PR] Fixup response code on incorrect credentials [cloudstack]

2024-02-16 Thread via GitHub
sureshanaparti commented on code in PR #8671: URL: https://github.com/apache/cloudstack/pull/8671#discussion_r1492522799 ## server/src/main/java/com/cloud/user/AccountManagerImpl.java: ## @@ -2650,7 +2650,8 @@ private UserAccount getUserAccount(String username, String

Re: [PR] Linstor 4.19 fix script alllines [cloudstack]

2024-02-16 Thread via GitHub
codecov[bot] commented on PR #8670: URL: https://github.com/apache/cloudstack/pull/8670#issuecomment-1948459285 ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/8670?src=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=apache) Report All

Re: [PR] Linstor 4.19 fix selecting non enabled hosts [cloudstack]

2024-02-16 Thread via GitHub
sureshanaparti commented on code in PR #8653: URL: https://github.com/apache/cloudstack/pull/8653#discussion_r1492518286 ## plugins/storage/volume/linstor/src/main/java/org/apache/cloudstack/storage/datastore/util/LinstorUtil.java: ## @@ -107,13 +107,23 @@ public static List

Re: [PR] Linstor 4.19 fix selecting non enabled hosts [cloudstack]

2024-02-16 Thread via GitHub
sureshanaparti commented on code in PR #8653: URL: https://github.com/apache/cloudstack/pull/8653#discussion_r1492518286 ## plugins/storage/volume/linstor/src/main/java/org/apache/cloudstack/storage/datastore/util/LinstorUtil.java: ## @@ -107,13 +107,23 @@ public static List

Re: [PR] Refactor updateDiskOffering API [cloudstack]

2024-02-16 Thread via GitHub
blueorangutan commented on PR #8446: URL: https://github.com/apache/cloudstack/pull/8446#issuecomment-1948446868 @DaanHoogland 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. To

Re: [PR] Refactor updateDiskOffering API [cloudstack]

2024-02-16 Thread via GitHub
DaanHoogland commented on PR #8446: URL: https://github.com/apache/cloudstack/pull/8446#issuecomment-1948446108 @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

(cloudstack) branch 4.18 updated: CKS: add kube config path in extra control nodes (#8658)

2024-02-16 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a commit to branch 4.18 in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/4.18 by this push: new 8d4b4dcec42 CKS: add kube config path in extra

Re: [PR] CKS: add kube config path in extra control nodes [cloudstack]

2024-02-16 Thread via GitHub
DaanHoogland merged PR #8658: URL: https://github.com/apache/cloudstack/pull/8658 -- 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:

Re: [PR] CKS: add kube config path in extra control nodes [cloudstack]

2024-02-16 Thread via GitHub
DaanHoogland commented on PR #8658: URL: https://github.com/apache/cloudstack/pull/8658#issuecomment-1948438959 verified manually -- 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

Re: [PR] Fixup response code on incorrect credentials [cloudstack]

2024-02-16 Thread via GitHub
codecov[bot] commented on PR #8671: URL: https://github.com/apache/cloudstack/pull/8671#issuecomment-1948433944 ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/8671?src=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=apache) Report All

Re: [PR] Polish UI instance schedule modal [cloudstack]

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

Re: [PR] Fixup response code on incorrect credentials [cloudstack]

2024-02-16 Thread via GitHub
blueorangutan commented on PR #8671: URL: https://github.com/apache/cloudstack/pull/8671#issuecomment-1948424448 @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] Fixup response code on incorrect credentials [cloudstack]

2024-02-16 Thread via GitHub
vishesh92 commented on PR #8671: URL: https://github.com/apache/cloudstack/pull/8671#issuecomment-1948421834 @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

[PR] Fixup response code on incorrect credentials [cloudstack]

2024-02-16 Thread via GitHub
vishesh92 opened a new pull request, #8671: URL: https://github.com/apache/cloudstack/pull/8671 ### Description This PR fixes https://github.com/apache/cloudstack/issues/8662 While logging in, if the user doesn't exist it fails with a valid error

(cloudstack) branch 4.19 updated: prevent nic removal on out of bounds router stop (#8371)

2024-02-16 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a commit to branch 4.19 in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/4.19 by this push: new a0e592e945e prevent nic removal on out of bounds

Re: [PR] prevent nic removal on out of bounds router stop [cloudstack]

2024-02-16 Thread via GitHub
DaanHoogland merged PR #8371: URL: https://github.com/apache/cloudstack/pull/8371 -- 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:

Re: [PR] api,server,ui: granular resource limit management [cloudstack]

2024-02-16 Thread via GitHub
DaanHoogland commented on PR #8362: URL: https://github.com/apache/cloudstack/pull/8362#issuecomment-1948390617 @JoaoJandre @GutoVeronezi , this has been reviewed by several and has gone through our QA procedure (manual and automated testing. Do you guys hace any concerns with merging

Re: [PR] kvm: support Rocky/RHEL/OL/Alma in the same cluster [cloudstack]

2024-02-16 Thread via GitHub
blueorangutan commented on PR #8641: URL: https://github.com/apache/cloudstack/pull/8641#issuecomment-1948355786 @DaanHoogland 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. To

Re: [PR] kvm: support Rocky/RHEL/OL/Alma in the same cluster [cloudstack]

2024-02-16 Thread via GitHub
DaanHoogland commented on PR #8641: URL: https://github.com/apache/cloudstack/pull/8641#issuecomment-1948354769 @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

Re: [PR] [Quota] Update pre-populated Quota tariffs' type [cloudstack]

2024-02-16 Thread via GitHub
blueorangutan commented on PR #8591: URL: https://github.com/apache/cloudstack/pull/8591#issuecomment-1948353084 @DaanHoogland 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. To

Re: [PR] Linstor 4.19 fix selecting non enabled hosts [cloudstack]

2024-02-16 Thread via GitHub
blueorangutan commented on PR #8653: URL: https://github.com/apache/cloudstack/pull/8653#issuecomment-1948353027 @DaanHoogland 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. To

Re: [PR] Linstor 4.19 fix selecting non enabled hosts [cloudstack]

2024-02-16 Thread via GitHub
DaanHoogland commented on PR #8653: URL: https://github.com/apache/cloudstack/pull/8653#issuecomment-1948351710 @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

Re: [PR] [Quota] Update pre-populated Quota tariffs' type [cloudstack]

2024-02-16 Thread via GitHub
DaanHoogland commented on PR #8591: URL: https://github.com/apache/cloudstack/pull/8591#issuecomment-1948350579 @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

Re: [PR] Polish UI instance schedule modal [cloudstack]

2024-02-16 Thread via GitHub
blueorangutan commented on PR #8620: URL: https://github.com/apache/cloudstack/pull/8620#issuecomment-1948342099 @DaanHoogland 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

Re: [PR] Polish UI instance schedule modal [cloudstack]

2024-02-16 Thread via GitHub
DaanHoogland commented on PR #8620: URL: https://github.com/apache/cloudstack/pull/8620#issuecomment-1948341057 @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

(cloudstack-www) branch staged-site updated: Automated CloudStack https://cloudstack.staged.apache.org staging website deployment with cloudstack-www branch @ 3102a1d6377180853b4247e5384c02112bd23968

2024-02-16 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a commit to branch staged-site in repository https://gitbox.apache.org/repos/asf/cloudstack-www.git The following commit(s) were added to refs/heads/staged-site by this push: new 7d20e119a Automated

(cloudstack-www) branch staging-site updated: enhancement/annotation of full publication procedure

2024-02-16 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a commit to branch staging-site in repository https://gitbox.apache.org/repos/asf/cloudstack-www.git The following commit(s) were added to refs/heads/staging-site by this push: new 3102a1d63

Re: [I] ACS 4.19 | No Auth Failure Message Displayed when login Password is wrong [cloudstack]

2024-02-16 Thread via GitHub
vishesh92 commented on issue #8662: URL: https://github.com/apache/cloudstack/issues/8662#issuecomment-1948325330 I am able to reproduce this. While logging in, if the user doesn't exist it fails with a valid error

[PR] Linstor 4.19 fix script alllines [cloudstack]

2024-02-16 Thread via GitHub
rp- opened a new pull request, #8670: URL: https://github.com/apache/cloudstack/pull/8670 ### Description This PR enabled draining on the `AllLinesParser` so `Script.execute()` doesn't timeout if it has to read larger data. Right now commands with larger output and

Re: [I] MinIO Bucket created in CloudStack 4.19 - How to connect via external client? [cloudstack]

2024-02-16 Thread via GitHub
vishesh92 commented on issue #8651: URL: https://github.com/apache/cloudstack/issues/8651#issuecomment-1948302093 @nxsbi Regarding Problem 2, CloudStack only creates and manages buckets on the Object Storage. It doesn't have any idea about the contents of buckets as of now. The Bucket

Re: [PR] vpc: optimize createMonitorServiceCommand() execution [cloudstack]

2024-02-16 Thread via GitHub
blueorangutan commented on PR #8385: URL: https://github.com/apache/cloudstack/pull/8385#issuecomment-1948300054 [SF] Trillian test result (tid-9247) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 42810 seconds Marvin logs:

Re: [I] Unable to attach a raw format volume to an instance running on KVM [cloudstack]

2024-02-16 Thread via GitHub
kiranchavala commented on issue #8652: URL: https://github.com/apache/cloudstack/issues/8652#issuecomment-1948266869 thanks @slavkap Since KVM (qemu) supports attaching raw disk images I think Cloustack should also support it -- This is an automated message from the Apache

Re: [PR] CKS: add ConfigDrive to cloud-init datasource_list in systemvm template [cloudstack]

2024-02-16 Thread via GitHub
blueorangutan commented on PR #7650: URL: https://github.com/apache/cloudstack/pull/7650#issuecomment-1948255294 [SF] Trillian test result (tid-9243) Environment: xenserver-71 (x2), Advanced Networking with Mgmt server 7 Total time taken: 42800 seconds Marvin logs:

Re: [I] Wrong ldap.bind.password after 4.19 upgrade [cloudstack]

2024-02-16 Thread via GitHub
BryanMLima commented on issue #8637: URL: https://github.com/apache/cloudstack/issues/8637#issuecomment-1948248260 > This may be related to #6812 > > @BryanMLima can you please have a look ? I will have a look at this. -- This is an automated message from the Apache Git

Re: [PR] Allow deletion of system VM templates [cloudstack]

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

Re: [PR] Improve logs in primary storage removal process [cloudstack]

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

Re: [PR] server: apply rules when VR of Domain VPC is recreated [cloudstack]

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

Re: [PR] CKS: add ConfigDrive to cloud-init datasource_list in systemvm template [cloudstack]

2024-02-16 Thread via GitHub
blueorangutan commented on PR #7650: URL: https://github.com/apache/cloudstack/pull/7650#issuecomment-1948204332 [SF] Trillian test result (tid-9245) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 39837 seconds Marvin logs:

Re: [PR] Polish UI instance schedule modal [cloudstack]

2024-02-16 Thread via GitHub
blueorangutan commented on PR #8620: URL: https://github.com/apache/cloudstack/pull/8620#issuecomment-1948200841 UI build: :heavy_check_mark: Live QA URL: https://qa.cloudstack.cloud/simulator/pr/8620 (QA-JID-285) -- This is an automated message from the Apache Git Service. To respond

Re: [I] Trying to register a template with a long URL results in error [cloudstack]

2024-02-16 Thread via GitHub
boring-cyborg[bot] commented on issue #8669: URL: https://github.com/apache/cloudstack/issues/8669#issuecomment-1948193009 Thanks for opening your first issue here! Be sure to follow the issue template! -- This is an automated message from the Apache Git Service. To respond to the

[I] Trying to register a template with a long URL results in error [cloudstack]

2024-02-16 Thread via GitHub
fabiomatavelli-f3 opened a new issue, #8669: URL: https://github.com/apache/cloudstack/issues/8669 # ISSUE TYPE * Bug Report # COMPONENT NAME ~~~ UI, API ~~~ # CLOUDSTACK VERSION ~~~ 4.18.1.0 ~~~ #

Re: [PR] Polish UI instance schedule modal [cloudstack]

2024-02-16 Thread via GitHub
blueorangutan commented on PR #8620: URL: https://github.com/apache/cloudstack/pull/8620#issuecomment-1948178709 @DaanHoogland 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

Re: [PR] Refactor updateDiskOffering API [cloudstack]

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

Re: [PR] Polish UI instance schedule modal [cloudstack]

2024-02-16 Thread via GitHub
DaanHoogland commented on PR #8620: URL: https://github.com/apache/cloudstack/pull/8620#issuecomment-1948177413 @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 specific

Re: [I] Any option to edit guest instance xml configuration file permanently? [cloudstack]

2024-02-16 Thread via GitHub
DaanHoogland commented on issue #8664: URL: https://github.com/apache/cloudstack/issues/8664#issuecomment-1948174993 @naseemvk are there any parameters you want changing that you cannot? this can be interpretted as two different featture requests. One specific for some missing knobs to

Re: [PR] Don't allow host (KVM) addition without PowerFlex SDC installed/connected [cloudstack]

2024-02-16 Thread via GitHub
sureshanaparti commented on code in PR #8628: URL: https://github.com/apache/cloudstack/pull/8628#discussion_r1492290426 ## plugins/storage/volume/scaleio/src/main/java/org/apache/cloudstack/storage/datastore/provider/ScaleIOHostListener.java: ## @@ -70,12 +71,33 @@ public

Re: [PR] Don't allow host (KVM) addition without PowerFlex SDC installed/connected [cloudstack]

2024-02-16 Thread via GitHub
vishesh92 commented on code in PR #8628: URL: https://github.com/apache/cloudstack/pull/8628#discussion_r1492268094 ## plugins/storage/volume/scaleio/src/main/java/org/apache/cloudstack/storage/datastore/provider/ScaleIOHostListener.java: ## @@ -70,12 +71,33 @@ public boolean

Re: [PR] [WIP] System VMs/VRs upgrade to Python3 and Debian 12 [cloudstack]

2024-02-16 Thread via GitHub
blueorangutan commented on PR #8497: URL: https://github.com/apache/cloudstack/pull/8497#issuecomment-1948155622 [SF] Trillian test result (tid-9234) Environment: vmware-80u1 (x3), Advanced Networking with Mgmt server r8 Total time taken: 53544 seconds Marvin logs:

Re: [PR] Move views into separate files for easier modification [cloudstack]

2024-02-16 Thread via GitHub
blueorangutan commented on PR #8647: URL: https://github.com/apache/cloudstack/pull/8647#issuecomment-1948145067 @DaanHoogland a [SL] Trillian-Jenkins matrix job (centos7 mgmt + xenserver71, rocky8 mgmt + vmware67u3, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests -- This

Re: [PR] Move views into separate files for easier modification [cloudstack]

2024-02-16 Thread via GitHub
DaanHoogland commented on PR #8647: URL: https://github.com/apache/cloudstack/pull/8647#issuecomment-1948144430 @blueorangutan test matrix -- 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

Re: [PR] Update `vpc.max.networks` setting [cloudstack]

2024-02-16 Thread via GitHub
weizhouapache commented on PR #8654: URL: https://github.com/apache/cloudstack/pull/8654#issuecomment-1948142376 > > I can understand your code. Your requirement is not clear to me. Can you explain more about the infrastructure? e.g. the hypervisor types, why the max vpc networks are

Re: [PR] UI: fix permission of VNF appliances [cloudstack]

2024-02-16 Thread via GitHub
codecov[bot] commented on PR #8668: URL: https://github.com/apache/cloudstack/pull/8668#issuecomment-1948142792 ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/8668?src=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=apache) Report All

[PR] UI: fix permission of VNF appliances [cloudstack]

2024-02-16 Thread via GitHub
weizhouapache opened a new pull request, #8668: URL: https://github.com/apache/cloudstack/pull/8668 ### Description This PR fixes #8666 There is no API for listing VNF appliances, use `listVnfTemplates` as a workaround.

Re: [PR] Improve logs in primary storage removal process [cloudstack]

2024-02-16 Thread via GitHub
blueorangutan commented on PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#issuecomment-1948125356 @DaanHoogland 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

  1   2   >