Bug 838516
| Summary: | RESTAPI: api incorrectly describes host.nics.add() parameters | ||
|---|---|---|---|
| Product: | Red Hat Enterprise Virtualization Manager | Reporter: | Michael Pasternak <mpastern> |
| Component: | ovirt-engine-restapi | Assignee: | Ori Liel <oliel> |
| Status: | CLOSED CURRENTRELEASE | QA Contact: | Yaniv Kaul <ykaul> |
| Severity: | high | Docs Contact: | |
| Priority: | unspecified | ||
| Version: | 3.1.0 | CC: | bazulay, dyasny, ecohen, iheim, jkt, lpeer, mpastern, Rhev-m-bugs, sgrinber, ykaul |
| Target Milestone: | --- | ||
| Target Release: | --- | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Whiteboard: | infra, network | ||
| Fixed In Version: | si13 | Doc Type: | Bug Fix |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2012-12-04 20:01:44 UTC | Type: | Bug |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
http://gerrit.ovirt.org/#/c/6111/ merged, commit 3de04fba32ccbb104d95ef9f8d0cf7ca8917471d ignore last commit number, correct commit: f39263aaae3d198f6ca9980b741948cc2ea86968 Verify on SI13.2 - FAILED http://emerald-31-vdc.qa.lab.tlv.redhat.com/api?rsdl: <parameter required="false" type="xs:string"> <name>hostnic.ip.gateway</name> </parameter> <parameter required="false" type="xs:string"> <name>hostnic.boot_protocol</name> </parameter> <parameter required="false" type="xs:string"> <name>hostnic.mac</name> </parameter> <parameter required="false" type="xs:string"> <name>hostnic.ip.address</name> </parameter> <parameter required="false" type="xs:string"> <name>hostnic.ip.netmask</name> mike - related to bug 846950? (In reply to comment #4) > mike - related to bug 846950? No, this bug is about host nics properties and the other is about adding nics to templates. The fix is actually ok. the parameters: 'hostnic.ip.gateway', 'hostnic.boot_protocol', 'hostnic.mac', 'hostnic.ip.address', 'hostnic.ip.netmask' should have been removed from 'add' signature, not from 'update' signature, and this is indeed the case. These parameters now only appear for update signature. Please re-check verify on si15.1 - PASS these fields appear only for PUT command |
Description of problem: 1. api reports that [@param hostnic.bonding.slaves.host_nic: collection] optional, while validates: "bonding.slaves.id|name" 2. api reports as add() parameters [@param hostnic.ip.gateway: string] [@param hostnic.boot_protocol: string] [@param hostnic.mac: string] [@param hostnic.ip.address: string] [@param hostnic.ip.netmask: string] while no ip related info is not manageable via this interface.