Bugzilla will be upgraded to version 5.0. The upgrade date is tentatively scheduled for 2 December 2018, pending final testing and feedback.
Bug 1429642 - Remove package profile JSON from production.log
Remove package profile JSON from production.log
Status: CLOSED ERRATA
Product: Red Hat Satellite 6
Classification: Red Hat
Component: Logging (Show other bugs)
6.2.7
Unspecified Unspecified
unspecified Severity unspecified (vote)
: 6.2.9
: Unused
Assigned To: Justin Sherrill
Renzo Nuccitelli
: PrioBumpField
Depends On:
Blocks: 1429670
  Show dependency treegraph
 
Reported: 2017-03-06 13:19 EST by Chris Duryee
Modified: 2017-05-01 10:00 EDT (History)
5 users (show)

See Also:
Fixed In Version: tfm-rubygem-katello-3.0.0.115-1
Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1429670 (view as bug list)
Environment:
Last Closed: 2017-05-01 10:00:55 EDT
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)


External Trackers
Tracker ID Priority Status Summary Last Updated
Foreman Issue Tracker 15940 None None None 2017-03-06 13:19 EST
Red Hat Product Errata RHBA-2017:1191 normal SHIPPED_LIVE Satellite 6.2.9 Async Bug Release 2017-05-01 13:49:42 EDT

  None (edit)
Description Chris Duryee 2017-03-06 13:19:58 EST
Description of problem:

The entire package profile json is logged in production.log. This makes the log extremely painful to read. These lines can be over 110K characters long.

Version-Release number of selected component (if applicable): 6.2.7


How reproducible: every time


Steps to Reproduce:
1. start a satellite and upload a package profile
2. view production.log via 'less'
3.

Actual results: extremely large log message, log takes about 5-10 seconds to page through for each package profile line


Expected results: package profile lines are not logged
Comment 2 pm-sat@redhat.com 2017-03-06 14:04:15 EST
Upstream bug assigned to jsherril@redhat.com
Comment 3 pm-sat@redhat.com 2017-03-06 14:04:17 EST
Moving this bug to POST for triage into Satellite 6 since the upstream issue http://projects.theforeman.org/issues/15940 has been resolved.
Comment 4 Lukas Zapletal 2017-03-07 03:29:07 EST
Just for better clarity, I am renaming to prevent confusion. We are removing one element from log that floods it. I also think this consumes a lot of memory because hash structures are large and causes lot of allocations on the Ruby VM stack.
Comment 5 Renzo Nuccitelli 2017-03-23 11:46:57 EDT
No large messages on 6.2.9. Log after uploading a package on production.log:

2017-03-23 11:42:43 40abcce7 [app] [I] Processing by Katello::Api::V2::RepositoriesController#upload_content as HTML
2017-03-23 11:42:43 40abcce7 [app] [I]   Parameters: {"content"=>[#<ActionDispatch::Http::UploadedFile:0x000000090e8f18 @tempfile=#<Tempfile:/tmp/RackMultipart20170323-17654-11tbe62>, @original_filename="PyPAM-0.5.0-12.el6.src.rpm", @content_type="application/x-rpm", @headers="Content-Disposition: form-data; name=\"content[]\"; filename=\"PyPAM-0.5.0-12.el6.src.rpm\"\r\nContent-Type: application/x-rpm\r\n">], "authenticity_token"=>"voDq5tertDzrjK8v9YoHpadOtYVH9ABp4LurGk+suLE=", "api_version"=>"v2", "id"=>"1"}
2017-03-23 11:42:48 40abcce7 [app] [I] Completed 200 OK in 5095ms (Views: 0.7ms | ActiveRecord: 63.7ms)
2017-03-23 11:42:48 40abcce7 [app] [I] Started GET "/katello/api/v2/repositories/1?organization_id=1" for 10.10.120.229 at 2017-03-23 11:42:48 -0400
2017-03-23 11:42:48 40abcce7 [app] [I] Processing by Katello::Api::V2::RepositoriesController#show as JSON
2017-03-23 11:42:48 40abcce7 [app] [I]   Parameters: {"organization_id"=>"1", "api_version"=>"v2", "id"=>"1"}
2017-03-23 11:42:49 40abcce7 [app] [I]   Rendered /opt/theforeman/tfm/root/usr/share/gems/gems/katello-3.0.0.117/app/views/katello/api/v2/repositories/show.json.rabl within katello/api/v2/layouts/resource (215.6ms)
2017-03-23 11:42:49 40abcce7 [app] [I] Completed 200 OK in 233ms (Views: 184.1ms | ActiveRecord: 39.0ms)
Comment 7 errata-xmlrpc 2017-05-01 10:00:55 EDT
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2017:1191

Note You need to log in before you can comment on or make changes to this bug.