search

upbound / up

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

up spaces upgrade does not check opentelemetry prerequisite #570

Open lsviben opened 3 days ago

lsviben commented 3 days ago

After installing up space init withouth the observablity flag, and then upgrading with:

up space upgrade --token-file="${SPACES_TOKEN_PATH}" "v${SPACES_VERSION}" \
 --set "account=${UPBOUND_ACCOUNT}" \
 --set "features.alpha.observability.enabled=true" \
 --set "observability.spacesCollector.config.exporters.otlphttp.endpoint=http://otel-lgtm.observability:4318" \
 --set "observability.spacesCollector.config.exportPipeline.logs={otlphttp}"

Getting

▄  Upgrading Space from v1.5.0-rc.2 to v1.5.0-rc.2 (5s)

up: error: space.upgradeCmd.Run(): [resource mapping not found for name: "telemetry-spaces-logs" namespace: "upbound-system" from "": no matches for kind "OpenTelemetryCollector" in version "opentelemetry.io/v1alpha1"
           ensure CRDs are installed first, resource mapping not found for name: "telemetry-spaces" namespace: "" from "": no matches for kind "OpenTelemetryCollector" in version "opentelemetry.io/v1alpha1"
           ensure CRDs are installed first]