Re: [PR] Set CORS headers for both http/https in SSVM apache config [cloudstack]

DaanHoogland commented on PR #9389: URL: https://github.com/apache/cloudstack/pull/9389#issuecomment-2464921130 > @DaanHoogland the author @salfers mentioned `I have not tested this.`, are we going to merge it ? not like that of course (duh?) -- This is an automated message from th

Re: [PR] Set CORS headers for both http/https in SSVM apache config [cloudstack]

weizhouapache commented on PR #9389: URL: https://github.com/apache/cloudstack/pull/9389#issuecomment-2464828442 @DaanHoogland the author @salfers mentioned `I have not tested this.`, are we going to merge it ? -- This is an automated message from the Apache Git Service. To respond to

Re: [PR] Set CORS headers for both http/https in SSVM apache config [cloudstack]

DaanHoogland commented on PR #9389: URL: https://github.com/apache/cloudstack/pull/9389#issuecomment-2464697556 @weizhouapache , can you have a look at this? -- 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] Set CORS headers for both http/https in SSVM apache config [cloudstack]

DaanHoogland commented on PR #9389: URL: https://github.com/apache/cloudstack/pull/9389#issuecomment-2431935806 > @DaanHoogland any news? needs another review and some 3rd person testing, @salfers . -- This is an automated message from the Apache Git Service. To respond to the messa

Re: [PR] Set CORS headers for both http/https in SSVM apache config [cloudstack]

salfers commented on PR #9389: URL: https://github.com/apache/cloudstack/pull/9389#issuecomment-2413216257 @DaanHoogland any news? -- 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] Set CORS headers for both http/https in SSVM apache config [cloudstack]

blueorangutan commented on PR #9389: URL: https://github.com/apache/cloudstack/pull/9389#issuecomment-2287879609 [SF] Trillian test result (tid-11071) Environment: kvm-ol8 (x2), Advanced Networking with Mgmt server ol8 Total time taken: 50927 seconds Marvin logs: https://github.com/

Re: [PR] Set CORS headers for both http/https in SSVM apache config [cloudstack]

DaanHoogland commented on PR #9389: URL: https://github.com/apache/cloudstack/pull/9389#issuecomment-2286412308 @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] Set CORS headers for both http/https in SSVM apache config [cloudstack]

blueorangutan commented on PR #9389: URL: https://github.com/apache/cloudstack/pull/9389#issuecomment-2286413629 @DaanHoogland a [SL] Trillian-Jenkins test job (ol8 mgmt + kvm-ol8) has been kicked to run smoke tests -- This is an automated message from the Apache Git Service. To respond t

Re: [PR] Set CORS headers for both http/https in SSVM apache config [cloudstack]

blueorangutan commented on PR #9389: URL: https://github.com/apache/cloudstack/pull/9389#issuecomment-2286306915 Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 10635 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to G

Re: [PR] Set CORS headers for both http/https in SSVM apache config [cloudstack]

blueorangutan commented on PR #9389: URL: https://github.com/apache/cloudstack/pull/9389#issuecomment-2286174075 @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] Set CORS headers for both http/https in SSVM apache config [cloudstack]

DaanHoogland commented on PR #9389: URL: https://github.com/apache/cloudstack/pull/9389#issuecomment-2286168299 @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] Set CORS headers for both http/https in SSVM apache config [cloudstack]

blueorangutan commented on PR #9389: URL: https://github.com/apache/cloudstack/pull/9389#issuecomment-2235050944 [SF] Trillian test result (tid-10893) Environment: kvm-alma9 (x2), Advanced Networking with Mgmt server a9 Total time taken: 51272 seconds Marvin logs: https://github.com

Re: [PR] Set CORS headers for both http/https in SSVM apache config [cloudstack]

blueorangutan commented on PR #9389: URL: https://github.com/apache/cloudstack/pull/9389#issuecomment-2232976867 [SF] Trillian Build Failed (tid-10892) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to g

Re: [PR] Set CORS headers for both http/https in SSVM apache config [cloudstack]

blueorangutan commented on PR #9389: URL: https://github.com/apache/cloudstack/pull/9389#issuecomment-2232919595 @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 respo

Re: [PR] Set CORS headers for both http/https in SSVM apache config [cloudstack]

DaanHoogland commented on PR #9389: URL: https://github.com/apache/cloudstack/pull/9389#issuecomment-2232918711 @blueorangutan test alma9 kvm-alma9 -- 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] Set CORS headers for both http/https in SSVM apache config [cloudstack]

blueorangutan commented on PR #9389: URL: https://github.com/apache/cloudstack/pull/9389#issuecomment-2232800222 [SF] Trillian test result (tid-10887) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 3067 seconds Marvin logs: https://github.com

Re: [PR] Set CORS headers for both http/https in SSVM apache config [cloudstack]

blueorangutan commented on PR #9389: URL: https://github.com/apache/cloudstack/pull/9389#issuecomment-2232292242 [SF] Trillian test result (tid-10868) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 57030 seconds Marvin logs: https://github.co

Re: [PR] Set CORS headers for both http/https in SSVM apache config [cloudstack]

