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

Include JobID in all ActiveJob info logs #28112

Merged
merged 1 commit into from Feb 23, 2017

Conversation

Projects
None yet
3 participants
@bolek
Contributor

bolek commented Feb 22, 2017

Currently we provide the Job ID in logs only related to enqueuing a job.

This adds the job id to the remaining ActiveJob logs when:

  • a job started performing
  • a job ended performing

Providing the job id in those logs will ease searching logs by job id.

Bolek Kurowski
Include JobID in all ActiveJob info logs
Currently we provide the Job ID in logs only related to enqueuing a job.

This adds the job id to the remaining ActiveJob logs when:
 - a job started performing
 - a job ended performing

Providing the job id in those logs will ease searching logs by job id.

@maclover7 maclover7 added the activejob label Feb 22, 2017

@rafaelfranca rafaelfranca merged commit cfa2664 into rails:master Feb 23, 2017

2 checks passed

codeclimate no new or fixed issues
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment