Re: [PR] ObjectStore Framework with MinIO and Simulator plugins [cloudstack]

2023-11-15 Thread via GitHub
blueorangutan commented on PR #7752: URL: https://github.com/apache/cloudstack/pull/7752#issuecomment-1813949126 Packaging result [SF]: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 7767 -- This is an auto

Re: [PR] [Draft] KVM Ingestion - Import Instance [cloudstack]

2023-11-15 Thread via GitHub
blueorangutan commented on PR #7976: URL: https://github.com/apache/cloudstack/pull/7976#issuecomment-1813949123 Packaging result [SF]: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 7768 -- This is an auto

Re: [I] In an SG zone a VM can't be connected to an L2 network only [cloudstack]

2023-11-15 Thread via GitHub
weizhouapache closed issue #8217: In an SG zone a VM can't be connected to an L2 network only URL: https://github.com/apache/cloudstack/issues/8217 -- 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 th

Re: [I] In an SG zone a VM can't be connected to an L2 network only [cloudstack]

2023-11-15 Thread via GitHub
weizhouapache commented on issue #8217: URL: https://github.com/apache/cloudstack/issues/8217#issuecomment-1813944284 fixed by #8122 -- 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

Re: [PR] [Draft] KVM Ingestion - Import Instance [cloudstack]

2023-11-15 Thread via GitHub
blueorangutan commented on PR #7976: URL: https://github.com/apache/cloudstack/pull/7976#issuecomment-1813885515 @shwstppr 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] [Draft] KVM Ingestion - Import Instance [cloudstack]

2023-11-15 Thread via GitHub
shwstppr commented on PR #7976: URL: https://github.com/apache/cloudstack/pull/7976#issuecomment-1813883835 @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 com

Re: [PR] ObjectStore Framework with MinIO and Simulator plugins [cloudstack]

2023-11-15 Thread via GitHub
blueorangutan commented on PR #7752: URL: https://github.com/apache/cloudstack/pull/7752#issuecomment-1813883629 @shwstppr 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] ObjectStore Framework with MinIO and Simulator plugins [cloudstack]

2023-11-15 Thread via GitHub
shwstppr commented on PR #7752: URL: https://github.com/apache/cloudstack/pull/7752#issuecomment-1813883411 @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 com

Re: [PR] Adding a default setter for domainPath in ControlledEntityResponse and populated the same in createNetworkResponse method of ApiResponseHelper #8086 [cloudstack]

2023-11-15 Thread via GitHub
shwstppr commented on code in PR #8207: URL: https://github.com/apache/cloudstack/pull/8207#discussion_r1395174935 ## server/src/main/java/com/cloud/api/ApiResponseHelper.java: ## @@ -2585,10 +2585,7 @@ public NetworkResponse createNetworkResponse(ResponseView view, Network net

Re: [PR] Externalize KVM HA heartbeat frequency [cloudstack]

2023-11-15 Thread via GitHub
blueorangutan commented on PR #6892: URL: https://github.com/apache/cloudstack/pull/6892#issuecomment-181356 [SF] Trillian test result (tid-8329) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 46510 seconds Marvin logs: https://github.com

Re: [PR] Adding a default setter for domainPath in ControlledEntityResponse and populated the same in createNetworkResponse method of ApiResponseHelper #8086 [cloudstack]

2023-11-15 Thread via GitHub
blueorangutan commented on PR #8207: URL: https://github.com/apache/cloudstack/pull/8207#issuecomment-1813708897 [SF] Trillian test result (tid-8328) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 46033 seconds Marvin logs: https://github.com

Re: [PR] Accept a role ID on linking an account to LDAP [cloudstack]

2023-11-15 Thread via GitHub
blueorangutan commented on PR #8236: URL: https://github.com/apache/cloudstack/pull/8236#issuecomment-1813608315 [SF] Trillian test result (tid-8330) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 34913 seconds Marvin logs: https://github.com

Re: [PR] [HEALTH] 4.19/main Health Check, please don't merge this! [cloudstack]

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

