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

Porting k8s processor from the Omnition distribution #92

Merged
merged 1 commit into from
Jan 16, 2020
Merged

Porting k8s processor from the Omnition distribution #92

merged 1 commit into from
Jan 16, 2020

Conversation

owais
Copy link
Contributor

@owais owais commented Jan 8, 2020

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

Copy link
Member

@tigrannajaryan tigrannajaryan left a 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
Copy link
Member

Choose a reason for hiding this comment

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

Why this replacement?

Copy link
Contributor Author

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

Copy link
Member

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.

Copy link
Contributor

@pjanotti pjanotti left a 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)

processor/k8sprocessor/config.go Outdated Show resolved Hide resolved
processor/k8sprocessor/factory_test.go Show resolved Hide resolved
processor/k8sprocessor/doc.go Show resolved Hide resolved
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.
@tigrannajaryan tigrannajaryan merged commit f89fe6d into open-telemetry:master Jan 16, 2020
@owais owais deleted the k8s-processor branch January 16, 2020 17:50
mxiamxia referenced this pull request in mxiamxia/opentelemetry-collector-contrib Jul 22, 2020
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.
ljmsc referenced this pull request in ljmsc/opentelemetry-collector-contrib Feb 21, 2022
bogdandrutu pushed a commit that referenced this pull request May 12, 2022
jj22ee pushed a commit to jj22ee/opentelemetry-collector-contrib that referenced this pull request Sep 21, 2023
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.

3 participants