Project

General

Profile

Bug #20862

Katello overrides apipie definition for hostgroups

Added by Marek Hulán over 1 year ago. Updated 11 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Marek Hulán
Category:
API doc
Target version:
Katello 3.4.6
Difficulty:
Triaged:
Yes
Bugzilla link:
Pull request:
https://github.com/Katello/katello/pull/6937
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

When some plugins wants to extend hostgroup API (e.g. foreman_openscap) through mechanisms supported either by core or by apipie-rails, Katello overrides the attribute set. What's worse, it overrides the hostgroup param group with the out of date list of attributes, effectively hiding description attribute. Katello should start using apipie-rails 0.5.2 new feature to extend the documentation with built-in helper.

This is a blocker for foreman_openscap critical fix, please consider it for Katello 3.4.6 release.

Related issues

Related to Katello - Bug #16484: Cannot create new hostgroup via API/CLIClosed2016-09-08
Blocks OpenSCAP - Bug #16490: API lacks ability to modify/specify hostgroup openscap capsule via API or hammer cliClosed2016-09-08
Copied to Katello - Bug #20863: Katello overrides apipie definition for Smart ProxiesClosed2017-09-06

Associated revisions

Revision ef092004 (diff)
Added by Marek Hulán over 1 year ago

Fixes #20862 - correctly extend hostgroup API

History

#1 Updated by Marek Hulán over 1 year ago

  • Related to Bug #16484: Cannot create new hostgroup via API/CLI added

#2 Updated by Marek Hulán over 1 year ago

  • Blocks Bug #16490: API lacks ability to modify/specify hostgroup openscap capsule via API or hammer cli added

#3 Updated by The Foreman Bot over 1 year ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/Katello/katello/pull/6937 added

#4 Updated by Marek Hulán over 1 year ago

if this is accepted to 3.4, please let me know, we'd need to bump apipie-rails in our 1.15 since it contains 0.4.0 only repos https://github.com/theforeman/foreman-packaging/tree/rpm/1.15/rubygem-apipie-rails

#5 Updated by Justin Sherrill over 1 year ago

  • Legacy Backlogs Release (now unused) set to 295

#6 Updated by Justin Sherrill over 1 year ago

  • Copied to Bug #20863: Katello overrides apipie definition for Smart Proxies added

#7 Updated by Marek Hulán over 1 year ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#8 Updated by The Foreman Bot over 1 year ago

  • Pull request https://github.com/Katello/katello/pull/6943 added

#9 Updated by Marek Hulán over 1 year ago

  • Pull request deleted (https://github.com/Katello/katello/pull/6943)

Also available in: Atom PDF