-
Notifications
You must be signed in to change notification settings - Fork 204
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
Introduce Source CRD #2037
Introduce Source CRD #2037
Conversation
fe4ab19
to
ffbe6d9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks mike, really love how this feature is shaping up.
I would like to discuss the name and consider calling it source
to align with the terminology we already use in many places.
04ae41f
to
28acc08
Compare
28acc08
to
8d72fea
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great.
Worth just to rename the PR title and description to Source CRD instead of Instrumentation.
This adds the
Source
CRD that will be used to enable auto instrumentation for a workload (vs the current label approach). Based on design docThis doesn't hook the CRD up to anything yet or enable any functionality based on it. That will be done in a feature branch to collaborate with frontend. This PR is meant to break the change into a logical chunk, but also to unblock other work that can rely on this API type.