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

Add Redfish provider logger #17485

Merged
merged 1 commit into from
May 31, 2018
Merged

Add Redfish provider logger #17485

merged 1 commit into from
May 31, 2018

Conversation

tadeboro
Copy link
Contributor

In order to keep the main log clean of Redfish provider messages, we
added new log file and associated logger that can be used by the
Redfish provider.

In order to keep the main log clean of Redfish provider messages, we
added new log file and associated logger that can be used by the
Redfish provider.
@tadeboro
Copy link
Contributor Author

@miq-bot assign @gtanzillo
/cc @gberginc @matejart

@miq-bot
Copy link
Member

miq-bot commented May 28, 2018

Checked commit xlab-si@6e9f73e with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 0 offenses detected
Everything looks fine. 🍪

Copy link
Contributor

@gberginc gberginc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Makes a lot of sense to separate the Redfish specific logs.

@carbonin carbonin requested a review from bdunne May 30, 2018 21:42
@bdunne bdunne merged commit 805cbde into ManageIQ:master May 31, 2018
@bdunne bdunne assigned bdunne and unassigned gtanzillo May 31, 2018
@bdunne bdunne added this to the Sprint 87 Ending Jun 4, 2018 milestone May 31, 2018
@tadeboro tadeboro deleted the redfish-logger branch June 1, 2018 05:19
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.

5 participants