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

Fixed log levels in worker code #249

Merged
merged 1 commit into from
Aug 12, 2020
Merged

Conversation

parauliya
Copy link
Contributor

Description

Changed log level from error to warning in worker code.

Why is this needed

How Has This Been Tested?

This has been tested with the workflow.

How are existing users impacted? What migration steps/scripts do we need?

No impact on existing user and no migration script needed.

@parauliya parauliya self-assigned this Aug 12, 2020
@parauliya parauliya added this to In Progress in Issues List via automation Aug 12, 2020
Signed-off-by: parauliya <aman@infracloud.io>
Copy link
Contributor

@gauravgahlot gauravgahlot left a comment

Choose a reason for hiding this comment

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

A much needed correction.

@gauravgahlot gauravgahlot added the ready-to-merge Signal to Mergify to merge the PR. label Aug 12, 2020
@mergify mergify bot merged commit b8a4a3d into tinkerbell:master Aug 12, 2020
Issues List automation moved this from In Progress to Just shipped Aug 12, 2020
@gauravgahlot gauravgahlot deleted the log_levels branch August 12, 2020 16:03
@mmlb mmlb removed the ready-to-merge Signal to Mergify to merge the PR. label Jan 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Issues List
  
Just shipped
Development

Successfully merging this pull request may close these issues.

None yet

3 participants