Re: [PR] Fixed Hashmap Key value comparision [cloudstack]

2023-11-15 Thread via GitHub
kavvya97 commented on PR #8238: URL: https://github.com/apache/cloudstack/pull/8238#issuecomment-1813346680 > I do not know anyone has faced the test failures, at least I never did. > > anyway, using LinkedHashMap is a more simple fix. I utilized a helper function 'containsPair'

Re: [PR] Fixed Hashmap Key value comparision [cloudstack]

2023-11-15 Thread via GitHub
weizhouapache commented on PR #8238: URL: https://github.com/apache/cloudstack/pull/8238#issuecomment-1813289652 I do not know anyone has faced the test failures, at least I never did. anyway, using LinkedHashMap is a more simple fix. -- This is an automated message from the Apac

Re: [PR] [HEALTH] 4.18 LTS Health Check, please don't merge this! [cloudstack]

2023-11-15 Thread via GitHub
blueorangutan commented on PR #7345: URL: https://github.com/apache/cloudstack/pull/7345#issuecomment-1813236389 [SF] Trillian test result (tid-8325) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 43361 seconds Marvin logs: https://github.com

Re: [I] ACS 4.18 diskofferingid missing from deployVirtualMachine response [cloudstack]

2023-11-15 Thread via GitHub
weizhouapache commented on issue #8120: URL: https://github.com/apache/cloudstack/issues/8120#issuecomment-1813181321 > Come to think of it; this will have performance repercussions when listing a lot of VMs. We might want to make it optional or add a flag or setting tho enable/disable the

Re: [PR] [HEALTH] 4.18 LTS Health Check, please don't merge this! [cloudstack]

2023-11-15 Thread via GitHub
blueorangutan commented on PR #7345: URL: https://github.com/apache/cloudstack/pull/7345#issuecomment-1813163265 [SF] Trillian test result (tid-8323) Environment: xenserver-71 (x2), Advanced Networking with Mgmt server 7 Total time taken: 40067 seconds Marvin logs: https://github.co

Re: [PR] Fixed Hashmap Key value comparision [cloudstack]

2023-11-15 Thread via GitHub
boring-cyborg[bot] commented on PR #8238: URL: https://github.com/apache/cloudstack/pull/8238#issuecomment-1813037571 Congratulations on your first Pull Request and welcome to the Apache CloudStack community! If you have any issues or are unsure about any anything please check our Contribut

[PR] Fixed Hashmap Key value comparision [cloudstack]

2023-11-15 Thread via GitHub
kavvya97 opened a new pull request, #8238: URL: https://github.com/apache/cloudstack/pull/8238 ### Description The PR fixes the following tests which could produce non-deterministic/flaky pass/fail result `com.cloud.network.as.AutoScaleVmProfileVOTest#tstSetOtherDeployParamsForUpdate`

Re: [PR] NSX: Add CKS Support & Firewall rules for Isolated Networks [cloudstack]

2023-11-15 Thread via GitHub
blueorangutan commented on PR #8189: URL: https://github.com/apache/cloudstack/pull/8189#issuecomment-1813031915 Packaging result [SF]: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 7764 -- This is an auto

Re: [PR] NSX: Add CKS Support & Firewall rules for Isolated Networks [cloudstack]

2023-11-15 Thread via GitHub
blueorangutan commented on PR #8189: URL: https://github.com/apache/cloudstack/pull/8189#issuecomment-1812947042 @Pearl1594 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] NSX: Add CKS Support & Firewall rules for Isolated Networks [cloudstack]

2023-11-15 Thread via GitHub
Pearl1594 commented on PR #8189: URL: https://github.com/apache/cloudstack/pull/8189#issuecomment-1812916975 @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

Re: [PR] Normalize loggers and upgrade log4j 1.2 to log4j 2.19 [cloudstack]

