@vladbologa: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Full PR test history. Your PR dashboard.

There are some module version conflicts, I tried to fix them but without success so far.

We might not be able to merge this until the repo upgrades to k8s 1.24.

The problem is with some v0 otel modules. Others are having similar issues too: open-telemetry/opentelemetry-go#2938

@porridge The helm-operator rebase does currently not work because of a transitive dependency from github.com/cloudflare/cfssl/helpers package caused by an upgrade to kubernetes client 0.24.

The upgrade of this dependency does not look very trivial to me and may require an overwrite in our go.mod.
My suggestion here would be to put the helm-operator rebase into our backlog and reverting the rebase like this:

• Reverting the rebase and force pushing main-backup-2022-09-01
• Rebasing pause-annotation PR ROX-12219: Add support for pause-reconcile annotation helm-operator#29 on master

WDYTH @porridge ?

Error:

go: finding module for package go.opentelemetry.io/otel/semconv
github.com/stackrox/rox/pkg/auth/authproviders/saml imports
	github.com/cloudflare/cfssl/helpers imports
	github.com/google/certificate-transparency-go imports
	go.etcd.io/etcd/v3 imports
	go.etcd.io/etcd/tests/v3/integration imports
	go.etcd.io/etcd/server/v3/embed imports
	go.opentelemetry.io/otel/semconv: module go.opentelemetry.io/otel@latest found (v1.9.0), but does not contain package go.opentelemetry.io/otel/semconv
github.com/stackrox/rox/pkg/auth/authproviders/saml imports
	github.com/cloudflare/cfssl/helpers imports
	github.com/google/certificate-transparency-go imports
	go.etcd.io/etcd/v3 imports
	go.etcd.io/etcd/tests/v3/integration imports
	go.etcd.io/etcd/server/v3/embed imports
	go.opentelemetry.io/otel/exporters/otlp imports
	go.opentelemetry.io/otel/sdk/metric/controller/basic imports
	go.opentelemetry.io/otel/metric/registry: module go.opentelemetry.io/otel/metric@latest found (v0.31.0), but does not contain package go.opentelemetry.io/otel/metric/registry

• what was the motivation for the rebase? Just hygiene, or did something else require it?
• by when do we need to land the pause-reconcile change?

@SimonBaeumer ☝️ ❓

@porridge

• what was the motivation for the rebase? Just hygiene, or did something else require it?

Hygiene, @vladbologa noticed we are behind upstream and tried to rebase. We don't have any hard need to do a rebase on upstream immediatley.

by when do we need to land the pause-reconcile change?

No hard deadline, but at some point it eventually blocks the actual upgrade implementation in the fleetshard-sync. So in 1- 2 weeks it would be great to have it merged.

Thanks, I'd revert the rebase, then. I'll do it, unless you prefer to @SimonBaeumer

Thanks, I'd revert the rebase, then. I'll do it, unless you prefer to @SimonBaeumer

Please go ahead, thank you @porridge for covering this.