Bug 1724114

Summary:	Cloud Tenant Placement is ignored in Add New Network Router for OpenStack Network Manager
Product:	Red Hat CloudForms Management Engine	Reporter:	Bryan Kinney <bkinney>
Component:	Providers	Assignee:	Sam Lucidi <slucidi>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Ido Ovadia <iovadia>
Severity:	high	Docs Contact:	Red Hat CloudForms Documentation <cloudforms-docs>
Priority:	medium
Version:	5.10.4	CC:	dmetzger, iovadia, jfrey, jhardy, maufart, mfeifer, obarenbo, simaishi, slucidi
Target Milestone:	GA	Keywords:	TestOnly, ZStream
Target Release:	5.11.0	Flags:	mfeifer: mirror+
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	5.11.0.19	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:
Clones:	1740230 (view as bug list)		Environment:
Last Closed:	2019-12-13 15:00:47 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	Bug
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	Openstack	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	1740230

Description Bryan Kinney 2019-06-26 09:30:29 UTC

Description of problem:
Trying to create a new Network Router in an OpenStack Network Manager and specify the mapped cloud tenant.
The router is created. Although, in the Router's details the Cloud Tenant is "admin". As such it can't be used by the cloud tenant we specified.  In Horizon the router is clearly set to the admin project.

1. Tenant Mapping is enabled.
2. admin user is a member of every project created on OpenStack 13.
3. The intended cloud tenant appears in the pull down menu.
4. Tested as both the tenant and Administrator user in CloudForms.

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1. In OSP add admin to all projects
2. Enable Tenant Mapping for OSP Infrastructure Provider 
3. Add a new router: Networks > Network Routers > Configuration > Add a new Router
4. Fill in all fields including Cloud Tenant Placement

Actual results:
New Network router is part of the admin project in OpenStack instead of the specified project/user

Expected results:
New network router should belong to the specified  project/user.

Additional info:

Comment 6 CFME Bot 2019-08-09 18:32:37 UTC

https://github.com/ManageIQ/manageiq-ui-classic/pull/5981

Comment 7 CFME Bot 2019-08-12 08:26:35 UTC

New commit detected on ManageIQ/manageiq-ui-classic/master:

https://github.com/ManageIQ/manageiq-ui-classic/commit/df8f02fa6103ee226b29d90af75121168ee65f91
commit df8f02fa6103ee226b29d90af75121168ee65f91
Author:     Sam Lucidi <slucidi>
AuthorDate: Fri Aug  9 14:06:41 2019 -0400
Commit:     Sam Lucidi <slucidi>
CommitDate: Fri Aug  9 14:06:41 2019 -0400

    NetworkRouter form_params should get tenant id from nested hash

    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1724114

 app/controllers/network_router_controller.rb | 4 +-
 spec/controllers/network_router_controller_spec.rb | 3 +-
 2 files changed, 4 insertions(+), 3 deletions(-)

Comment 8 CFME Bot 2019-08-12 13:43:52 UTC

New commit detected on ManageIQ/manageiq-ui-classic/ivanchuk:

https://github.com/ManageIQ/manageiq-ui-classic/commit/a79841bef48fb2a7d3ae1697efa848621dc193de
commit a79841bef48fb2a7d3ae1697efa848621dc193de
Author:     Milan Zázrivec <mzazrivec>
AuthorDate: Mon Aug 12 04:22:13 2019 -0400
Commit:     Milan Zázrivec <mzazrivec>
CommitDate: Mon Aug 12 04:22:13 2019 -0400

    Merge pull request #5981 from mansam/fix-1724114

    NetworkRouter form_params should get tenant id from nested hash

    (cherry picked from commit bd9c08839a688d3a7bb811670e4dfcc590e38c23)

    https://bugzilla.redhat.com/show_bug.cgi?id=1724114

 app/controllers/network_router_controller.rb | 4 +-
 spec/controllers/network_router_controller_spec.rb | 3 +-
 2 files changed, 4 insertions(+), 3 deletions(-)

Comment 10 Marek Aufart 2019-08-14 08:13:54 UTC

*** Bug 1726285 has been marked as a duplicate of this bug. ***

Comment 11 Ido Ovadia 2019-08-18 15:07:45 UTC

Verified
========
5.11.0.19