Bug 1904669
| Summary: | oVirt 4.3 -> 4.4 production upgrade: OpenStack Block Storage Provider (Cinder) regression | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [oVirt] ovirt-engine | Reporter: | Konstantin Shalygin <shalygin.k> | ||||
| Component: | BLL.Storage | Assignee: | Tal Nisan <tnisan> | ||||
| Status: | CLOSED WONTFIX | QA Contact: | Avihai <aefrat> | ||||
| Severity: | urgent | Docs Contact: | |||||
| Priority: | unspecified | ||||||
| Version: | 4.4.3.12 | CC: | aoconnor, bugs, dholler, eshenitz, logank, michal.skrivanek, sbonazzo | ||||
| Target Milestone: | --- | Flags: | aoconnor:
blocker-
|
||||
| Target Release: | --- | ||||||
| Hardware: | x86_64 | ||||||
| OS: | Linux | ||||||
| Whiteboard: | |||||||
| Fixed In Version: | Doc Type: | If docs needed, set a value | |||||
| Doc Text: | Story Points: | --- | |||||
| Clone Of: | Environment: | ||||||
| Last Closed: | 2020-12-21 15:28:08 UTC | Type: | Bug | ||||
| Regression: | --- | Mount Type: | --- | ||||
| Documentation: | --- | CRM: | |||||
| Verified Versions: | Category: | --- | |||||
| oVirt Team: | Storage | RHEL 7.3 requirements from Atomic Host: | |||||
| Cloudforms Team: | --- | Target Upstream Version: | |||||
| Embargoed: | |||||||
| Attachments: |
|
||||||
|
Description
Konstantin Shalygin
2020-12-05 12:43:58 UTC
Did you check cinderlib integration for use with ceph (https://www.ovirt.org/develop/release-management/features/storage/cinderlib-integration.html)? It's going to replace the aging cinder integration which doesn't work with recent Openstack versions The documentation text flag should only be set after 'doc text' field is provided. Please provide the documentation text and set the flag to '?' again. Michal, yes I know about cinderlib. Exactly for this feature was I start EL8 + oVirt 4.4 upgrade. We (oVirt admin's with large Cinder integrations) already mentioned almost two years ago, about "we need migration guide" before remove old external provider for OpenStack [1]. [1] https://bugzilla.redhat.com/show_bug.cgi?id=1539837#c6 1. I found 1797528 [1] where disk can't be deleted from Managed Storage Block pool. 2. AFAIU current Managed Storage Block realization support only krbd (kernel rbd) driver - it's also not a option, because kernel client is always lagging behind librbd, and every update\bugfix we should reboot whole host instead simple migration of all vms and then migrate it back. 3. Also with krbd host will be use kernel page cache, and will not be unmounted if VM will crash (qemu with librbd is one userland process). So I see only regressions for now, hope we'll found some code owner who can catch this oVirt 4.4 only bugs. [1] https://bugzilla.redhat.com/show_bug.cgi?id=1797528 We still need patch for this blocker. Thanks I was found a bunch of patches in BZ#1704349, where tenantId just dropped from the code and tests was perfromed without auth... ``` - OpenStackRequest<VolumeTypes> listRequest = getClient(getTenantId()).volumeTypes().list(); + OpenStackRequest<VolumeTypes> listRequest = getClient().volumeTypes().list(); ``` This exact our problem, I CC'ed Dominik Holler... (In reply to Konstantin Shalygin from comment #6) > I was found a bunch of patches in BZ#1704349, where tenantId just dropped > from the code and tests was perfromed without auth... > > ``` > - OpenStackRequest<VolumeTypes> listRequest = > getClient(getTenantId()).volumeTypes().list(); > + OpenStackRequest<VolumeTypes> listRequest = > getClient().volumeTypes().list(); > ``` > > This exact our problem, I CC'ed Dominik Holler... Konstantin, thanks for your analysis, I just can acknowledge that the change provider: Support OpenStack Identity API v3 introduced the regression of missing the {project_id} during the construction of the url of the OpenStack Block Storage API. (In reply to Dominik Holler from comment #7) > Konstantin, thanks for your analysis, I just can acknowledge that the change > provider: Support OpenStack Identity API v3 > introduced the regression of missing the {project_id} during the > construction of the url of the OpenStack Block Storage API. Thank you Dominik, glad to hear that the regression acknowledged. Any news for this? (In reply to Konstantin Shalygin from comment #9) > Any news for this? The "old" way of using Cinder as an external provider is now deprecated. Please see the new Cinder integration using cinderlib AKA as Managed Block Storage. For more info, you can see - https://www.ovirt.org/develop/release-management/features/storage/cinderlib-integration.html (In reply to Eyal Shenitzky from comment #10) > (In reply to Konstantin Shalygin from comment #9) > > Any news for this? > > The "old" way of using Cinder as an external provider is now deprecated. Please open a proper deprecation bug and get it documented as deprecated feature. > > Please see the new Cinder integration using cinderlib AKA as Managed Block > Storage. > > For more info, you can see - > https://www.ovirt.org/develop/release-management/features/storage/cinderlib- > integration.html This feature page is obsolete and not a valid user documentation. I'm working with doc team to get it properly added to oVirt Installation Guide. (In reply to Sandro Bonazzola from comment #11) > (In reply to Eyal Shenitzky from comment #10) > > (In reply to Konstantin Shalygin from comment #9) > > > Any news for this? > > > > The "old" way of using Cinder as an external provider is now deprecated. > > Please open a proper deprecation bug and get it documented as deprecated > feature. Already opened by Tal - bug 1899453. |