Skip to content

Add support for Trace-Context#100

Draft
qligier wants to merge 1 commit intoIHE:mainfrom
qligier:ql_trace_context
Draft

Add support for Trace-Context#100
qligier wants to merge 1 commit intoIHE:mainfrom
qligier:ql_trace_context

Conversation

@qligier
Copy link

@qligier qligier commented Feb 23, 2024

Closes #94

📑 Description

This PR adds two codes to the BasicAuditEntityType CodeSystem and the BasicAuditEntityTypesVS ValueSet: Traceparent and Tracestate. The slicing is updated, an exemple is modified to use traceparent instead of XrequestId and the documentation is updated accordingly.

✅ Checks

  • My pull request adheres to the code style of this project
  • My code requires changes to the documentation
  • I have updated the documentation as required
  • All the tests have passed
  • I have selected a committee co-chair to review the PR

ℹ Additional Information

@qligier
Copy link
Author

qligier commented Mar 15, 2024

Hi @JohnMoehrke! What do you think of this proposition? What would be the next step to fix the linked issue?

@JohnMoehrke
Copy link
Contributor

will add this to the next CP call.

@JohnMoehrke
Copy link
Contributor

CP-ITI-1303

@JohnMoehrke
Copy link
Contributor

There is interest, but the changes are big enough to require CP processing. The CP can refer to this Pull-Request, but the CP does need to be drafted with rational for the change.

@JohnMoehrke
Copy link
Contributor

@qligier can you prepare ITI-CP-1303 for a CP Ballot as discussed above?

@JohnMoehrke JohnMoehrke marked this pull request as draft November 14, 2024 13:10
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.

Specify tracing identifiers in AuditEvents

3 participants