Skip to content

feat: metadata propagation through events#12273

Draft
jvillafanez wants to merge 1 commit intomasterfrom
event_propagation
Draft

feat: metadata propagation through events#12273
jvillafanez wants to merge 1 commit intomasterfrom
event_propagation

Conversation

@jvillafanez
Copy link
Copy Markdown
Member

@jvillafanez jvillafanez commented Apr 30, 2026

Requires owncloud/reva#582

NOTE:

  • "eventhistory" and "sse" services should be included for completion. However, they don't contact any other service, and there isn't anything worthy to report for tracing. At least the "eventhistory" needs access to the store, but we aren't tracing it at the moment and it will need refactoring.
  • "antivirus" service uses a HTTP client. This might need to be rechecked because it might affect the propagation.

Known issue: #12281 (it should only affect to the tracing, the metadata propagation should work fine)

Description

Related Issue

  • Fixes <issue_link>

Motivation and Context

How Has This Been Tested?

  • test environment:
  • test case 1:
  • test case 2:
  • ...

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised:

@jvillafanez jvillafanez self-assigned this Apr 30, 2026
@update-docs
Copy link
Copy Markdown

update-docs Bot commented Apr 30, 2026

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@jvillafanez jvillafanez force-pushed the event_propagation branch from 00d2353 to 776ce5c Compare May 5, 2026 12:35
return
}

ctx := context.Background()
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Need to double-check if we can remove this. The potential problem is if the context is canceled.
If we require the goroutine below to finish, we'll need a new context, but we'll need to move the metadata from one context to another.

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.

1 participant