Project

General

Profile

Bug #28491

Move stuff from content_view_versions/base.json.rabl to content_view_versions/show.json.rabl

Added by Partha Aji over 2 years ago. Updated 5 months ago.

Status:
Ready For Testing
Priority:
Normal
Assignee:
-
Category:
Content Views
Target version:
Difficulty:
Triaged:
Yes
Bugzilla link:
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

The content view versions list index call takes a very long time because of the following attributes in the following file
https://github.com/Katello/katello/blob/master/app/views/katello/api/v2/content_view_versions/base.json.rabl

In my tests commenting out

child :last_event => :last_event do
  extends 'katello/api/v2/content_view_histories/show'
end
child :active_history => :active_history do
  extends 'katello/api/v2/content_view_histories/show'
end

Shaved off almost 2 seconds from the index call (I had a CV with 37 versions)
Similarly I shaved off an additional couple of seconds by commenting out
child :archived_repos => :repositories do
  attributes :id, :name, :label, :content_type, :library_instance_id
end

Creating this task to investigate and move stuff from content_view_versions/base.json.rabl to content_view_versions/show.json.rabl
List page should need less details than details page.

History

#1 Updated by Partha Aji over 2 years ago

#2 Updated by Ian Ballou over 2 years ago

  • Triaged changed from No to Yes
  • Target version set to Katello 4.0.0

#3 Updated by Chris Roberts almost 2 years ago

  • Target version changed from Katello 4.0.0 to Katello 3.17.0

#4 Updated by Chris Roberts over 1 year ago

  • Target version changed from Katello 3.17.0 to Katello 3.18.0

#5 Updated by Jonathon Turel over 1 year ago

  • Category set to Content Views

#6 Updated by Jonathon Turel over 1 year ago

  • Target version changed from Katello 3.18.0 to Katello 4.1.0

#7 Updated by Ian Ballou over 1 year ago

  • Blocks deleted (Tracker #28490: katello 4.0 breaking changes tracker)

#8 Updated by Jonathon Turel about 1 year ago

  • Target version changed from Katello 4.1.0 to Katello 4.2.0

#9 Updated by Justin Sherrill 10 months ago

  • Target version changed from Katello 4.2.0 to Katello 4.3.0

#10 Updated by The Foreman Bot 9 months ago

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

#11 Updated by Jonathon Turel 5 months ago

  • Target version changed from Katello 4.3.0 to Katello Backlog

Considering the rework required on the PR I'm moving this to the backlog for now and we can re-prioritize when it's ready for review! Thanks

Also available in: Atom PDF