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.

Sign up for GitHub

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

Jump to bottom

Update failure and deploytime exporter logging statements #157

Closed
themoosman opened this issue Aug 11, 2020 · 1 comment · Fixed by #187
Closed

Update failure and deploytime exporter logging statements #157

themoosman opened this issue Aug 11, 2020 · 1 comment · Fixed by #187
Labels
Debt good first issue Denotes an issue ready for a new contributor, according to the "help wanted" guidelines. software delivery performance Outcome
Milestone
v1.3

Comments

@themoosman
Copy link
Collaborator

Currently, the deploytime and failure exporters are using print() statements to log messages. These should be modified to use logging.info() or logging.debug(), etc. as used in the committime exporter.
@themoosman themoosman added good first issue Denotes an issue ready for a new contributor, according to the "help wanted" guidelines. Debt labels Aug 11, 2020
@themoosman themoosman added this to the v1.3 milestone Aug 11, 2020
@etsauer etsauer linked a pull request Sep 21, 2020 that will close this issue
Deploytime api fix #187
Merged
@etsauer
Copy link
Collaborator

etsauer commented Sep 21, 2020

This issue has been resolved.

@etsauer etsauer closed this as completed Sep 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Debt good first issue Denotes an issue ready for a new contributor, according to the "help wanted" guidelines. software delivery performance Outcome
Projects
None yet
Milestone
v1.3
Development

Successfully merging a pull request may close this issue.

Deploytime api fix redhat-mal/pelorus
3 participants
@themoosman @etsauer @prakritikoller