Project

General

Profile

Bug #14259

product API call with 'enabled=true' slow, causing product page to load slowly

Added by Chris Duryee over 3 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Performance
Target version:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

The product listing page takes about 5-7 seconds to load with 10 custom products and 7 RH products enabled.

It looks like the 'enabled=true' flag when calling the products API has something to do with this. For example:

# time curl -sk 'https://admin:changeme@localhost/katello/api/products?organization_id=3&page=1&paged=true&search=&sort_by=name&sort_order=ASC' > /dev/null

real    0m1.853s
user    0m0.034s
sys     0m0.024s

# time curl -sk 'https://admin:changeme@localhost/katello/api/products?enabled=true&organization_id=3&page=1&paged=true&search=&sort_by=name&sort_order=ASC' > /dev/null

real    0m4.246s
user    0m0.024s
sys     0m0.028s

There is a 'self.enabled' method on the product model that may need investigation.

Associated revisions

Revision 98e9acf6 (diff)
Added by Adam Price over 3 years ago

fixes #14259 - just make one method call, rather than 13

Revision e9526161
Added by Adam Price over 3 years ago

Merge pull request #5991 from komidore64/rmi14259-products-index

fixes #14259 - just make one method call, rather than 13

Revision c11d6c0e (diff)
Added by Adam Price over 3 years ago

refs #14259 - reduce data sent back from products index (#6012)

History

#1 Updated by Eric Helms over 3 years ago

  • Legacy Backlogs Release (now unused) set to 143

#2 Updated by Mike McCune over 3 years ago

  • Assignee set to Adam Price

#3 Updated by Adam Price over 3 years ago

  • Bugzilla link set to 1330307

#4 Updated by The Foreman Bot over 3 years ago

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

#5 Updated by Anonymous over 3 years ago

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

#6 Updated by The Foreman Bot over 3 years ago

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

#7 Updated by Chris Duryee over 3 years ago

  • Status changed from Closed to Assigned

#8 Updated by Adam Price over 3 years ago

  • Status changed from Assigned to Ready For Testing

PR #6012 has been merged.

#9 Updated by Chris Duryee about 3 years ago

  • Status changed from Ready For Testing to Closed

both PRs merged, moving issue to closed

Also available in: Atom PDF