Bug 1337921 - Backport for api/groups to 5.5.3 request
Summary: Backport for api/groups to 5.5.3 request
Keywords:
Status: CLOSED WORKSFORME
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: API
Version: 5.5.0
Hardware: All
OS: All
high
high
Target Milestone: GA
: 5.7.0
Assignee: abellott
QA Contact: Dave Johnson
URL:
Whiteboard:
Depends On:
Blocks: 1339361 1346973
TreeView+ depends on / blocked
 
Reported: 2016-05-20 13:18 UTC by Josh Carter
Modified: 2016-06-20 20:06 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1339361 1346973 (view as bug list)
Environment:
Last Closed: 2016-06-20 20:06:47 UTC
Category: ---
Cloudforms Team: ---
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

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


Note You need to log in before you can comment on or make changes to this bug.