Bug 1337921

Summary:	Backport for api/groups to 5.5.3 request
Product:	Red Hat CloudForms Management Engine	Reporter:	Josh Carter <jocarter>
Component:	API	Assignee:	abellott
Status:	CLOSED WORKSFORME	QA Contact:	Dave Johnson <dajohnso>
Severity:	high	Docs Contact:
Priority:	high
Version:	5.5.0	CC:	cpelland, jhardy, obarenbo
Target Milestone:	GA	Keywords:	ZStream
Target Release:	5.7.0
Hardware:	All
OS:	All
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:
Clones:	1339361 1346973 (view as bug list)		Environment:
Last Closed:	2016-06-20 20:06:47 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:
Bug Depends On:
Bug Blocks:	1339361, 1346973

Comment 3 CFME Bot 2016-05-31 16:19:09 UTC

New commit detected on cfme/5.5.z:
https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=18f4651d04706e14eb85b6bef73337718ad10723

commit 18f4651d04706e14eb85b6bef73337718ad10723
Author:     Alberto Bellotti <abellott>
AuthorDate: Wed Nov 25 11:38:51 2015 -0500
Commit:     Alberto Bellotti <abellott>
CommitDate: Wed May 25 15:35:53 2016 -0400

    [api] Group CRUD
    
    - Enabling post/put/patch/delete on /api/groups
    - Refactored/re-using parse_<type> methods
    - Made role/tenant fetches RBAC aware
    - Added filters structure validation in model
    - Added specs
    - PR Review changes
    
    https://trello.com/c/wPzTtz4S/147-api-need-full-crud-on-api-groups
    
    Original PR https://github.com/ManageIQ/manageiq/pull/5611
    Clean Cherry Pick 39c8d9a, 94e535a
    https://bugzilla.redhat.com/show_bug.cgi?id=1337921

 app/controllers/api_controller.rb         |   1 +
 app/controllers/api_controller/groups.rb  |  48 ++++++
 app/controllers/api_controller/parser.rb  |  52 ++++--
 app/controllers/api_controller/roles.rb   |   2 +-
 app/controllers/api_controller/tenants.rb |  11 +-
 app/models/miq_group.rb                   |   7 +
 config/api.yml                            |   6 +-
 spec/requests/api/groups_spec.rb          | 264 ++++++++++++++++++++++++++++++
 spec/requests/api/tenants_spec.rb         |   3 +-
 9 files changed, 370 insertions(+), 24 deletions(-)
 create mode 100644 app/controllers/api_controller/groups.rb
 create mode 100644 spec/requests/api/groups_spec.rb

Comment 4 CFME Bot 2016-05-31 16:19:25 UTC

New commit detected on cfme/5.5.z:
https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=eaef822527f8e7f40f228b773725e5631c705af2

commit eaef822527f8e7f40f228b773725e5631c705af2
Merge: 418e1a6 18f4651
Author:     Gregg Tanzillo <gtanzill>
AuthorDate: Tue May 31 12:06:45 2016 -0400
Commit:     Gregg Tanzillo <gtanzill>
CommitDate: Tue May 31 12:06:45 2016 -0400

    Merge branch '55z_api_group_crud' into '5.5.z'
    
    [api] Group CRUD
    
    - Enabling post/put/patch/delete on /api/groups
    - Refactored/re-using parse_<type> methods
    - Made role/tenant fetches RBAC aware
    - Added filters structure validation in model
    - Added specs
    - PR Review changes
    
    https://trello.com/c/wPzTtz4S/147-api-need-full-crud-on-api-groups
    
    Original PR https://github.com/ManageIQ/manageiq/pull/5611
    
    Clean Cherry Pick 39c8d9a, 94e535a
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1337921
    
    See merge request !951

 app/controllers/api_controller.rb         |   1 +
 app/controllers/api_controller/groups.rb  |  48 ++++++
 app/controllers/api_controller/parser.rb  |  52 ++++--
 app/controllers/api_controller/roles.rb   |   2 +-
 app/controllers/api_controller/tenants.rb |  11 +-
 app/models/miq_group.rb                   |   7 +
 config/api.yml                            |   6 +-
 spec/requests/api/groups_spec.rb          | 264 ++++++++++++++++++++++++++++++
 spec/requests/api/tenants_spec.rb         |   3 +-
 9 files changed, 370 insertions(+), 24 deletions(-)

Comment 6 abellott 2016-06-20 20:06:47 UTC

API Groups support already in 5.6