Skip to content

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Sep 2, 2025

Bumps sigs.k8s.io/controller-tools from 0.18.0 to 0.19.0.

Release notes

Sourced from sigs.k8s.io/controller-tools's releases.

v0.19.0

What's Changed

Misc

envtest

Dependency bumps

New Contributors

Full Changelog: kubernetes-sigs/controller-tools@v0.18.0...v0.19.0

Commits
  • 257e3a0 Merge pull request #1268 from sbueringer/pr-promote-envtest-1.34
  • 2b454ea Promotion of envtest release for Kubernetes v1.34.0
  • 650fc15 Merge pull request #1266 from sbueringer/pr-bump-v0.34
  • 3b0dd92 Merge pull request #1267 from sbueringer/pr-envtest-0.34
  • 1640706 Release envtest v1.34.0
  • 36f98ef Bump to k8s.io/* v0.34
  • df911f1 Merge pull request #1264 from kubernetes-sigs/dependabot/go_modules/all-go-mo...
  • 767291d 🌱 Bump github.com/onsi/gomega
  • b624019 Merge pull request #1261 from mjlshen/webhook-testdata
  • d53aa90 Generate DeepCopy methods for testdata/cronjob
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

@dependabot dependabot bot added dependencies Pull requests that update a dependency file go Pull requests that update Go code labels Sep 2, 2025
@k8s-ci-robot
Copy link

Hi @dependabot[bot]. Thanks for your PR.

I'm waiting for a etcd-io member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@dependabot dependabot bot force-pushed the dependabot/go_modules/tools/mod/sigs.k8s.io/controller-tools-0.19.0 branch from 23048a2 to 3130b49 Compare September 2, 2025 09:46
@ahrtr
Copy link
Member

ahrtr commented Sep 2, 2025

/ok-to-test

@ahrtr
Copy link
Member

ahrtr commented Sep 2, 2025

@hwdef the failure seems to be related to #198
can you please take a look?

@ahrtr
Copy link
Member

ahrtr commented Sep 7, 2025

/rebase

@ahrtr
Copy link
Member

ahrtr commented Sep 7, 2025

@dependabot rebase

@dependabot dependabot bot force-pushed the dependabot/go_modules/tools/mod/sigs.k8s.io/controller-tools-0.19.0 branch from 3130b49 to 09bd2c8 Compare September 7, 2025 12:59
@hwdef
Copy link
Contributor

hwdef commented Sep 8, 2025

@hwdef the failure seems to be related to #198 can you please take a look?

Sorry, I just saw this — I get too many GitHub notifications and it got buried.

@ahrtr
Copy link
Member

ahrtr commented Sep 8, 2025

@dependabot rebase

Bumps [sigs.k8s.io/controller-tools](https://github.com/kubernetes-sigs/controller-tools) from 0.18.0 to 0.19.0.
- [Release notes](https://github.com/kubernetes-sigs/controller-tools/releases)
- [Changelog](https://github.com/kubernetes-sigs/controller-tools/blob/main/envtest-releases.yaml)
- [Commits](kubernetes-sigs/controller-tools@v0.18.0...v0.19.0)

---
updated-dependencies:
- dependency-name: sigs.k8s.io/controller-tools
  dependency-version: 0.19.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot dependabot bot force-pushed the dependabot/go_modules/tools/mod/sigs.k8s.io/controller-tools-0.19.0 branch from 09bd2c8 to 485a7bc Compare September 8, 2025 19:53
@k8s-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ahrtr, dependabot[bot]

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ahrtr ahrtr merged commit 71fb5d3 into main Sep 8, 2025
6 checks passed
@dependabot dependabot bot deleted the dependabot/go_modules/tools/mod/sigs.k8s.io/controller-tools-0.19.0 branch September 8, 2025 20:33
@ivanvc
Copy link
Member

ivanvc commented Sep 9, 2025

Noting that this pull request broke tools/mod tidiness. I reverted this merge and tried bumping the version locally, but it still fails.

$ go mod tidy 
go: go.etcd.io/etcd-operator/tools/mod imports
        sigs.k8s.io/controller-tools/cmd/controller-gen imports
        sigs.k8s.io/controller-tools/pkg/crd tested by
        sigs.k8s.io/controller-tools/pkg/crd.test imports
        k8s.io/apiextensions-apiserver/pkg/apis/apiextensions/validation imports
        github.com/google/cel-go/cel imports
        google.golang.org/genproto/googleapis/api/expr/v1alpha1: ambiguous import: found package google.golang.org/genproto/googleapis/api/expr/v1alpha1 in multiple modules:
        google.golang.org/genproto v0.0.0-20220519153652-3a47de7e79bd (/home/ivan/.local/share/asdf/installs/golang/1.25.0/packages/pkg/mod/google.golang.org/[email protected]/googleapis/api/expr/v1alpha1)
        google.golang.org/genproto/googleapis/api v0.0.0-20250603155806-513f23925822 (/home/ivan/.local/share/asdf/installs/golang/1.25.0/packages/pkg/mod/google.golang.org/genproto/googleapis/[email protected]/expr/v1alpha1)
go: go.etcd.io/etcd-operator/tools/mod imports
        sigs.k8s.io/controller-tools/cmd/controller-gen imports
        sigs.k8s.io/controller-tools/pkg/crd tested by
        sigs.k8s.io/controller-tools/pkg/crd.test imports
        k8s.io/apiextensions-apiserver/pkg/apis/apiextensions/validation imports
        k8s.io/apiserver/pkg/util/webhook imports
        k8s.io/component-base/tracing imports
        go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracegrpc imports
        google.golang.org/genproto/googleapis/rpc/errdetails: ambiguous import: found package google.golang.org/genproto/googleapis/rpc/errdetails in multiple modules:
        google.golang.org/genproto v0.0.0-20220519153652-3a47de7e79bd (/home/ivan/.local/share/asdf/installs/golang/1.25.0/packages/pkg/mod/google.golang.org/[email protected]/googleapis/rpc/errdetails)
        google.golang.org/genproto/googleapis/rpc v0.0.0-20250603155806-513f23925822 (/home/ivan/.local/share/asdf/installs/golang/1.25.0/packages/pkg/mod/google.golang.org/genproto/googleapis/[email protected]/errdetails)
go: go.etcd.io/etcd-operator/tools/mod imports
        sigs.k8s.io/controller-tools/cmd/controller-gen imports
        sigs.k8s.io/controller-tools/pkg/crd tested by
        sigs.k8s.io/controller-tools/pkg/crd.test imports
        k8s.io/apiextensions-apiserver/pkg/apis/apiextensions/validation imports
        k8s.io/apiserver/pkg/util/webhook imports
        k8s.io/apiserver/pkg/server/egressselector imports
        google.golang.org/grpc imports
        google.golang.org/grpc/internal/status imports
        google.golang.org/genproto/googleapis/rpc/status: ambiguous import: found package google.golang.org/genproto/googleapis/rpc/status in multiple modules:
        google.golang.org/genproto v0.0.0-20220519153652-3a47de7e79bd (/home/ivan/.local/share/asdf/installs/golang/1.25.0/packages/pkg/mod/google.golang.org/[email protected]/googleapis/rpc/status)
        google.golang.org/genproto/googleapis/rpc v0.0.0-20250603155806-513f23925822 (/home/ivan/.local/share/asdf/installs/golang/1.25.0/packages/pkg/mod/google.golang.org/genproto/googleapis/[email protected]/status)
go: go.etcd.io/etcd-operator/tools/mod imports
        sigs.k8s.io/controller-tools/cmd/controller-gen imports
        sigs.k8s.io/controller-tools/pkg/crd tested by
        sigs.k8s.io/controller-tools/pkg/crd.test imports
        k8s.io/apiextensions-apiserver/pkg/apis/apiextensions/validation imports
        k8s.io/apiserver/pkg/util/webhook imports
        k8s.io/component-base/tracing imports
        go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracegrpc imports
        go.opentelemetry.io/proto/otlp/collector/trace/v1 imports
        github.com/grpc-ecosystem/grpc-gateway/v2/runtime imports
        google.golang.org/genproto/googleapis/api/httpbody: ambiguous import: found package google.golang.org/genproto/googleapis/api/httpbody in multiple modules:
        google.golang.org/genproto v0.0.0-20220519153652-3a47de7e79bd (/home/ivan/.local/share/asdf/installs/golang/1.25.0/packages/pkg/mod/google.golang.org/[email protected]/googleapis/api/httpbody)
        google.golang.org/genproto/googleapis/api v0.0.0-20250603155806-513f23925822 (/home/ivan/.local/share/asdf/installs/golang/1.25.0/packages/pkg/mod/google.golang.org/genproto/googleapis/[email protected]/httpbody)

@ivanvc ivanvc mentioned this pull request Sep 9, 2025
@ahrtr
Copy link
Member

ahrtr commented Sep 9, 2025

thx, that's why we need a workflow to verify it.

Let's stop bumping any deps before the workflow is ready.

@ivanvc
Copy link
Member

ivanvc commented Sep 9, 2025

The problem is that there's no current solution for this. If we want the check from #212 to be green, we need to revert this pull request (and possibly update it in the next controller-tools version bump).

ivanvc added a commit to ivanvc/etcd-operator that referenced this pull request Sep 10, 2025
…ules/tools/mod/sigs.k8s.io/controller-tools-0.19.0"

This reverts commit 71fb5d3, reversing
changes made to 89b8b94.

Signed-off-by: Ivan Valdes <[email protected]>
ivanvc added a commit to ivanvc/etcd-operator that referenced this pull request Sep 10, 2025
Regenerate go.mod and go.sum, so go mod tidy has a clean exit.

Properly updates the dependency identified by dependabot in pull
request etcd-io#204.

Signed-off-by: Ivan Valdes <[email protected]>
ivanvc added a commit to ivanvc/etcd-operator that referenced this pull request Sep 10, 2025
Adds google.golang.org/genproto/googleapis/api and
google.golang.org/genproto/googleapis/rpc as indirect dependencies, so
go mod has a clean exit.

Follow up from pull request etcd-io#204.

Signed-off-by: Ivan Valdes <[email protected]>
@ahrtr
Copy link
Member

ahrtr commented Sep 10, 2025

After merging #214, we should be good?

@ivanvc
Copy link
Member

ivanvc commented Sep 10, 2025

After merging #214, we should be good?

Yes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved dependencies Pull requests that update a dependency file go Pull requests that update Go code ok-to-test size/L
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants