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

stop logging of createOrUpdateJob #136

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ushitora-anqou
Copy link
Contributor

The current implementation of createOrUpdateJob prints "createOrUpdate" and "createOrUpdateJob Finished" info logs every time the function runs, both before and after execution. Since this function is called repeatedly from the reconciliation loop, these logs are generated excessively. However, they don't provide any meaningful information and clutter the log readability.

This commit removes these logs to make the logs for pie much clearer.

The current implementation of createOrUpdateJob prints "createOrUpdate"
and "createOrUpdateJob Finished" info logs every time the function runs,
both before and after execution. Since this function is called
repeatedly from the reconciliation loop, these logs are generated
excessively. However, they don't provide any meaningful information and
clutter the log readability.

This commit removes these logs to make the logs for pie much clearer.

Signed-off-by: Ryotaro Banno <ryotaro.banno@gmail.com>
@ushitora-anqou ushitora-anqou marked this pull request as ready for review October 4, 2024 08:17
@ushitora-anqou ushitora-anqou requested a review from a team as a code owner October 4, 2024 08:17
Copy link
Contributor

@llamerada-jp llamerada-jp left a comment

Choose a reason for hiding this comment

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

There is a similar log output at the end of this function. Does this log need to be reconsidered?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Review in progress
Development

Successfully merging this pull request may close these issues.

2 participants