Project

General

Profile

Bug #14106

Remove puppetclass no_route page

Added by Daniel Lobato Garcia almost 5 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Normal
Category:
Puppet integration
Target version:
Difficulty:
Triaged:
Bugzilla link:

Description

Using Rack::Deflater we can compress our responses for HTML and assets using gzip automatically. This allows us to serve considerably smaller (in kb) pages, which should go over the wire faster as well.

Trying out /login, /hosts, and /dashboard, processing times were similar with and without this, but the page size was reduced up to 80% in some cases.

See the following gist with benchmarks using siege: https://gist.github.com/dLobatog/600f151fd0676345c95a - notice some pages like /login are reduced a whooping 75% in size.


Related issues

Related to Foreman - Bug #1556: Puppet rdoc generation is broken in ForemanClosed2012-03-28

Associated revisions

Revision 5dbbc05b (diff)
Added by Daniel Lobato Garcia almost 5 years ago

Fixes #14106 - Remove puppetclasses no_route page

I don't think it's being used anywhere anymore

History

#1 Updated by Daniel Lobato Garcia almost 5 years ago

  • Subject changed from Return 15-30% smaller controller responses to Compress 15-30% (on avg) controller responses with Rack::Deflater

#2 Updated by The Foreman Bot almost 5 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/3295 added

#3 Updated by Daniel Lobato Garcia almost 5 years ago

  • Tracker changed from Feature to Bug
  • Subject changed from Compress 15-30% (on avg) controller responses with Rack::Deflater to Remove puppetclass no_route page
  • Category changed from Performance to Puppet integration

I don't think it's being used anywhere anymore

#4 Updated by The Foreman Bot almost 5 years ago

  • Pull request https://github.com/theforeman/foreman/pull/3300 added

#5 Updated by Dominic Cleal almost 5 years ago

  • Related to Bug #1556: Puppet rdoc generation is broken in Foreman added

#6 Updated by Daniel Lobato Garcia almost 5 years ago

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

#7 Updated by Dominic Cleal almost 5 years ago

  • Legacy Backlogs Release (now unused) set to 136

Also available in: Atom PDF