2023-11-15 Thread via GitHub
weizhouapache commented on PR #7131: URL: https://github.com/apache/cloudstack/pull/7131#issuecomment-1812845318 > > > moving this to 4.20 > > > > @DaanHoogland aren't we going to merge this ? > > I thought we agreed to move it to 4.20 (or 4.19.1 which makes more sense to me) S

Re: [PR] Rollback of changes with errors during the VM assign [cloudstack]

2023-11-15 Thread via GitHub
GaOrtiga commented on PR #7061: URL: https://github.com/apache/cloudstack/pull/7061#issuecomment-1812802095 Hi guys, sorry for the delay. > > @stephankruggg is anybody looking at this (or should we postpone)? > > @GaOrtiga can you take at this one, please? Yes.

Re: [PR] Set number of hours per month with actual number in Quota [cloudstack]

2023-11-15 Thread via GitHub
blueorangutan commented on PR #7210: URL: https://github.com/apache/cloudstack/pull/7210#issuecomment-1812793988 Packaging result [SF]: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 7763 -- This is an auto

Re: [PR] Rollback of changes with errors during the VM assign [cloudstack]

2023-11-15 Thread via GitHub
stephankruggg commented on PR #7061: URL: https://github.com/apache/cloudstack/pull/7061#issuecomment-1812783901 > @stephankruggg is anybody looking at this (or should we postpone)? @GaOrtiga can you take at this one, please? -- This is an automated message from the Apache Git Servi

[I] Ability to change users Project role without deleting and re-adding user [cloudstack]

2023-11-15 Thread via GitHub
damonb123 opened a new issue, #8237: URL: https://github.com/apache/cloudstack/issues/8237 # ISSUE TYPE * Improvement Request # COMPONENT NAME ~~~ UI, API ~~~ # CLOUDSTACK VERSION ~~~ 4.18.1.0 ~~~ # SUM

Re: [PR] Accept a role ID on linking an account to LDAP [cloudstack]

2023-11-15 Thread via GitHub
blueorangutan commented on PR #8236: URL: https://github.com/apache/cloudstack/pull/8236#issuecomment-1812751193 @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 r

Re: [PR] Accept a role ID on linking an account to LDAP [cloudstack]

2023-11-15 Thread via GitHub
DaanHoogland commented on PR #8236: URL: https://github.com/apache/cloudstack/pull/8236#issuecomment-1812747604 @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

Re: [PR] Rollback of changes with errors during the VM assign [cloudstack]

2023-11-15 Thread via GitHub
DaanHoogland commented on PR #7061: URL: https://github.com/apache/cloudstack/pull/7061#issuecomment-1812730086 @stephankruggg is anybody looking at this (or should we postpone)? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

Re: [PR] Externalize KVM HA heartbeat frequency [cloudstack]

2023-11-15 Thread via GitHub
blueorangutan commented on PR #6892: URL: https://github.com/apache/cloudstack/pull/6892#issuecomment-1812729729 @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 r

Re: [PR] Externalize KVM HA heartbeat frequency [cloudstack]

2023-11-15 Thread via GitHub
DaanHoogland commented on PR #6892: URL: https://github.com/apache/cloudstack/pull/6892#issuecomment-1812726094 @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 co

Re: [PR] Database VIEW management via separated files [cloudstack]

2023-11-15 Thread via GitHub
DaanHoogland commented on PR #7417: URL: https://github.com/apache/cloudstack/pull/7417#issuecomment-1812724299 > @DaanHoogland I'm all for it based on the concept. I haven't had a chance to check all the codes. But from what I've seen so far, it looks promising. I can check the upgrade fro

[PR] Improved concatenation way in PropertiesStorage.java [cloudstack]

2023-11-15 Thread via GitHub
nikam14 opened a new pull request, #7486: URL: https://github.com/apache/cloudstack/pull/7486 ### Description This PR... changed the '+' operator for concatenation to .concat method for concatenation because '+' operator takes more time then .concat method. [Optimisation].

Re: [PR] Improved concatenation way in PropertiesStorage.java [cloudstack]

