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.

Sign up for GitHub

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

Jump to bottom

Explicitly embed CRDs for installation ONLY in the OKS #1214

Merged
merged 8 commits into from
Jun 10, 2024
Merged

Explicitly embed CRDs for installation ONLY in the OKS #1214

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
4df40a6
Embed crds
jaronoff97 Jun 5, 2024
8eef6e3
bumpy
jaronoff97 Jun 5, 2024
84acd3e
fix syntax
jaronoff97 Jun 5, 2024
aa5e604
add a note
jaronoff97 Jun 6, 2024
f27234d
more notes
jaronoff97 Jun 6, 2024
55e9272
update note
jaronoff97 Jun 6, 2024
03bdfbe
Merge branch 'main' into embed-crds-oks
JaredTan95 Jun 9, 2024
2d92f08
Merge branch 'main' into embed-crds-oks
TylerHelmuth Jun 10, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions charts/opentelemetry-kube-stack/charts/crds/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,9 @@ This chart contains the CRDs for _*installation*_ only right now for the opentel

This approach is inspired by the kube-prometheus-stack approach which you can see discussion on [here](https://github.com/prometheus-community/helm-charts/issues/3548).

> [!NOTE]
> This chart explicitly _does not_ support the conversion webhook that is currently in the opentelemetry-operator chart. This is because the opentelemetry-kube-stack chart will only work with v1beta1 CRDs. If you wish to use v1alpha1 CRDs to install collectors, you must disable the operator installation as part of the opentelemetry-kube-stack and install the operator separately.
jaronoff97 marked this conversation as resolved.
Show resolved Hide resolved

# Upgrade Notes

Right now, upgrades are NOT handled by this chart, however that could change in the future. This is what is run to bring in the CRDs today.
Expand Down
Loading