Some nits and pieces (#338)

* Remove Go version check

Signed-off-by: Nelo-T. Wallus <red.brush9525@fastmail.com>
Signed-off-by: Nelo-T. Wallus <n.wallus@sap.com>

* Remove redundant dex config

Signed-off-by: Nelo-T. Wallus <red.brush9525@fastmail.com>
Signed-off-by: Nelo-T. Wallus <n.wallus@sap.com>

* Update go.mod

Signed-off-by: Nelo-T. Wallus <red.brush9525@fastmail.com>
Signed-off-by: Nelo-T. Wallus <n.wallus@sap.com>

* Fix host argument to log entry

Signed-off-by: Nelo-T. Wallus <red.brush9525@fastmail.com>
Signed-off-by: Nelo-T. Wallus <n.wallus@sap.com>

* Use require instead of panicking

Signed-off-by: Nelo-T. Wallus <red.brush9525@fastmail.com>
Signed-off-by: Nelo-T. Wallus <n.wallus@sap.com>

---------

Signed-off-by: Nelo-T. Wallus <red.brush9525@fastmail.com>
Signed-off-by: Nelo-T. Wallus <n.wallus@sap.com>

Author: ntnn

cli/pkg/kubectl/bind-apiservice/plugin/konnector.go

@@ -119,7 +119,7 @@ func (b *BindAPIServiceOptions) deployKonnector(ctx context.Context, config *res
 			return true, nil
 		}
 
-		logger.V(2).Info("Waiting for APIServiceBindings to be served", "error", err, "host", bindClient.RESTClient())
+		logger.V(2).Info("Waiting for APIServiceBindings to be served", "error", err, "host", config.Host)
 		if first {
 			fmt.Fprint(b.Options.IOStreams.ErrOut, "   Waiting for the konnector to be ready")
 			first = false

contrib/kcp/dex-config-dev.yaml

@@ -2,11 +2,11 @@ module github.com/kube-bind/kube-bind/contrib/kcp
 
 go 1.24.0
 
-replace github.com/kube-bind/kube-bind => ../../
-
-replace github.com/kube-bind/kube-bind/sdk => ../../sdk
-
-replace github.com/kube-bind/kube-bind/cli => ../../cli
+replace (
+	github.com/kube-bind/kube-bind => ../../
+	github.com/kube-bind/kube-bind/cli => ../../cli
+	github.com/kube-bind/kube-bind/sdk => ../../sdk
+)
 
 require (
 	github.com/kcp-dev/client-go v0.0.0-20250728134101-0355faa9361b
@@ -145,7 +145,3 @@ replace (
 	k8s.io/sample-cli-plugin => github.com/kcp-dev/kubernetes/staging/src/k8s.io/sample-cli-plugin v0.0.0-20250816165010-ffe1d7c8649b
 	k8s.io/sample-controller => github.com/kcp-dev/kubernetes/staging/src/k8s.io/sample-controller v0.0.0-20250816165010-ffe1d7c8649b
 )
-
-replace sigs.k8s.io/multicluster-runtime => github.com/mjudeikis/sigs-multicluster-runtime v0.0.0-20250818101434-d8ebc45e169b
-
-replace github.com/kcp-dev/multicluster-provider => github.com/mjudeikis/kcp-multicluster-provider v0.0.0-20250818102159-3d31cbb06ebe

contrib/kcp/go.mod

@@ -22,7 +22,3 @@ VERSION=$(grep "go 1." go.mod | sed 's/go //' | sed 's/.0$//')
 grep "FROM golang:" Dockerfile | { ! grep -v "${VERSION}"; } || { echo "Wrong go version in Dockerfile, expected ${VERSION}"; exit 1; }
 grep "go-version:" .github/workflows/*.yaml | { ! grep -v "go-version: v${VERSION}"; } || { echo "Wrong go version in .github/workflows/*.yaml, expected ${VERSION}"; exit 1; }
 grep "golang:" .ko.yaml | { ! grep -v "golang:${VERSION}"; } || { echo "Wrong go version in .ko.yaml, expected ${VERSION}"; exit 1; }
-# Note CONTRIBUTING.md isn't copied in the Dockerfile
-if [ -z "${IGNORE_GO_VERSION}" ]; then
-  go version | { ! grep -v go${VERSION}; } || { echo "Unexpected go version installed, expected ${VERSION}. Use IGNORE_GO_VERSION=1 to skip this check."; exit 1; }
-fi

hack/verify-go-versions.sh

@@ -42,9 +42,7 @@ import (
 
 func StartBackend(t *testing.T, clientConfig *rest.Config, args ...string) (net.Addr, *backend.Server) {
 	signingKey := securecookie.GenerateRandomKey(32)
-	if len(signingKey) == 0 {
-		panic("error creating signing key")
-	}
+	require.NotEmpty(t, signingKey, "error creating signing key")
 
 	return StartBackendWithoutDefaultArgs(t, clientConfig, append([]string{
 		"--oidc-issuer-client-secret=ZXhhbXBsZS1hcHAtc2VjcmV0",