-
Notifications
You must be signed in to change notification settings - Fork 8
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 debug logs for job enqueue #27
Conversation
issue.go
Outdated
@@ -47,9 +48,10 @@ func (s *IssueSyncer) QueueRepository(q queue.Queue, owner, repo string) error { | |||
return err | |||
} | |||
|
|||
logger.Infof("queue request") | |||
l := logger.With(log.Fields{"issue": i.GetNumber()}) | |||
l.Infof("queue request") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe use debug
level here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just extended what we had. I'm ok changing everything to debug, and adding LOG_LEVEL=debug to the compose.
Is that preferable?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would love to keep starting to publish queue jobs type:reporitory/issues/...
, finished to publish queue jobs type:reporitory/issues/...
as INFO. So we can see what it is doing. But change queue request
to DEBUG so it doesn't spam a lot by default.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Done. Force-pushed since it's a small change.
Signed-off-by: Carlos Martín <carlos.martin.sanchez@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thank you
Some extra logs to see what's going on and detect problems.