Bug 1131800
Summary: | disk <decription> element is missing in xml | ||
---|---|---|---|
Product: | [Retired] oVirt | Reporter: | Gal Amado <gamado> |
Component: | ovirt-engine-api | Assignee: | Juan Hernández <juan.hernandez> |
Status: | CLOSED WONTFIX | QA Contact: | Pavel Stehlik <pstehlik> |
Severity: | unspecified | Docs Contact: | |
Priority: | unspecified | ||
Version: | 3.5 | CC: | acathrow, amureini, bugs, ecohen, gklein, iheim, juan.hernandez, mgoldboi, yeylon |
Target Milestone: | --- | Keywords: | Triaged |
Target Release: | 3.5.0 | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | infra | ||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2014-08-26 12:21:44 UTC | Type: | Bug |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | Infra | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Gal Amado
2014-08-20 05:49:14 UTC
This seems to be the general behavior of the system (e.g., VM's comment field), and AFAIK, it is by design. Moving to infra to decide how/if this should be handled. Juan - if I've missed anything, feel free to explain what I've missed and move it back. I'm not aware of any explicit design decision about this. We could, if considered appropriate, change this behavior. However, I think that a robust client should be prepared for both scenarios, and should treat missing and empty descriptions the same. This isn't true in general, as in some situations we may want to do different things for missing and empty elements, but for descriptions we should treat them the same. Anyhow, the RESTAPI doesn't have any logic around this, at least not for VMs: it just takes whatever the backend provides. If the backend gives null for the description the RESTAPI won't generate the <description> element. If the backend gives an empty string instead then the RESTAPI will generate the empty <description></description> element. Whatever we decide the logic should remain in the backend. |