Project

General

Profile

Bug #10497

Improve system rabl a bit more

Added by Justin Sherrill over 4 years ago. Updated about 2 months ago.

Status:
Rejected
Priority:
Normal
Assignee:
-
Category:
API
Target version:
-
Difficulty:
easy
Triaged:
Yes
Bugzilla link:
Pull request:
Fixed in Releases:
Found in Releases:

Description

While reviewing https://github.com/Katello/katello/pull/5180

i noticed a few mistakes in the rabl and further speedups that could be performed:

  • katello_agent_installed? could be moved from base to show
  • :content_view should not be used, as this does does not use rabl to form the structure, should just use :id and :name as attributes
  • :environment should not call the other show, should just :id and :name
  • :installedProducts should probably be moved to show, don't think it is needed in base

History

#1 Updated by Eric Helms over 4 years ago

> * katello_agent_installed? could be moved from base to show

You may want to consider this request when making this change -- http://projects.theforeman.org/issues/10432

#2 Updated by Eric Helms over 4 years ago

  • Legacy Backlogs Release (now unused) changed from 31 to 70

#3 Updated by Justin Sherrill almost 4 years ago

  • Legacy Backlogs Release (now unused) changed from 70 to 86

#4 Updated by Eric Helms over 3 years ago

  • Legacy Backlogs Release (now unused) changed from 86 to 114

#5 Updated by John Mitsch about 2 months ago

  • Target version deleted (Katello Backlog)
  • Status changed from New to Rejected

Thanks for reporting this issue. This issue was created over 4 years ago and hasn't seen an update in 1 year. We are closing this in an effort to keep a realistic backlog. Please open up a new issue that includes a link to this issue if you feel this still needs to be addressed. We can then triage the new issue and reassess.

Also available in: Atom PDF