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

[operator] - friction on install #550

Closed
puckpuck opened this issue Dec 8, 2022 · 2 comments
Closed

[operator] - friction on install #550

puckpuck opened this issue Dec 8, 2022 · 2 comments
Assignees
Labels
chart:operator Issue related to opentelemetry-operator helm chart Stale

Comments

@puckpuck
Copy link
Contributor

puckpuck commented Dec 8, 2022

Ran into 2 friction issues when installing the operator:

  1. The operator requires a cert-manager and several cert CRDs to be setup before it can be installed. There are instructions in the readme to do so, forcing the user to go elsewhere, have to install another helm repo or run a kubectl apply script. This makes for a not so great experience. We could clean this up by including the cert-manager as a sub chart, allowing the user to get setup quickly.

  2. When installing the chart I came across this error:

√ otel-operator 🤘 : helm install opentelemetry-operator open-telemetry/opentelemetry-operator                                                           
Error: INSTALLATION FAILED: create: failed to create: namespaces "opentelemetry-operator-system" not found

After creating the mentioned namespace it worked fine, but this feels like it should be handled with installation instructions and/or by the Helm chart itself.

@TylerHelmuth TylerHelmuth added the chart:operator Issue related to opentelemetry-operator helm chart label Dec 8, 2022
@TylerHelmuth
Copy link
Member

/cc @Allex1

@Allex1 Allex1 self-assigned this Dec 8, 2022
@Allex1
Copy link
Contributor

Allex1 commented Dec 8, 2022

@puckpuck Thanks for opening this.

  1. I think we should disable cert-manager and use self signed cert by default while deploying cert-manager under a feature flag. Would you be willing to look at this?
  2. helm has the --create-namespace flag and we could probably add it in the readme but it's a newer flag and I haven't seen it yet on other helm git repos.

@github-actions github-actions bot added the Stale label Dec 8, 2024
@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Jan 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chart:operator Issue related to opentelemetry-operator helm chart Stale
Projects
None yet
Development

No branches or pull requests

3 participants