Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[foreman] Skip collection of generic resources #489

Merged
merged 1 commit into from Feb 3, 2015

Conversation

lzap
Copy link
Contributor

@lzap lzap commented Feb 3, 2015

New option was added to foreman-debug (-g) to skip collecting of generic
resources (firewall, selinux, cpu, memory). This patch leverages that.

https://github.com/theforeman/foreman/blob/develop/script/foreman-debug

Since katello-debug is now part of foreman-debug (as a plugin), there is no
need of separate sosreport plugin.

https://github.com/Katello/katello/blob/master/deploy/script/katello-debug.sh

New subpackage called foreman-debug was introduced. It is present in both
upstream Foreman and Katello installations as well as on both Satellite 6 main
node and capsule instances. It can be used to detect if foreman-debug script
is present.

Signed-off-by: Lukas Zapletal lzap+git@redhat.com

New option was added to foreman-debug (-g) to skip collecting of generic
resources (firewall, selinux, cpu, memory). This patch leverages that.

https://github.com/theforeman/foreman/blob/develop/script/foreman-debug

Since katello-debug is now part of foreman-debug (as a plugin), there is no
need of separate sosreport plugin.

https://github.com/Katello/katello/blob/master/deploy/script/katello-debug.sh

New subpackage called foreman-debug was introduced. It is present in both
upstream Foreman and Katello installations as well as on both Satellite 6 main
node and capsule instances. It can be used to detect if foreman-debug script is
present.

Signed-off-by: Lukas Zapletal <lzap+git@redhat.com>
@lzap
Copy link
Contributor Author

lzap commented Feb 3, 2015

Our upstream bug is: http://projects.theforeman.org/issues/8710

Downstream bug is: https://bugzilla.redhat.com/show_bug.cgi?id=1134109

This also solves the two dependant bugs we commented today. Ideally we should wait until everything is merged. If you want, I can ping you when appropriate. But I don't expect changes in this field. Thanks for help!

@bmr-cymru bmr-cymru merged commit 5cc0613 into sosreport:master Feb 3, 2015
@bmr-cymru
Copy link
Member

Since it's upstream (and won't be in a release for some time) I'm happy to merge it now - anyone who's running master should expect some turbulence or dependencies on unreleased changes :)

Thanks for working on this!

@lzap
Copy link
Contributor Author

lzap commented Feb 3, 2015

Roger wilco. Thanks.

@lzap lzap deleted the foreman-debug branch February 3, 2015 16:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants