Bug 1324087 - [RFE] Add the possibility in the API to add a macpool by name or by id
Summary: [RFE] Add the possibility in the API to add a macpool by name or by id
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Red Hat Enterprise Virtualization Manager
Classification: Red Hat
Component: ovirt-engine
Version: unspecified
Hardware: x86_64
OS: Linux
low
medium
Target Milestone: ---
: ---
Assignee: Nobody
QA Contact: meital avital
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-04-05 13:39 UTC by Martin Tessun
Modified: 2022-04-21 06:46 UTC (History)
10 users (show)

Fixed In Version:
Doc Type: Enhancement
Doc Text:
Clone Of:
Environment:
Last Closed: 2018-08-08 21:23:12 UTC
oVirt Team: Network
Target Upstream Version:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker RHV-45828 0 None None None 2022-04-21 06:46:05 UTC
oVirt gerrit 64444 0 master ABANDONED core,restapi: allow identification of mac pool by name. 2017-12-24 01:56:22 UTC
oVirt gerrit 76208 0 master ABANDONED core: added possibility to find mac pool by name 2018-01-28 01:56:18 UTC

Description Martin Tessun 2016-04-05 13:39:41 UTC
1. What is the nature and description of the request?  
      Extend the API for datacenters, so that it is possible to add the macpool by name as well.
      Currently it is only possible to add a macpool by id.

    2. Why does the customer need this? (List the business requirements here)  
      <Waiting for Philipp to add this>

    3. How would the customer like to achieve this? (List the functional requirements here)  
      Besides changing the macpool like
      <mac_pool id="id"/> add an option to also add it by name, e.g.
      <mac_pool>
        <name>My Name</name>
      </mac_pool>

    4. For each functional requirement listed, specify how Red Hat and the customer can test to confirm the requirement is successfully implemented.  
      Modify the macpool of a datacenter with the following PUT directive:
      <data_center>
          <mac_pool>
              <name>testname</name>
          </mac_pool>
      </data_center>

    5. Is there already an existing RFE upstream or in Red Hat Bugzilla?  
      No, not that I know

    6. Does the customer have any specific timeline dependencies and which release would they like to target (i.e. RHEL5, RHEL6)?  
      RHEV 3.6 lifecycle

    7. Is the sales team involved in this request and do they have any additional input?  
      No

    8. List any affected packages or components.  
      RHEV-M

    9. Would the customer be able to assist in testing this functionality if implemented?  
      yes

Comment 2 Martin Tessun 2016-04-05 13:41:40 UTC
Sorry, I missed to update (2):

    2. Why does the customer need this? (List the business requirements here)  

Simplify the comparison with user-readable user-defined input (configuration management). Coherent behavior of the API. In many cases in the API it is already possible to reference by name.
examples: Domain of a group, network of a host nic, datacenter of a network, datacenter of a cluster.

The API should not only accept the name but also output the name directly in the XML representation of the Datacenter.
<data_center>
....
      <mac_pool id="id">
        <name>My Name</name>
      </mac_pool>
</data_center>

Comment 3 Martin Mucha 2017-04-26 06:57:24 UTC
Why this was moved back to assigned? Code is posted and it's waiting for code reviewer to 'unpause' himself.

Comment 4 Dan Kenigsberg 2017-05-12 08:36:44 UTC
We've invested much effort relative to the importance of this RFE, but development drags on. It seems to be easily worked around (by looking the uuid of a macpool). Does the customer still care for it?

Comment 5 Philipp Gassmann 2017-05-17 15:26:09 UTC
What's missing? Code is committed and in review.

You should care too about a clean and simple API.

But the original customer does not really care for it, I implemented a workaround then.

Comment 6 Yaniv Kaul 2017-05-17 17:59:34 UTC
(In reply to Philipp Gassmann from comment #5)
> What's missing? Code is committed and in review.

Hi Philipp,
The code is not committed - just in review.


> 
> You should care too about a clean and simple API.

If you have comments about it - please raise them in Gerrit - we'd be happy to get comments.

> 
> But the original customer does not really care for it, I implemented a
> workaround then.

Thanks, good to know.

Comment 7 spower 2018-07-03 10:39:58 UTC
We agreed to remove RFEs component from Bugzilla, if you feel the component has been renamed incorrectly please reach out.


Note You need to log in before you can comment on or make changes to this bug.