blueorangutan commented on PR #9389: URL: https://github.com/apache/cloudstack/pull/9389#issuecomment-2230619127 @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] Set CORS headers for both http/https in SSVM apache config [cloudstack]

DaanHoogland commented on PR #9389: URL: https://github.com/apache/cloudstack/pull/9389#issuecomment-2230617066 @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] Set CORS headers for both http/https in SSVM apache config [cloudstack]

blueorangutan commented on PR #9389: URL: https://github.com/apache/cloudstack/pull/9389#issuecomment-2230340442 Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 10380 -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [PR] Set CORS headers for both http/https in SSVM apache config [cloudstack]

blueorangutan commented on PR #9389: URL: https://github.com/apache/cloudstack/pull/9389#issuecomment-2230209840 @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] Set CORS headers for both http/https in SSVM apache config [cloudstack]

DaanHoogland commented on PR #9389: URL: https://github.com/apache/cloudstack/pull/9389#issuecomment-2230207639 @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] Set CORS headers for both http/https in SSVM apache config [cloudstack]

salfers commented on PR #9389: URL: https://github.com/apache/cloudstack/pull/9389#issuecomment-2230205660 I just noticed that removing `useHttpsToUpload` entirely could affect other code. I got rid of that now. -- This is an automated message from the Apache Git Service. To respond to th

Re: [PR] Set CORS headers for both http/https in SSVM apache config [cloudstack]

blueorangutan commented on PR #9389: URL: https://github.com/apache/cloudstack/pull/9389#issuecomment-2230202773 [SF] Trillian Build Failed (tid-10864) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to g

Re: [PR] Set CORS headers for both http/https in SSVM apache config [cloudstack]

blueorangutan commented on PR #9389: URL: https://github.com/apache/cloudstack/pull/9389#issuecomment-2230193799 @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 respo

Re: [PR] Set CORS headers for both http/https in SSVM apache config [cloudstack]

DaanHoogland commented on PR #9389: URL: https://github.com/apache/cloudstack/pull/9389#issuecomment-2230191349 @blueorangutan test alma9 kvm-alma9 -- 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] Set CORS headers for both http/https in SSVM apache config [cloudstack]

DaanHoogland commented on PR #9389: URL: https://github.com/apache/cloudstack/pull/9389#issuecomment-2230190903 > Smoke tests completed. 113 look OK, 24 have errors, 0 did not run seems strange to be a result of this code; trying again -- This is an automated message from the Apache

Re: [PR] Set CORS headers for both http/https in SSVM apache config [cloudstack]

blueorangutan commented on PR #9389: URL: https://github.com/apache/cloudstack/pull/9389#issuecomment-2229958527 [SF] Trillian test result (tid-10852) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 57851 seconds Marvin logs: https://github.co

Re: [PR] Set CORS headers for both http/https in SSVM apache config [cloudstack]

blueorangutan commented on PR #9389: URL: https://github.com/apache/cloudstack/pull/9389#issuecomment-2228267354 @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] Set CORS headers for both http/https in SSVM apache config [cloudstack]

DaanHoogland commented on PR #9389: URL: https://github.com/apache/cloudstack/pull/9389#issuecomment-2228264733 @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] Set CORS headers for both http/https in SSVM apache config [cloudstack]

blueorangutan commented on PR #9389: URL: https://github.com/apache/cloudstack/pull/9389#issuecomment-2228189096 Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 10365 -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [PR] Set CORS headers for both http/https in SSVM apache config [cloudstack]

blueorangutan commented on PR #9389: URL: https://github.com/apache/cloudstack/pull/9389#issuecomment-2228047134 @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] Set CORS headers for both http/https in SSVM apache config [cloudstack]

DaanHoogland commented on PR #9389: URL: https://github.com/apache/cloudstack/pull/9389#issuecomment-2228044295 @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] Set CORS headers for both http/https in SSVM apache config [cloudstack]

blueorangutan commented on PR #9389: URL: https://github.com/apache/cloudstack/pull/9389#issuecomment-2227989800 Packaging result [SF]: ✖️ el7 ✖️ el8 ✖️ el9 ✖️ debian ✖️ suse15. SL-JID 10363 -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [PR] Set CORS headers for both http/https in SSVM apache config [cloudstack]

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

Re: [PR] Set CORS headers for both http/https in SSVM apache config [cloudstack]

blueorangutan commented on PR #9389: URL: https://github.com/apache/cloudstack/pull/9389#issuecomment-2227921132 @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] Set CORS headers for both http/https in SSVM apache config [cloudstack]

DaanHoogland commented on PR #9389: URL: https://github.com/apache/cloudstack/pull/9389#issuecomment-2227918142 @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] Set CORS headers for both http/https in SSVM apache config [cloudstack]

boring-cyborg[bot] commented on PR #9389: URL: https://github.com/apache/cloudstack/pull/9389#issuecomment-2227851581 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] Set CORS headers for both http/https in SSVM apache config [cloudstack]

salfers opened a new pull request, #9389: URL: https://github.com/apache/cloudstack/pull/9389 ### Description This PR sets the necessary CORS headers on the SSVM independently of protocol. Fixes: #9366 ### Types of changes - [ ] Breaking change (fix or feature