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

docs: outline go support policy #2839

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

frzifus
Copy link
Member

@frzifus frzifus commented Apr 11, 2024

Description:

Link to tracking Issue(s):

Testing:

Documentation:

@frzifus frzifus added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Apr 11, 2024
@frzifus frzifus requested a review from a team as a code owner April 11, 2024 16:26
@@ -714,50 +714,6 @@ spec:
exporters: [debug]
```

Copy link
Contributor

Choose a reason for hiding this comment

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

We should link to support.md here.

Copy link
Contributor

Choose a reason for hiding this comment

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

+1

support.md Show resolved Hide resolved
support.md Show resolved Hide resolved

| OpenTelemetry Operator | Kubernetes | Cert-Manager |
| ---------------------- | -------------- | ------------ |
| v0.97.0 | v1.23 to v1.29 | v1 |
Copy link
Member

Choose a reason for hiding this comment

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

This needs to be updated

Copy link
Contributor

Choose a reason for hiding this comment

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

yeah the ordering here looks off too...

Signed-off-by: Benedikt Bongartz <bongartz@klimlive.de>
@@ -714,50 +714,6 @@ spec:
exporters: [debug]
```

Copy link
Contributor

Choose a reason for hiding this comment

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

+1


| OpenTelemetry Operator | Kubernetes | Cert-Manager |
| ---------------------- | -------------- | ------------ |
| v0.97.0 | v1.23 to v1.29 | v1 |
Copy link
Contributor

Choose a reason for hiding this comment

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

yeah the ordering here looks off too...


### OpenTelemetry Operator vs. Kubernetes vs. Cert Manager

We strive to be compatible with the widest range of Kubernetes versions as possible, but some changes to Kubernetes itself require us to break compatibility with older Kubernetes versions, be it because of code incompatibilities, or in the name of maintainability. Every released operator will support a specific range of Kubernetes versions, to be determined at the latest during the release.
Copy link
Contributor

Choose a reason for hiding this comment

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

i assume a follow up will be done to close #2798 ?


The OpenTelemetry Operator _might_ work on versions outside of the given range, but when opening new issues, please make sure to test your scenario on a supported version.

| OpenTelemetry Operator | Kubernetes | Cert-Manager |
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we mention in the RELEASE.md that this file needs to be upgraded?

@frzifus
Copy link
Member Author

frzifus commented May 23, 2024

Ouch, let me update this...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Define a policy how the go version gets updated
5 participants