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

[DO NOT MERGE] Remove device ID processor #2125

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

edeNFed
Copy link
Contributor

@edeNFed edeNFed commented Jan 4, 2025

As we move to the new webhook mechanism, we no longer need the device id processor (which has many drawbacks such as connectivity to kubelet).
This PR removes it and anything dependent on it.

Before merging this PR the following tasks needs to be completed:

  • Wait a few weeks to allow users to upgrade their Odigos version and restart their pods
  • Communicate this change over Slack
  • Change Python Instrumentation to remove device id
  • Change NodeJS instrumentation to remove device id

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