2023-11-15 Thread via GitHub
DaanHoogland closed pull request #7486: Improved concatenation way in PropertiesStorage.java URL: https://github.com/apache/cloudstack/pull/7486 -- 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

Re: [PR] enable to create VPC portfowarding rules with source cidr [cloudstack]

2023-11-15 Thread via GitHub
DaanHoogland commented on PR #7081: URL: https://github.com/apache/cloudstack/pull/7081#issuecomment-1812676241 ping @RodrigoDLopez -- 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

Re: [PR] Improved concatenation way in PropertiesStorage.java [cloudstack]

2023-11-15 Thread via GitHub
DaanHoogland commented on PR #7486: URL: https://github.com/apache/cloudstack/pull/7486#issuecomment-1812713332 formatting log messages only, should be good if compiles -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use t

Re: [PR] Improved concatenation way in PropertiesStorage.java [cloudstack]

2023-11-15 Thread via GitHub
DaanHoogland commented on code in PR #7486: URL: https://github.com/apache/cloudstack/pull/7486#discussion_r1394339714 ## agent/src/main/java/com/cloud/agent/dao/impl/PropertiesStorage.java: ## @@ -92,11 +92,14 @@ public synchronized boolean configure(String name, Map params) {

Re: [PR] [Veeam] Fix restore of reimported VMs [cloudstack]

2023-11-15 Thread via GitHub
DaanHoogland commented on PR #7239: URL: https://github.com/apache/cloudstack/pull/7239#issuecomment-1812689393 ping @SadiJr -- 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.

Re: [PR] Set number of hours per month with actual number in Quota [cloudstack]

2023-11-15 Thread via GitHub
blueorangutan commented on PR #7210: URL: https://github.com/apache/cloudstack/pull/7210#issuecomment-1812682795 @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 progress

Re: [PR] [VMware] apply IOPS in resize/migrate [cloudstack]

2023-11-15 Thread via GitHub
DaanHoogland commented on PR #7226: URL: https://github.com/apache/cloudstack/pull/7226#issuecomment-1812687041 ping @SadiJr -- 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.

Re: [PR] Adding Count Resource Domain [cloudstack]

2023-11-15 Thread via GitHub
DaanHoogland commented on PR #7274: URL: https://github.com/apache/cloudstack/pull/7274#issuecomment-1812694514 ping @CharlesQueiroz -- 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] Set number of hours per month with actual number in Quota [cloudstack]

2023-11-15 Thread via GitHub
DaanHoogland commented on PR #7210: URL: https://github.com/apache/cloudstack/pull/7210#issuecomment-1812683323 ui build error is in codecov push, not in code. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

Re: [PR] Set number of hours per month with actual number in Quota [cloudstack]

2023-11-15 Thread via GitHub
DaanHoogland commented on PR #7210: URL: https://github.com/apache/cloudstack/pull/7210#issuecomment-1812681434 @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

Re: [PR] Externalize KVM HA heartbeat frequency [cloudstack]

2023-11-15 Thread via GitHub
blueorangutan commented on PR #6892: URL: https://github.com/apache/cloudstack/pull/6892#issuecomment-1812679961 Packaging result [SF]: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 7762 -- This is an auto

Re: [I] Doubt - Assigning resource pools in different VPC offerings. [cloudstack]

2023-11-15 Thread via GitHub
DaanHoogland commented on issue #8219: URL: https://github.com/apache/cloudstack/issues/8219#issuecomment-1812674921 > @IgaoWolf @DaanHoogland PR #6918 adds the support for network-scope settings. After it gets merged, we could also consider adding VPC-scope settings to allow overriding set

Re: [PR] Support network-scope settings and add setting to override vr service offering [cloudstack]

2023-11-15 Thread via GitHub
DaanHoogland commented on PR #6918: URL: https://github.com/apache/cloudstack/pull/6918#issuecomment-1812672426 @soreana @weizhouapache any update? -- 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] XCP-NG / XenServer live migration of unattached volumes [cloudstack]

