Skip to content

Conversation

@edif2008
Copy link
Member

@edif2008 edif2008 commented Jun 15, 2023

The steps performed to upgrade the Operator SDK for Go from 1.25.0 to 1.29.0 were the following:

  • Update dependencies in Makefile and config/default/manager_auth_proxy_patch.yaml.

No Go mod dependencies were updated for the following reasons:

  • According to the upgrade to v1.28.0, we needed the following version for the following packages, which we already have or are higher:
    k8s.io/api v0.26.2 
    k8s.io/apiextensions-apiserver v0.26.2 
    k8s.io/apimachinery v0.26.2 
    k8s.io/client-go v0.26.2 
    k8s.io/kubectl v0.26.2 
    sigs.k8s.io/controller-runtime v0.14.5 
  • Updating these to the latest version (v0.27.3 as of this writing) seems to break building the operator with the following error:
    # k8s.io/client-go/applyconfigurations/meta/v1
    vendor/k8s.io/client-go/applyconfigurations/meta/v1/unstructured.go:64:38: cannot use doc (variable of type *"github.com/google/gnostic/openapiv2".Document) as *"github.com/google/gnostic-models/openapiv2".Document value in argument to proto.NewOpenAPIData
    

These are based on the steps presented in upgrading to v1.28.0, since that is the version that has changes relevant to a Go operator (which is our case).

@edif2008 edif2008 force-pushed the update-operator-go-sdk branch from 23e10ea to aaddfd0 Compare June 15, 2023 18:26
@edif2008 edif2008 marked this pull request as ready for review June 15, 2023 18:39
@edif2008 edif2008 merged commit 2dbfc7e into main Jun 15, 2023
@edif2008 edif2008 deleted the update-operator-go-sdk branch June 15, 2023 18:43
@jillianwilson jillianwilson mentioned this pull request Jun 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants