Bug 1194271

Summary: [REGENERATE PYTHON SDK] Missing namespace and prinicipal parameters for managing users in CLI
Product: [Retired] oVirt Reporter: Juan Hernández <juan.hernandez>
Component: ovirt-engine-sdkAssignee: Juan Hernández <juan.hernandez>
Status: CLOSED CURRENTRELEASE QA Contact: Karolína Hajná <khajna>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 3.5CC: alonbl, bugs, ecohen, gklein, juan.hernandez, lsurette, omachace, oourfali, pnovotny, pstehlik, rbalakri, yeylon
Target Milestone: ---   
Target Release: 3.5.2   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: infra
Fixed In Version: ovirt-engine-sdk-python-3.5.2.1 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: 1188176 Environment:
Last Closed: 2015-04-29 06:18:30 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:
Bug Depends On: 1188176    
Bug Blocks: 1193058    

Description Juan Hernández 2015-02-19 13:04:23 UTC
+++ This bug was initially created as a clone of Bug #1188176 +++

Description of problem:


Version-Release number of selected component (if applicable):
rhevm-restapi-3.5.0-0.30.el6ev.noarch
rhevm-cli-3.5.0.5-1.el6ev.noarch

How reproducible:
always

Steps to Reproduce:
1. Add ldap domain into engine(ie via engine-manage-domains)
2. add user from this domain to engine. 
shell (connected)]# add user 
correlation_id  domain-id       domain-name     expect          user_name

Actual results:
Missing principal and namespace parameter

Expected results:
principal and namespace parameter are not missing

Additional info:

--- Additional comment from Juan Hernández on 2015-02-02 06:19:13 EST ---

This happens because the "principal" and "namespace" parameters aren't correctly documented in the RSDL metadata. Once the metadata is fixed the Python and Java SDKs will need to be regenerated. The CLI itself won't need any change.

--- Additional comment from Yair Zaslavsky on 2015-02-05 08:57:18 EST ---

Juan, any reason groups:add is not represented in RSDL?

--- Additional comment from Juan Hernández on 2015-02-05 09:14:58 EST ---

(In reply to Yair Zaslavsky from comment #2)
> Juan, any reason groups:add is not represented in RSDL?

It is represented. From "rsdl_metadata.yaml":

- name: /groups|rel=add
  description: add a new group to the system
  request:
    body:
      parameterType: Group
      signatures:
      - mandatoryArguments: {group.name: 'xs:string'}
        description: add a new group to the system
    urlparams: {}
    headers:
      Content-Type: {value: application/xml|json, required: true}
      Correlation-Id: {value: 'any string', required: false}

And from the generated RSDL document:

<link href="groups" rel="add">
  <description>add a new group to the system</description>
  <request>
    <http_method>POST</http_method>
    <headers>
      <header required="true">
        <name>Content-Type</name>
        <value>application/xml|json</value>
      </header>
      <header required="false">
        <name>Correlation-Id</name>
        <value>any string</value>
      </header>
    </headers>
    <body>
      <type>Group</type>
      <parameters_set>
        <description>add a new group to the system</description>
        <parameter required="true" type="xs:string">
          <name>group.name</name>
        </parameter>
      </parameters_set>
    </body>
  </request>
  <response>
    <type>Group</type>
  </response>
</link>

--- Additional comment from Juan Hernández on 2015-02-05 09:21:30 EST ---

Please set a target release.

Comment 1 Ondra Machacek 2015-03-19 17:32:32 UTC
If ovirt 3.5.2 == rhevm 3.5.1, should this fix be also in rhev 3.5.1? As
I see there older package(rhevm-sdk-python-3.5.2.0-1.el6ev.noarch)

Comment 2 Juan Hernández 2015-03-24 09:43:57 UTC
This issue was fixed in ovirt-engine-sdk-python-3.5.2.1 and rhevm-sdk-python-3.5.2.0 doesn't contain it because it is based on ovirt-engine-sdk-python-3.5.2.0.

Currently there is no downstream bug requesting this fix, so at the moment it is not planned to include this fix in the RHEV-M 3.5.1 release.

However, if the next 3.5.1 build is rebased using ovirt-engine-3.5.2 then the engine fix will be included in RHEV-M, and then the rhevm-sdk-python will need to be regenerated. Once it is regenerated this fix will be included.

Comment 3 Pavel Novotny 2015-04-02 10:55:13 UTC
ghf

Comment 4 Juan Hernández 2015-04-06 07:56:23 UTC
What is the question?

Comment 5 Pavel Novotny 2015-04-07 12:48:56 UTC
Oops, sorry, the needinfo and comment message have been submitted by mistake, please ignore.

Comment 6 Karolína Hajná 2015-04-07 13:25:30 UTC
Verified on ovirt-engine-sdk-python-3.5.2.1

Comment 7 Eyal Edri 2015-04-29 06:18:30 UTC
ovirt 3.5.2 was GA'd. closing current release.