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

Add content/en/docs/kubernetes/operator/api.md #6104

Closed

Conversation

CharlieTLe
Copy link
Contributor

@CharlieTLe CharlieTLe commented Jan 29, 2025

Copying from opentelemetry-operator/docs/api.md so that it can accessible from the website.

Related to: open-telemetry/opentelemetry-operator#3681

@CharlieTLe CharlieTLe requested a review from a team as a code owner January 29, 2025 01:37
@opentelemetrybot opentelemetrybot requested a review from a team January 29, 2025 01:38
Copy link
Contributor

@chalin chalin left a comment

Choose a reason for hiding this comment

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

Hi. We can't just use a copy of that file, it'll quickly drift from its canonical source. @open-telemetry/operator-approvers should comment as to whether they'd actually like this to appear on the website, and if so, we'll see how we can setup a means of keeping this copy and the operator repo copy in sync.

@swiatekm
Copy link
Contributor

I don't think it's a good idea to host the CRD definitions on opentelemetry.io, if only because they change between versions. We can solve the original issue in the operator repo without involving this one.

@tiffany76
Copy link
Contributor

Closing this PR since the Operator SIG will handle in their repo. Thanks for being proactive, @CharlieTLe!

@chalin
Copy link
Contributor

chalin commented Feb 4, 2025

@CharlieTLe - I've created the following issue so that we can address your very legitimate concern of wanting to be able to view the API docs :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants