-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Porting k8s processor from the Omnition distribution #92
Conversation
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.
I reviewed and approved this earlier on Omnition fork.
Approving here too.
go.mod
Outdated
replace k8s.io/client-go => k8s.io/client-go v0.0.0-20190620085101-78d2af792bab | ||
|
||
replace github.com/open-telemetry/opentelemetry-collector => github.com/owais/opentelemetry-collector v0.1.1-0.20200108145118-7d7e1c8f7254 |
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.
Why this replacement?
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.
Needed for CI until the other PR gets merged upstream
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.
Please merge this only after upstream is merged and this replacement is removed.
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.
LGTM (I had reviewed this code in the past, just did a check regarding the docs)
Introducing a new processor to enable automatic tagging of k8s resources K8S processor discovers running pods in a cluster based on some discovery rules, extracts metadata from such pods using user specified extraction rules, and adds the extracted metadata as attributes to all spans. The processor uses the originating IP address of a pod to determine where the spans are coming from.
These 2 functions had error on their return but never returned error. This made confuse on caller sites since it gave the impression that spans could be dropped without proper observability.
Introducing a new processor to enable automatic tagging of k8s resources
K8S processor discovers running pods in a cluster based on some
discovery rules, extracts metadata from such pods using user specified
extraction rules, and adds the extracted metadata as attributes to all
spans. The processor uses the originating IP address of a pod to determine
where the spans are coming from.
Depends on: open-telemetry/opentelemetry-collector#326