Skip to content

feat(extensions): add event tracing extension with correlationid and causationid#1353

Merged
duglin merged 1 commit into
cloudevents:mainfrom
yordis:yordis/causationid-extension
Jul 24, 2025
Merged

feat(extensions): add event tracing extension with correlationid and causationid#1353
duglin merged 1 commit into
cloudevents:mainfrom
yordis:yordis/causationid-extension

Conversation

@yordis
Copy link
Copy Markdown
Contributor

@yordis yordis commented Jul 10, 2025

closes #25

Signed-off-by: Yordis Prieto yordis.prieto@gmail.com

@yordis yordis marked this pull request as ready for review July 10, 2025 17:29
Comment thread cloudevents/extensions/tracing.md Outdated
@duglin
Copy link
Copy Markdown
Collaborator

duglin commented Jul 10, 2025

can you wrap the text at 80 chars?

@duglin
Copy link
Copy Markdown
Collaborator

duglin commented Jul 10, 2025

you'll need to create two files:
cloudevents/extensions/tracing.md: Translation file cloudevents/languages/zh-CN/extensions/tracing.md does not exist
cloudevents/extensions/tracing.md: Translation file cloudevents/languages/he/extensions/tracing.md does not exist

as placeholders for the other languages. See some other extension in those dirs to see a file you can just copy and replace the appropriate text/url in there

@yordis yordis force-pushed the yordis/causationid-extension branch from 623a202 to 14af82c Compare July 10, 2025 20:07
@yordis
Copy link
Copy Markdown
Contributor Author

yordis commented Jul 10, 2025

done

@yordis yordis requested a review from duglin July 11, 2025 20:09
@duglin
Copy link
Copy Markdown
Collaborator

duglin commented Jul 14, 2025

the translation files you added appear to be empty. Please see the other language ones for samples you can copy

Comment thread cloudevents/extensions/tracing.md Outdated
@yordis yordis force-pushed the yordis/causationid-extension branch from 14af82c to 2e0c423 Compare July 14, 2025 18:41
@yordis
Copy link
Copy Markdown
Contributor Author

yordis commented Jul 14, 2025

Renamed to correlation and fix the Chinese and Hebrew files

Comment thread cloudevents/languages/he/extensions/correlation.md Outdated
Comment thread cloudevents/extensions/correlation.md Outdated
Comment thread cloudevents/extensions/correlation.md Outdated
@yordis yordis force-pushed the yordis/causationid-extension branch from 2e0c423 to 008d37e Compare July 14, 2025 19:00
@yordis yordis requested a review from duglin July 14, 2025 19:00
…causationid

closes cloudevents#25

Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>
@yordis yordis force-pushed the yordis/causationid-extension branch from 008d37e to a43f836 Compare July 14, 2025 19:02
@duglin
Copy link
Copy Markdown
Collaborator

duglin commented Jul 17, 2025

On 7/17 call - we agreed with the general direction - one more week for review of the text.

@duglin
Copy link
Copy Markdown
Collaborator

duglin commented Jul 24, 2025

Approved on the 7/24 call

@duglin duglin merged commit e2d119e into cloudevents:main Jul 24, 2025
2 checks passed
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.

causation-id context attribute

2 participants