2023-11-15 Thread via GitHub
DaanHoogland commented on PR #6833: URL: https://github.com/apache/cloudstack/pull/6833#issuecomment-1812669643 @dpassante are you still interested in this for 4.19? I'll move it to unplanned otherwise. -- This is an automated message from the Apache Git Service. To respond to the message

(cloudstack-cloudmonkey) branch main updated: README.md: add info about make dist

2023-11-15 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-cloudmonkey.git The following commit(s) were added to refs/heads/main by this push: new 6323a31 README.md: add info about ma

Re: [PR] [Veeam] disable jobs but keep backups [cloudstack]

2023-11-15 Thread via GitHub
DaanHoogland commented on PR #6589: URL: https://github.com/apache/cloudstack/pull/6589#issuecomment-1812665598 @SadiJr I moved this out of this milestone as it needs work. If you find time to get it into 4.19, feel free to move it back in. -- This is an automated message from the Apache

Re: [I] ACS 4.18 diskofferingid missing from deployVirtualMachine response [cloudstack]

2023-11-15 Thread via GitHub
DaanHoogland commented on issue #8120: URL: https://github.com/apache/cloudstack/issues/8120#issuecomment-1812641987 Come to think of it; this will have performance repercussions when listing a lot of VMs. We might want to make it optional or add a flag or setting tho enable/disable the beh

Re: [I] ACS 4.18 diskofferingid missing from deployVirtualMachine response [cloudstack]

2023-11-15 Thread via GitHub
DaanHoogland commented on issue #8120: URL: https://github.com/apache/cloudstack/issues/8120#issuecomment-1812632652 > But if one can deployVirtualMachine with multiple disks and/or NICs, I would expect tem to be listed in their order in the result. agreed -- This is an automated m

Re: [PR] Handle errors while scaling kubernetes cluster [cloudstack]

2023-11-15 Thread via GitHub
DaanHoogland commented on PR #8107: URL: https://github.com/apache/cloudstack/pull/8107#issuecomment-1812627316 @harikrishna-patnala any progress/prognosis? -- 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: [I] ACS 4.18 diskofferingid missing from deployVirtualMachine response [cloudstack]

2023-11-15 Thread via GitHub
MartinEmrich commented on issue #8120: URL: https://github.com/apache/cloudstack/issues/8120#issuecomment-1812573504 I guess I don't have a strong opinion on that. In the meantime, I changed my application to no longer rely on the diskoffering fields, so I'll take whatever happens. But if o

Re: [PR] FiberChannel Multipath for KVM + Pure Flash Array and HPE-Primera Support [cloudstack]

2023-11-15 Thread via GitHub
DaanHoogland commented on PR #7889: URL: https://github.com/apache/cloudstack/pull/7889#issuecomment-1812565078 @rg9975 please see https://github.com/apache/cloudstack/actions/runs/6855902006/job/18655439266?pr=7889#step:7:16636 -- This is an automated message from the Apache Git Service.

Re: [PR] Adding a default setter for domainPath in ControlledEntityResponse and populated the same in createNetworkResponse method of ApiResponseHelper #8086 [cloudstack]

2023-11-15 Thread via GitHub
blueorangutan commented on PR #8207: URL: https://github.com/apache/cloudstack/pull/8207#issuecomment-1812555101 @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 r

Re: [PR] Adding a default setter for domainPath in ControlledEntityResponse and populated the same in createNetworkResponse method of ApiResponseHelper #8086 [cloudstack]

2023-11-15 Thread via GitHub
DaanHoogland commented on PR #8207: URL: https://github.com/apache/cloudstack/pull/8207#issuecomment-1812552069 @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

Re: [PR] Adding a default setter for domainPath in ControlledEntityResponse and populated the same in createNetworkResponse method of ApiResponseHelper #8086 [cloudstack]

2023-11-15 Thread via GitHub
blueorangutan commented on PR #8207: URL: https://github.com/apache/cloudstack/pull/8207#issuecomment-1812538592 Packaging result [SF]: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 7760 -- This is an auto

