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

added batch job related attributes to OpenTelemetry span #5972

Merged
merged 1 commit into from
Jun 6, 2024

Conversation

mrdnctrk
Copy link
Collaborator

@mrdnctrk mrdnctrk commented May 28, 2024

This PR introduces OpenTelemetry spans named hapifhir.batch_job.execute created by the batch job worker threads.
These spans have the following span attributes related to the batch job:

  • hapifhir.batch_job.definition_id: The name of the job, such as BULK_EXPORT, REINDEX.
  • hapifhir.batch_job.definition_version: The job definition version.
  • hapifhir.batch_job.instance_id: The job id.
  • hapifhir.batch_job.step_id: The name of the step being executed.
  • hapifhir.batch_job.chunk_id: The id of the work chunk being processed. This is not applicable to reduction steps.

close #5980

Copy link

Formatting check succeeded!

Copy link

codecov bot commented May 28, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.42%. Comparing base (497b9f2) to head (7613fc5).
Report is 82 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #5972      +/-   ##
============================================
+ Coverage     83.39%   83.42%   +0.03%     
- Complexity    26927    27117     +190     
============================================
  Files          1681     1694      +13     
  Lines        103965   104733     +768     
  Branches      13189    13275      +86     
============================================
+ Hits          86702    87376     +674     
- Misses        11613    11658      +45     
- Partials       5650     5699      +49     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@mrdnctrk mrdnctrk force-pushed the emre-batch-job-span branch 2 times, most recently from ec7bbe7 to 301525b Compare May 31, 2024 17:43
@mrdnctrk mrdnctrk marked this pull request as ready for review May 31, 2024 18:23
@mrdnctrk mrdnctrk force-pushed the emre-batch-job-span branch 2 times, most recently from b1a0d89 to 806703c Compare June 4, 2024 12:53
@mrdnctrk mrdnctrk force-pushed the emre-batch-job-span branch 2 times, most recently from e502f39 to 589bf5a Compare June 5, 2024 19:40
Copy link
Collaborator

@tadgh tadgh left a comment

Choose a reason for hiding this comment

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

LGTM

@tadgh tadgh merged commit 48c387e into master Jun 6, 2024
65 of 66 checks passed
@tadgh tadgh deleted the emre-batch-job-span branch June 6, 2024 18:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add OpenTelemetry attributes for batch jobs
3 participants