upbound / up

The @upbound CLI
Apache License 2.0
49 stars 42 forks source link

Obervability: Wait for OpenTelemetry operator pod to be ready #550

Closed lsviben closed 3 weeks ago

lsviben commented 4 weeks ago

Description of your changes

Adds a wait on the installation step of the OpenTelmetry Operator for the operator pod to be ready.

This fixes the issue in #548 which was failing the Spaces installation as the webhook was not ready to review the Space level OpenTelemetryCollector resource.

Fixes #548

I have:

How has this code been tested

locally built binary with make and then ran up space init

_output/bin/darwin_arm64/up space init --token-file="../key.json" "v1.4.0-rc.0.331.g417a41db" \
  --set "account=bob" \
  --set "features.alpha.observability.enabled=true"
 INFO  Setting defaults for vanilla Kubernetes (type kind)
 WARNING  One or more required prerequisites are not installed:

❌ cert-manager
❌ universal-crossplane
❌ ingress-nginx
❌ provider-kubernetes
❌ provider-helm
❌ opentelemetry-operator

Would you like to install them now? [y/N]: Yes

  ✓   [1/6]: Installing cert-manager
  ✓   [2/6]: Installing universal-crossplane
  ✓   [3/6]: Installing ingress-nginx
  ✓   [4/6]: Installing provider-kubernetes
  ✓   [5/6]: Installing provider-helm
  ✓   [6/6]: Installing opentelemetry-operator
 INFO  Required prerequisites met!
 INFO  Proceeding with Upbound Spaces installation...
  ✓   [1/3]: Creating pull secret upbound-pull-secret
  ✓   [2/3]: Initializing Space components
  ✓   [3/3]: Starting Space Components
  🙌  Your Upbound Space is Ready!

  👀  Next Steps 👇

👉 Check out Upbound Spaces docs @ https://docs.upbound.io/concepts/upbound-spaces