Re: [PR] Accept a role ID on linking an account to LDAP [cloudstack]

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

Re: [PR] Accept a role ID on linking an account to LDAP [cloudstack]

2023-11-15 Thread via GitHub
blueorangutan commented on PR #8236: URL: https://github.com/apache/cloudstack/pull/8236#issuecomment-1812520908 @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 progress

[PR] Accept a role ID on linking an account to LDAP [cloudstack]

2023-11-15 Thread via GitHub
DaanHoogland opened a new pull request, #8236: URL: https://github.com/apache/cloudstack/pull/8236 ### Description This PR implements #3677 Fixes: #3677 ### Types of changes - [ ] Breaking change (fix or feature that wou

Re: [PR] Accept a role ID on linking an account to LDAP [cloudstack]

2023-11-15 Thread via GitHub
DaanHoogland commented on PR #8236: URL: https://github.com/apache/cloudstack/pull/8236#issuecomment-1812512711 @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

Re: [PR] Adding a default setter for domainPath in ControlledEntityResponse and populated the same in createNetworkResponse method of ApiResponseHelper #8086 [cloudstack]

2023-11-15 Thread via GitHub
blueorangutan commented on PR #8207: URL: https://github.com/apache/cloudstack/pull/8207#issuecomment-1812451459 @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 progress

Re: [PR] Adding a default setter for domainPath in ControlledEntityResponse and populated the same in createNetworkResponse method of ApiResponseHelper #8086 [cloudstack]

2023-11-15 Thread via GitHub
DaanHoogland commented on PR #8207: URL: https://github.com/apache/cloudstack/pull/8207#issuecomment-1812451042 @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

Re: [PR] cleanup some smoke tests [cloudstack]

2023-11-15 Thread via GitHub
blueorangutan commented on PR #8196: URL: https://github.com/apache/cloudstack/pull/8196#issuecomment-1812389081 Packaging result [SF]: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 7759 -- This is an auto

Re: [PR] [HEALTH] 4.19/main Health Check, please don't merge this! [cloudstack]

2023-11-15 Thread via GitHub
sonarcloud[bot] commented on PR #7344: URL: https://github.com/apache/cloudstack/pull/7344#issuecomment-1812305962 Kudos, SonarCloud Quality Gate passed!    [![Quality Gate passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png 'Q

Re: [PR] [HEALTH] 4.18 LTS Health Check, please don't merge this! [cloudstack]

2023-11-15 Thread via GitHub
sonarcloud[bot] commented on PR #7345: URL: https://github.com/apache/cloudstack/pull/7345#issuecomment-1812281537 Kudos, SonarCloud Quality Gate passed!    [![Quality Gate passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png 'Q

Re: [PR] cleanup some smoke tests [cloudstack]

2023-11-15 Thread via GitHub
blueorangutan commented on PR #8196: URL: https://github.com/apache/cloudstack/pull/8196#issuecomment-1812241207 @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 progress

Re: [PR] cleanup some smoke tests [cloudstack]

2023-11-15 Thread via GitHub
DaanHoogland commented on PR #8196: URL: https://github.com/apache/cloudstack/pull/8196#issuecomment-1812236426 @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

(cloudstack-documentation) branch build-fix-202311 deleted (was 4a7f2c9)

2023-11-15 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a change to branch build-fix-202311 in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git was 4a7f2c9 build: use sphinx 1.8.6 instead of latest 8.2.6 The revisions that were on this br

