Bug 1320112
| Summary: | Call to getImagesList on NFS on host without connected storage pool but with SD, return {'status': {'message': 'OK', 'code': 0}, 'imageslist': []} also when we have images | ||
|---|---|---|---|
| Product: | [oVirt] vdsm | Reporter: | Yaniv Lavi <ylavi> |
| Component: | General | Assignee: | Idan Shaby <ishaby> |
| Status: | CLOSED WORKSFORME | QA Contact: | Raz Tamir <ratamir> |
| Severity: | medium | Docs Contact: | |
| Priority: | medium | ||
| Version: | 4.17.23.1 | CC: | alukiano, amureini, bugs, istein, lzitnits, mavital, nsoffer, ratamir, sbonazzo, stirabos, tnisan, ylavi |
| Target Milestone: | ovirt-4.1.2 | Flags: | amureini:
ovirt-4.1?
rule-engine: planning_ack? rule-engine: devel_ack? rule-engine: testing_ack? |
| Target Release: | --- | ||
| Hardware: | x86_64 | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | Bug Fix | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | 1319721 | Environment: | |
| Last Closed: | 2017-03-06 13:39:23 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: | |||
|
Description
Yaniv Lavi
2016-03-22 10:34:41 UTC
This bug report has Keywords: Regression or TestBlocker. Since no regressions or test blockers are allowed between releases, it is also being identified as a blocker for this release. Please resolve ASAP. Moving from 4.0 alpha to 4.0 beta since 4.0 alpha has been already released and bug is not ON_QA. oVirt 4.0 beta has been released, moving to RC milestone. oVirt 4.0 beta has been released, moving to RC milestone. Raz, I see that the patches are merged, can you try and reproduce and let us know if it's still an open issue? (In reply to Tal Nisan from comment #5) > Raz, I see that the patches are merged, can you try and reproduce and let us > know if it's still an open issue? cli.getConnectedStoragePoolsList() returns: {'status': {'message': 'OK', 'code': 0}, 'poollist': []} cli.getStorageDomainsList() returns: {'status': {'message': 'OK', 'code': 0}, 'domlist': ['3f433c06-5057-4a13-afc7-8d70953e34b5']} and the cli.getImagesList('3f433c06-5057-4a13-afc7-8d70953e34b5') returns: {'status': {'message': 'OK', 'code': 0}, 'imageslist': ['e8b3eb91-3892-4e74-8962-b73405295fbe', '25c6e941-12ed-4ea7-a334-b71fed8ab0e7', 'a535c196-eeda-4f3f-8593-1380bd8a7cb1', '374c37d9-2387-48c6-86ae-4ef3d64c6ee8']} I'm not sure if that's the expected results, because I couldn't really understand why the expected results is 'list index out of range'. vdsm version: vdsm-4.18.999-1216.git34aa313.el7.centos.x86_64 Tal, Please reply to comment #6 the patches attached are workarounds to the storage function issue. We want a solution that HE will not need to workaround anymore. I don't understand the flow here. "Call to getImagesList on host without connected storage pool, but with SD" - what is a host without a storage pool, but with a storage domain? Is it a host in status = maintenance and with a storage domain that its status is up? Is it an activated host with a detached storage domain? Can you please explain in more details what's the flow here, who calls getImages and when? Thanks! I believe we talk about the scenario when we did HE deployment but still did not add the master storage domain to the engine. I do not sure if the bug still relevant for the 4.1, so maybe Simone can clarify the situation. (In reply to Artyom from comment #10) > I believe we talk about the scenario when we did HE deployment but still did > not add the master storage domain to the engine. > I do not sure if the bug still relevant for the 4.1, so maybe Simone can > clarify the situation. Yes, correct, still worth to check it. Moving out all non blocker\exceptions. We do not support anything when you are not connected to storage pool, except starting/stopping monitoring on external domains. This sounds like RFE for future version, and does not fit bug fix for 4.1. For 4.1 we should accept now only critical bug fixes in exiting features, not add features we do not have. Anything else is risking the stability of 4.1. I installed HE without adding a storage domain.
I ran "vdsm-client StorageDomain getImages storagedomainID=<uuid>" with the NFS domain id that I created during the installation, and I got 4 images:
[root@localhost data-center]# vdsm-client StorageDomain getImages storagedomainID=74683202-0d17-4992-86a0-4f865d59f4cd
[
"9dd2b426-87f0-44b7-b5f8-48f3ed3c266f",
"fa67d28c-44d8-4e3d-b435-26935ead0d61",
"80bfe663-7b94-49ee-8a63-42a25fb78272",
"ddebc6b7-57c8-4978-aa94-4a48b6c1faad"
]
These are config and regular images of hosted engine.
Closing as works for me, as it seems to be the right behavior.
|