Project

General

Profile

Actions

Bug #22968

closed

Drop access logs and add qpid logs

Added by Chris Roberts about 6 years ago. Updated almost 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
foreman-debug
Target version:
-
Fixed in Releases:
Found in Releases:

Description

Description of problem:
I suggest below changes what f-d shall / should not collect:

It shall newly collect:

- /var/log/httpd/katello-reverse-proxy_access_ssl.log*
- /var/log/httpd/katello-reverse-proxy_error_ssl.log*
- both are equivalents of foreman-ssl_*_ssl.log but for Capsules

- /var/log/foreman/dynflow_executor.output*
- the logfile showed an error useful several times already (like unexpected "No executor available")

- /var/log/httpd/puppet_error_ssl.log*
- ?? it might show some puppet related problems but it has been very rarely used, no problem if this sint collected

- commands "katello-service status" and "hammer ping"
- both can reveal basi issues of Satellite health

foreman-debug should not further collect cmds output stored in these files:
qpid_stat_queues
qpid_stat_subscriptions
qpid-stat-resource_manager
qpid-rpm-qa

The first two are duplicates of qpid-stat-c and qpid-stat-u.

resource_manager detailed stats were added to deal with deadlocks in pulp/celery/kombu/python-qpid, not sure if it makes still sense to collect (bmbouter might know more)

qpid-rpm-qa is a subset of "installed_package", hence redundant as is

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

How reproducible:
100%

Steps to Reproduce:
1. Collect foreman-debug
2. Check if the archive contains above files / cmds output.

Actual results:
see Description

Expected results:
see Description

Actions #1

Updated by The Foreman Bot about 6 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman-packaging/pull/2300 added
Actions #2

Updated by The Foreman Bot about 6 years ago

  • Pull request https://github.com/theforeman/foreman/pull/5344 added
Actions #3

Updated by The Foreman Bot about 6 years ago

  • Pull request https://github.com/theforeman/foreman-packaging/pull/2302 added
Actions #4

Updated by Chris Roberts about 6 years ago

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

Updated by Chris Roberts almost 6 years ago

  • Triaged set to No
  • Bugzilla link set to 1546099
Actions #6

Updated by Chris Roberts almost 6 years ago

  • Pull request deleted (https://github.com/theforeman/foreman-packaging/pull/2300)
Actions

Also available in: Atom PDF