(cloudstack-documentation) branch main updated: build: use sphinx 1.8.6 instead of latest 8.2.6 (#358)

2023-11-15 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 7942423 build: use sphinx 1.8.6 ins

(cloudstack-documentation) branch build-fix-202311 created (now 4a7f2c9)

2023-11-15 Thread weizhou
This is an automated email from the ASF dual-hosted git repository. weizhou pushed a change to branch build-fix-202311 in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git at 4a7f2c9 build: use sphinx 1.8.6 instead of latest 8.2.6 This branch includes the follow

(cloudstack-documentation) 01/01: build: use sphinx 1.8.6 instead of latest 8.2.6

2023-11-15 Thread weizhou
This is an automated email from the ASF dual-hosted git repository. weizhou pushed a commit to branch build-fix-202311 in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git commit 4a7f2c9031dd71e9d3c8b7c68825cabf2ca627db Author: Wei Zhou AuthorDate: Wed Nov 15 10:23:20 2

(cloudstack) branch dummy updated (429c5cc00c2 -> 210f7161de4)

2023-11-15 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a change to branch dummy in repository https://gitbox.apache.org/repos/asf/cloudstack.git omit 429c5cc00c2 do not merge add 1f29f6f0409 Public IP quarantine feature (#7378) add be4a648f5ab Create global

Re: [I] Prometheus Plugin not support chinese characters [cloudstack]

2023-11-15 Thread via GitHub
DaanHoogland closed issue #8203: Prometheus Plugin not support chinese characters URL: https://github.com/apache/cloudstack/issues/8203 -- 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

Re: [I] Prometheus Plugin not support chinese characters [cloudstack]

2023-11-15 Thread via GitHub
DaanHoogland commented on issue #8203: URL: https://github.com/apache/cloudstack/issues/8203#issuecomment-1812174089 fixed by #8228 -- 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

(cloudstack) branch main updated: Create global configuration to allow changing the default nic adapter for user VMs in VMware (#7954)

2023-11-15 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 be4a648f5ab Create global configuration to allow c

Re: [PR] Create global configuration to allow changing the default nic adapter for user VMs in VMware [cloudstack]

2023-11-15 Thread via GitHub
DaanHoogland merged PR #7954: URL: https://github.com/apache/cloudstack/pull/7954 -- 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

Re: [PR] Add dynamic secondary storage selection [cloudstack]

2023-11-15 Thread via GitHub
rajujith commented on PR #7659: URL: https://github.com/apache/cloudstack/pull/7659#issuecomment-1812133055 @BryanMLima any documentation available that provides some guidance on the usage of this feature? -- This is an automated message from the Apache Git Service. To respond to the me

Re: [PR] Global ACL for VPCs [cloudstack]

2023-11-15 Thread via GitHub
DaanHoogland commented on PR #7150: URL: https://github.com/apache/cloudstack/pull/7150#issuecomment-1812128578 @BryanMLima can you update this one? -- 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 t

Re: [PR] Add dynamic secondary storage selection [cloudstack]

2023-11-15 Thread via GitHub
DaanHoogland commented on PR #7659: URL: https://github.com/apache/cloudstack/pull/7659#issuecomment-1812127922 @BryanMLima , conflict :( can you resolve it? -- 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

Re: [PR] Adding a default setter for domainPath in ControlledEntityResponse and populated the same in createNetworkResponse method of ApiResponseHelper #8086 [cloudstack]

2023-11-15 Thread via GitHub
DaanHoogland commented on PR #8207: URL: https://github.com/apache/cloudstack/pull/8207#issuecomment-1812126955 @isupersky can you resolve the conflict? -- 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

Re: [PR] Adding a default setter for domainPath in ControlledEntityResponse and populated the same in createNetworkResponse method of ApiResponseHelper #8086 [cloudstack]

2023-11-15 Thread via GitHub
github-actions[bot] commented on PR #8207: URL: https://github.com/apache/cloudstack/pull/8207#issuecomment-1812111364 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] [Draft] KVM Ingestion - Import Instance [cloudstack]

2023-11-15 Thread via GitHub
github-actions[bot] commented on PR #7976: URL: https://github.com/apache/cloudstack/pull/7976#issuecomment-1812111326 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] New Feature: Import VMware VMs into KVM [cloudstack]

2023-11-15 Thread via GitHub
github-actions[bot] commented on PR #7881: URL: https://github.com/apache/cloudstack/pull/7881#issuecomment-1812111274 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] ObjectStore Framework with MinIO and Simulator plugins [cloudstack]

2023-11-15 Thread via GitHub
github-actions[bot] commented on PR #7752: URL: https://github.com/apache/cloudstack/pull/7752#issuecomment-1812111211 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] Add dynamic secondary storage selection [cloudstack]

2023-11-15 Thread via GitHub
github-actions[bot] commented on PR #7659: URL: https://github.com/apache/cloudstack/pull/7659#issuecomment-181268 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] Global ACL for VPCs [cloudstack]

2023-11-15 Thread via GitHub
github-actions[bot] commented on PR #7150: URL: https://github.com/apache/cloudstack/pull/7150#issuecomment-181214 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] Normalize loggers and upgrade log4j 1.2 to log4j 2.19 [cloudstack]

2023-11-15 Thread via GitHub
github-actions[bot] commented on PR #7131: URL: https://github.com/apache/cloudstack/pull/7131#issuecomment-1812111061 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] ObjectStore Framework with MinIO and Simulator plugins [cloudstack]

2023-11-15 Thread via GitHub
NuxRo commented on code in PR #7752: URL: https://github.com/apache/cloudstack/pull/7752#discussion_r1393925385 ## ui/public/locales/en.json: ## @@ -1406,6 +1412,16 @@ "label.oauth.configuration": "OAuth configuration", "label.oauth.verification": "OAuth verification", "label

Re: [PR] Flexible tags for hosts and storage pools [cloudstack]

2023-11-15 Thread via GitHub
blueorangutan commented on PR #7489: URL: https://github.com/apache/cloudstack/pull/7489#issuecomment-1812108997 [SF] Trillian test result (tid-8317) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 61921 seconds Marvin logs: https://github.com

Re: [PR] multi local storage handling for kvm [cloudstack]

2023-11-15 Thread via GitHub
DaanHoogland commented on PR #6699: URL: https://github.com/apache/cloudstack/pull/6699#issuecomment-1812100570 @JoaoJandre can you approve now? -- 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 th

Re: [PR] multi local storage handling for kvm [cloudstack]

2023-11-15 Thread via GitHub
blueorangutan commented on PR #6699: URL: https://github.com/apache/cloudstack/pull/6699#issuecomment-1812098121 @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 r

Re: [PR] multi local storage handling for kvm [cloudstack]

2023-11-15 Thread via GitHub
DaanHoogland commented on PR #6699: URL: https://github.com/apache/cloudstack/pull/6699#issuecomment-1812097781 @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 co

(cloudstack) branch main updated: Public IP quarantine feature (#7378)

2023-11-15 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 1f29f6f0409 Public IP quarantine feature (#7378) 1

Re: [I] Public IP quarantine feature [cloudstack]

2023-11-15 Thread via GitHub
DaanHoogland closed issue #7369: Public IP quarantine feature URL: https://github.com/apache/cloudstack/issues/7369 -- 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 unsubscri

Re: [PR] Public IP quarantine feature [cloudstack]

2023-11-15 Thread via GitHub
DaanHoogland merged PR #7378: URL: https://github.com/apache/cloudstack/pull/7378 -- 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

Re: [PR] Public IP quarantine feature [cloudstack]

2023-11-15 Thread via GitHub
DaanHoogland commented on PR #7378: URL: https://github.com/apache/cloudstack/pull/7378#issuecomment-1812093197 in a clean env: ``` Tests Enable HA when host is in Maintenance mode, should be Ineligible ... === TestName: test_hostha_enable_ha_when_host_in_maintenance | Status : SUCCESS

(cloudstack-documentation) branch minor-doc-update updated (8558c3b -> 87dfb81)

2023-11-15 Thread weizhou
This is an automated email from the ASF dual-hosted git repository. weizhou pushed a change to branch minor-doc-update in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git from 8558c3b build: use ubuntu 20.04 instead of ubuntu 22.04 new 9711a03 Revert "build:

  1   2   >