kube-rs / controller-rs

A kubernetes reference controller with actix-web
Apache License 2.0
282 stars 27 forks source link

0.75 bump #31

Closed clux closed 2 years ago

clux commented 2 years ago

Tested locally before release of 0.75.0

clux commented 2 years ago

Seems failing due to opentelemetry needing protoc for opentelemetry now...

  Caused by:
    process didn't exit successfully: `/home/runner/work/controller-rs/controller-rs/target/debug/build/opentelemetry-proto-54b408751cf9d2b7/build-script-build` (exit status: 101)
    --- stdout
    cargo:rerun-if-changed=src/proto/opentelemetry-proto/opentelemetry/proto/common/v1/common.proto
    cargo:rerun-if-changed=src/proto/opentelemetry-proto/opentelemetry/proto/resource/v1/resource.proto
    cargo:rerun-if-changed=src/proto/opentelemetry-proto/opentelemetry/proto/trace/v1/trace.proto
    cargo:rerun-if-changed=src/proto/opentelemetry-proto/opentelemetry/proto/trace/v1/trace_config.proto
    cargo:rerun-if-changed=src/proto/opentelemetry-proto/opentelemetry/proto/collector/trace/v1/trace_service.proto
    cargo:rerun-if-changed=src/proto/opentelemetry-proto/opentelemetry/proto/metrics/v1/metrics.proto
    cargo:rerun-if-changed=src/proto/opentelemetry-proto/opentelemetry/proto/collector/metrics/v1/metrics_service.proto
    cargo:rerun-if-changed=src/proto/opentelemetry-proto/opentelemetry/proto/logs/v1/logs.proto
    cargo:rerun-if-changed=src/proto/opentelemetry-proto/opentelemetry/proto/collector/logs/v1/logs_service.proto
    cargo:rerun-if-changed=src/proto/opentelemetry-proto

    --- stderr
    thread 'main' panicked at '
    Could not find `protoc` installation and this build crate cannot proceed without
    this knowledge. If `protoc` is installed and this crate had trouble finding
    it, you can set the `PROTOC` environment variable with the specific path to your
    installed `protoc` binary.

will try adding it

clux commented 2 years ago

nightly works. problem here was that clux/muslrust:stable did not have a build that had propagated the latest protoc inclusion via https://github.com/clux/muslrust/pull/106. triggering it first to avoid pinning a nightly.