Bug #22968
closedDrop access logs and add qpid logs
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
Updated by The Foreman Bot about 7 years ago
- Status changed from New to Ready For Testing
- Pull request https://github.com/theforeman/foreman-packaging/pull/2300 added
Updated by The Foreman Bot about 7 years ago
- Pull request https://github.com/theforeman/foreman/pull/5344 added
Updated by The Foreman Bot about 7 years ago
- Pull request https://github.com/theforeman/foreman-packaging/pull/2302 added
Updated by Chris Roberts about 7 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset rpms:foreman-packaging|80bc5ddfec99ba4a540a789f4a1e55e738890d76.
Updated by Chris Roberts almost 7 years ago
- Triaged set to No
- Bugzilla link set to 1546099
Updated by Chris Roberts almost 7 years ago
- Pull request deleted (
https://github.com/theforeman/foreman-packaging/pull/2300)