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
Add otel #3289
base: master
Are you sure you want to change the base?
Add otel #3289
Conversation
🤖 Beep beep! I’m a robot speaking on behalf of @ahmetb. 🤖 Thanks for submitting your kubectl plugin to Krew! In the meanwhile, here are a few tips to make your plugin manifest better:
Thanks for your patience! |
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: agardnerIT The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Welcome @agardnerIT! |
Hi, unless this is an official otel project, we cannot grant this name to your plugin. |
Why? Nothing in the plugin naming guide forbids this? The name What would a valid plugin name be?
|
It just so happens
You'll find inconsistencies before we started enforcing this. We strive to not to grant first-comer advantage to project/vendor names --or terms like It's not too hard to be creative and still have natural-sounding cmds here like: /hold |
@ahmetb I've spoken to @austinlparker (an OpenTelemetry maintainer) and he stated that he doesn't believe the project reserves |
We don't reserve it from a trademark perspective, certainly. |
I would say that I'm not super familiar with this PR but it seems like it only pushes traces? I would probably suggest more specific naming since OpenTelemetry supports metrics, traces, logs, etc. |
Yeah, otel really isn't a good name for this plugin. It's the means to an end, I think we can get creative and find something better here that doesn't give the impression that this is the official kubectl plugin for otel. |
Correct, this plugin (currently) emits only traces, but there's no reason it couldn't emit metrics and logs (good enhancement). In terms of naming, |
I'll suggest a couple more alternatives that are less likely to be taken away by anyone else:
|
The Kubernetes project currently lacks enough contributors to adequately respond to all PRs. This bot triages PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
This PR
otel
pluginThis Plugin
Uses tracepusher to generate OpenTelemetry traces for
kubectl
commands.