Skip to content

Conversation

bdunne
Copy link
Member

@bdunne bdunne commented May 4, 2015

Add logging to LinuxAdmin to help in debugging.

Also, set AwesomeSpawn logger if it isn't already set

https://bugzilla.redhat.com/show_bug.cgi?id=1069247

@bdunne
Copy link
Member Author

bdunne commented May 4, 2015

@Fryguy Please review

@miq-bot
Copy link
Member

miq-bot commented May 4, 2015

Checked commits bdunne@eb9c6c4 .. bdunne@98ce6b8 with rubocop 0.27.1
8 files checked, 1 offense detected

lib/linux_admin.rb

@coveralls
Copy link

Coverage Status

Coverage increased (+0.01%) to 99.32% when pulling 98ce6b8 on brandondunne:add_logging into 0019009 on ManageIQ:master.

Fryguy added a commit that referenced this pull request May 4, 2015
@Fryguy Fryguy merged commit e028e67 into ManageIQ:master May 4, 2015
@Fryguy
Copy link
Member

Fryguy commented May 4, 2015

👍

@bdunne bdunne deleted the add_logging branch May 4, 2015 18:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants