You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by GitBox <gi...@apache.org> on 2021/01/15 16:46:38 UTC
[GitHub] [cloudstack] olivierlemasle opened a new pull request #4588: Enable SNI in SSVM
olivierlemasle opened a new pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588
### Description
This PR enables SNI support in the SSVM. This is required to download templates from web servers with HTTPS and SNI (e.g. behind Cloudfront).
<!--- Describe your changes in DETAIL - And how has behaviour functionally changed. -->
<!-- For new features, provide link to FS, dev ML discussion etc. -->
<!-- In case of bug fix, the expected and actual behaviours, steps to reproduce. -->
<!-- When "Fixes: #<id>" is specified, the issue/PR will automatically be closed when this PR gets merged -->
<!-- For addressing multiple issues/PRs, use multiple "Fixes: #<id>" -->
Fixes: #4587
### Types of changes
- [ ] Breaking change (fix or feature that would cause existing functionality to change)
- [ ] New feature (non-breaking change which adds functionality)
- [x] Bug fix (non-breaking change which fixes an issue)
- [ ] Enhancement (improves an existing feature and functionality)
- [ ] Cleanup (Code refactoring and cleanup, that may add test cases)
### Feature/Enhancement Scale or Bug Severity
#### Feature/Enhancement Scale
- [ ] Major
- [x] Minor
#### Bug Severity
- [ ] BLOCKER
- [ ] Critical
- [ ] Major
- [x] Minor
- [ ] Trivial
### Screenshots (if appropriate):
N/A
### How Has This Been Tested?
<!-- Please describe in detail how you tested your changes. -->
<!-- Include details of your testing environment, and the tests you ran to -->
<!-- see how your change affects other areas of the code, etc. -->
Manually tested (cf section "Temporary fix" in #4587)
<!-- Please read the [CONTRIBUTING](https://github.com/apache/cloudstack/blob/master/CONTRIBUTING.md) document -->
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] rhtyd commented on pull request #4588: Enable SNI in SSVM
Posted by GitBox <gi...@apache.org>.
rhtyd commented on pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588#issuecomment-766620268
@olivierlemasle should this apply for master/4.15 or do you want to change the base branch to 4.14?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] DaanHoogland commented on pull request #4588: Enable SNI in SSVM
Posted by GitBox <gi...@apache.org>.
DaanHoogland commented on pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588#issuecomment-771640819
> @olivierlemasle have you tested if copy template across works with the fix?
>
> Otherwise, it will require this manual test case - setup zone/env with SSL enable for systemvms, add another zone with cluster/hosts etc and try to use the copyTemplate API.
Good call, @rhtyd it fails:
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] olivierlemasle commented on pull request #4588: Enable SNI in SSVM
Posted by GitBox <gi...@apache.org>.
olivierlemasle commented on pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588#issuecomment-863482565
The issue #4587 is still relevant.
However, my fix did cause a regression. This went out of my radar, but I'll definitely investigate and look for a proper fix.
I think we can close this PR, and I'll open a new one if/when I find a proper fix.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4588: Enable SNI in SSVM
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588#issuecomment-764324220
@rhtyd a 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 respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] rhtyd commented on pull request #4588: Enable SNI in SSVM
Posted by GitBox <gi...@apache.org>.
rhtyd commented on pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588#issuecomment-762892055
I suppose we'll need to test copy template (one zone to another) with this change cc @shwstppr
@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 comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] rhtyd commented on pull request #4588: Enable SNI in SSVM
Posted by GitBox <gi...@apache.org>.
rhtyd commented on pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588#issuecomment-768877731
@olivierlemasle have you tested if copy template across works with the fix?
Otherwise, it will require this manual test case - setup zone/env with SSL enable for systemvms, add another zone with cluster/hosts etc and try to use the copyTemplate API.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4588: Enable SNI in SSVM
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588#issuecomment-771581948
@DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4588: Enable SNI in SSVM
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588#issuecomment-768303453
@rhtyd a 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 respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] rhtyd commented on pull request #4588: Enable SNI in SSVM
Posted by GitBox <gi...@apache.org>.
rhtyd commented on pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588#issuecomment-773155445
Moved to next milestone, needs several round of multi-zone testing.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] DaanHoogland edited a comment on pull request #4588: Enable SNI in SSVM
Posted by GitBox <gi...@apache.org>.
DaanHoogland edited a comment on pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588#issuecomment-814717936
No @rhtyd see my test results above. I'm :-1: on this for now, and don't know of the way forward.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4588: Enable SNI in SSVM
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588#issuecomment-764324220
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] rhtyd commented on pull request #4588: Enable SNI in SSVM
Posted by GitBox <gi...@apache.org>.
rhtyd commented on pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588#issuecomment-771541515
@shwstppr can you confirm manual testing? Thanks
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] rhtyd commented on pull request #4588: Enable SNI in SSVM
Posted by GitBox <gi...@apache.org>.
rhtyd commented on pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588#issuecomment-771541515
@shwstppr can you confirm manual testing? Thanks
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4588: Enable SNI in SSVM
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588#issuecomment-764672756
<b>Trillian test result (tid-3404)</b>
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 31851 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4588-t3404-kvm-centos7.zip
Smoke tests completed. 86 look OK, 0 have error(s)
Only failed tests results shown below:
Test | Result | Time (s) | Test File
--- | --- | --- | ---
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] olivierlemasle commented on pull request #4588: Enable SNI in SSVM
Posted by GitBox <gi...@apache.org>.
olivierlemasle commented on pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588#issuecomment-814746684
@rhtyd Sorry I did not have access to a multi-AZ CloudStack environment for the moment. @DaanHoogland's tests show that this `enableSNIExtension=false` was there on purpose, even if don't understand the reason and if it causes #4587...
I have to investigate further.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] rhtyd commented on pull request #4588: Enable SNI in SSVM
Posted by GitBox <gi...@apache.org>.
rhtyd commented on pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588#issuecomment-766620268
@olivierlemasle should this apply for master/4.15 or do you want to change the base branch to 4.14?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4588: Enable SNI in SSVM
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588#issuecomment-762892492
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] rhtyd commented on pull request #4588: Enable SNI in SSVM
Posted by GitBox <gi...@apache.org>.
rhtyd commented on pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588#issuecomment-814633045
@DaanHoogland @olivierlemasle are you working/testing this - should this be moved to 4.16?
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] olivierlemasle closed pull request #4588: Enable SNI in SSVM
Posted by GitBox <gi...@apache.org>.
olivierlemasle closed pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] rhtyd commented on pull request #4588: Enable SNI in SSVM
Posted by GitBox <gi...@apache.org>.
rhtyd commented on pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588#issuecomment-764322345
@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 comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] shwstppr commented on pull request #4588: Enable SNI in SSVM
Posted by GitBox <gi...@apache.org>.
shwstppr commented on pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588#issuecomment-768157374
@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 comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] DaanHoogland edited a comment on pull request #4588: Enable SNI in SSVM
Posted by GitBox <gi...@apache.org>.
DaanHoogland edited a comment on pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588#issuecomment-771645654
cloudmonkey call:
```
(localcloud) SBCM5> > copy template id=a0a6f548-d14d-4f0c-bff5-4c3f3633d155 sourcezoneid=1f739580-43f3-4d93-828a-39f6c4d8847a destzoneid=e9a804d5-2555-4618-b6ed-d34683f6b15c
{
"template": {
"account": "admin",
"bits": 0,
"checksum": "7824c3610f07a79b46ce83e5f34d49caddb3740746b74e37d26ff69ef33635b192e2511d88c902bb40981eb6ae3a211489b24c913232304be9dcd4f56454db27",
"childtemplates": [],
"created": "2021-02-02T13:44:17+0000",
"crossZones": false,
"details": {},
"directdownload": false,
"displaytext": "mini",
"domain": "ROOT",
"domainid": "27255132-6554-11eb-840f-1e0048013b28",
"format": "OVA",
"hypervisor": "VMware",
"id": "a0a6f548-d14d-4f0c-bff5-4c3f3633d155",
"isdynamicallyscalable": true,
"isextractable": true,
"isfeatured": true,
"ispublic": true,
"isready": false,
"name": "mini",
"ostypeid": "36f30ac6-6554-11eb-840f-1e0048013b28",
"ostypename": "Linux 4.x Kernel (64-bit)",
"passwordenabled": true,
"requireshvm": true,
"sshkeyenabled": false,
"status": "PKIX path validation failed: java.security.cert.CertPathValidatorException: validity check failed",
"tags": [],
"templatetype": "USER",
"url": "http://dl.openvm.eu/cloudstack/macchinina/x86_64/macchinina-vmware.ova",
"zoneid": "e9a804d5-2555-4618-b6ed-d34683f6b15c",
"zonename": "ref-trl-2282-k-M7-daan-hoogland"
}
}
```
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] DaanHoogland edited a comment on pull request #4588: Enable SNI in SSVM
Posted by GitBox <gi...@apache.org>.
DaanHoogland edited a comment on pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588#issuecomment-771645654
cloudmonkey call:
```
(localcloud) SBCM5> > copy template id=a0a6f548-d14d-4f0c-bff5-4c3f3633d155 sourcezoneid=1f739580-43f3-4d93-828a-39f6c4d8847a destzoneid=e9a804d5-2555-4618-b6ed-d34683f6b15c
{
"template": {
"account": "admin",
"bits": 0,
"checksum": "7824c3610f07a79b46ce83e5f34d49caddb3740746b74e37d26ff69ef33635b192e2511d88c902bb40981eb6ae3a211489b24c913232304be9dcd4f56454db27",
"childtemplates": [],
"created": "2021-02-02T13:44:17+0000",
"crossZones": false,
"details": {},
"directdownload": false,
"displaytext": "mini",
"domain": "ROOT",
"domainid": "27255132-6554-11eb-840f-1e0048013b28",
"format": "OVA",
"hypervisor": "VMware",
"id": "a0a6f548-d14d-4f0c-bff5-4c3f3633d155",
"isdynamicallyscalable": true,
"isextractable": true,
"isfeatured": true,
"ispublic": true,
"isready": false,
"name": "mini",
"ostypeid": "36f30ac6-6554-11eb-840f-1e0048013b28",
"ostypename": "Linux 4.x Kernel (64-bit)",
"passwordenabled": true,
"requireshvm": true,
"sshkeyenabled": false,
"status": "PKIX path validation failed: java.security.cert.CertPathValidatorException: validity check failed",
"tags": [],
"templatetype": "USER",
"url": "http://dl.openvm.eu/cloudstack/macchinina/x86_64/macchinina-vmware.ova",
"zoneid": "e9a804d5-2555-4618-b6ed-d34683f6b15c",
"zonename": "ref-trl-2282-k-M7-daan-hoogland"
}
}
```
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] DaanHoogland commented on pull request #4588: Enable SNI in SSVM
Posted by GitBox <gi...@apache.org>.
DaanHoogland commented on pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588#issuecomment-771645654
cloudmonkey call does work however:
```
(localcloud) SBCM5> > copy template id=a0a6f548-d14d-4f0c-bff5-4c3f3633d155 sourcezoneid=1f739580-43f3-4d93-828a-39f6c4d8847a destzoneid=e9a804d5-2555-4618-b6ed-d34683f6b15c
{
"template": {
"account": "admin",
"bits": 0,
"checksum": "7824c3610f07a79b46ce83e5f34d49caddb3740746b74e37d26ff69ef33635b192e2511d88c902bb40981eb6ae3a211489b24c913232304be9dcd4f56454db27",
"childtemplates": [],
"created": "2021-02-02T13:44:17+0000",
"crossZones": false,
"details": {},
"directdownload": false,
"displaytext": "mini",
"domain": "ROOT",
"domainid": "27255132-6554-11eb-840f-1e0048013b28",
"format": "OVA",
"hypervisor": "VMware",
"id": "a0a6f548-d14d-4f0c-bff5-4c3f3633d155",
"isdynamicallyscalable": true,
"isextractable": true,
"isfeatured": true,
"ispublic": true,
"isready": false,
"name": "mini",
"ostypeid": "36f30ac6-6554-11eb-840f-1e0048013b28",
"ostypename": "Linux 4.x Kernel (64-bit)",
"passwordenabled": true,
"requireshvm": true,
"sshkeyenabled": false,
"status": "PKIX path validation failed: java.security.cert.CertPathValidatorException: validity check failed",
"tags": [],
"templatetype": "USER",
"url": "http://dl.openvm.eu/cloudstack/macchinina/x86_64/macchinina-vmware.ova",
"zoneid": "e9a804d5-2555-4618-b6ed-d34683f6b15c",
"zonename": "ref-trl-2282-k-M7-daan-hoogland"
}
}
```
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4588: Enable SNI in SSVM
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588#issuecomment-762915304
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2550
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] DaanHoogland commented on pull request #4588: Enable SNI in SSVM
Posted by GitBox <gi...@apache.org>.
DaanHoogland commented on pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588#issuecomment-771581514
@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 comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] rhtyd commented on pull request #4588: Enable SNI in SSVM
Posted by GitBox <gi...@apache.org>.
rhtyd commented on pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588#issuecomment-768302845
@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 comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] olivierlemasle commented on pull request #4588: Enable SNI in SSVM
Posted by GitBox <gi...@apache.org>.
olivierlemasle commented on pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588#issuecomment-767605047
Hi @rhtyd, I'm ok to change the base branch to 4.14 and include it in 4.14.x.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4588: Enable SNI in SSVM
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588#issuecomment-771581948
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4588: Enable SNI in SSVM
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588#issuecomment-768190153
Packaging result: ✔centos7 ✖centos8 ✔debian. JID-2597
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] rhtyd commented on pull request #4588: Enable SNI in SSVM
Posted by GitBox <gi...@apache.org>.
rhtyd commented on pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588#issuecomment-764322345
@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 comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] DaanHoogland commented on pull request #4588: Enable SNI in SSVM
Posted by GitBox <gi...@apache.org>.
DaanHoogland commented on pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588#issuecomment-814717936
No @rhtyd see my test results above. I'm :-1: on this for now, and don't of the way forward.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4588: Enable SNI in SSVM
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588#issuecomment-768157892
@shwstppr a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] rhtyd commented on pull request #4588: Enable SNI in SSVM
Posted by GitBox <gi...@apache.org>.
rhtyd commented on pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588#issuecomment-773155445
Moved to next milestone, needs several round of multi-zone testing.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4588: Enable SNI in SSVM
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588#issuecomment-768678563
<b>Trillian test result (tid-3438)</b>
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 37500 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4588-t3438-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_kubernetes_clusters.py
Smoke tests completed. 83 look OK, 0 have error(s)
Only failed tests results shown below:
Test | Result | Time (s) | Test File
--- | --- | --- | ---
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] DaanHoogland commented on pull request #4588: Enable SNI in SSVM
Posted by GitBox <gi...@apache.org>.
DaanHoogland commented on pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588#issuecomment-771581514
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] DaanHoogland commented on pull request #4588: Enable SNI in SSVM
Posted by GitBox <gi...@apache.org>.
DaanHoogland commented on pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588#issuecomment-771647537
@olivierlemasle please see above. I think the disabling was with good reason.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4588: Enable SNI in SSVM
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4588:
URL: https://github.com/apache/cloudstack/pull/4588#issuecomment-771600100
Packaging result: ✔centos7 ✖centos8 ✔debian. JID-2635
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org