Skip to content

Conversation

djeebus
Copy link
Contributor

@djeebus djeebus commented Sep 4, 2025

This line is 30% of the last 60 minutes of logs in production

@djeebus djeebus marked this pull request as ready for review September 4, 2025 20:38
Copy link
Member

@jakubno jakubno left a comment

Choose a reason for hiding this comment

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

Not sure about removing the log before we solve this, we identified it because of these logs, without them we have 0 observability

@djeebus djeebus self-assigned this Sep 10, 2025
@djeebus djeebus marked this pull request as draft September 10, 2025 00:42
@djeebus djeebus added the improvement Improvement for current functionality label Sep 10, 2025
@djeebus
Copy link
Contributor Author

djeebus commented Sep 15, 2025

Not sure about removing the log before we solve this, we identified it because of these logs, without them we have 0 observability

I don't think there's an actual bug here, it's working as designed, we're just spamming a lot of logs in a happy path.

@jakubno
Copy link
Member

jakubno commented Sep 15, 2025

I though you reported an issue where the start was slowed down and it correlated to seeing a lot of these logs

@djeebus
Copy link
Contributor Author

djeebus commented Sep 15, 2025

I though you reported an issue where the start was slowed down and it correlated to seeing a lot of these logs

Pretty sure the slowdown ended up being the clock sync issue fixed here. Regardless, this was something I found while I was looking for errors, but it was not the error itself.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Improvement for current functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants