Bug 1316021

Summary: rest openstacknetworkproviders should accept plugin_type in lower case
Product: [oVirt] ovirt-engine Reporter: Marcin Mirecki <mmirecki>
Component: RestAPIAssignee: Marcin Mirecki <mmirecki>
Status: CLOSED CURRENTRELEASE QA Contact: Meni Yakove <myakove>
Severity: low Docs Contact:
Priority: medium    
Version: 4.0.0CC: bugs, danken, gklein, myakove, ylavi
Target Milestone: ovirt-4.0.0-rcKeywords: Automation
Target Release: 4.0.0Flags: rule-engine: ovirt-4.0.0+
rule-engine: planning_ack+
danken: devel_ack+
rule-engine: testing_ack+
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-07-05 07:48:32 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Network RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1314375    

Description Marcin Mirecki 2016-03-09 09:37:18 UTC
Description of problem:


Adding an OpenStack Neutron provider using rest api fails when plugin type is specified using lower case.

A typical content of the query:
    <openstack_network_provider>
        <name>openstack_network_provider</name>
        <url>http://10.35.128.217:9696</url>
        <requires_authentication>true</requires_authentication>
        <username>neutron</username>
        <password>6fc1fe009a7740d2</password>
        <authentication_url>http://10.35.128.217:35357/v2.0/</authentication_url>
        <tenant_name>services</tenant_name>
        <plugin_type>open_vswitch</plugin_type>
        <agent_configuration>
            <network_mappings>vmnet:br-neutron</network_mappings>
            <broker_type>rabbit_mq</broker_type>
            <address>10.35.128.217</address>
            <port>5672</port>
            <username>guest</username>
            <password>guest</password>
        </agent_configuration>
    </openstack_network_provider>

Only <plugin_type> is supposed to be upper case, which is misleading to users.

Change to accept any case.

Comment 1 Mike McCune 2016-03-28 23:08:19 UTC
This bug was accidentally moved from POST to MODIFIED via an error in automation, please see mmccune with any questions

Comment 2 Sandro Bonazzola 2016-05-02 09:55:50 UTC
Moving from 4.0 alpha to 4.0 beta since 4.0 alpha has been already released and bug is not ON_QA.

Comment 3 Yaniv Lavi 2016-05-23 13:17:15 UTC
oVirt 4.0 beta has been released, moving to RC milestone.

Comment 4 Yaniv Lavi 2016-05-23 13:23:46 UTC
oVirt 4.0 beta has been released, moving to RC milestone.

Comment 5 Meni Yakove 2016-06-22 08:25:37 UTC
rhevm-4.0.0.5-0.1.el7ev.noarch

Comment 6 Sandro Bonazzola 2016-07-05 07:48:32 UTC
oVirt 4.0.0 has been released, closing current release.