Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rebase: bump sigs.k8s.io/controller-runtime from 0.17.3 to 0.18.0 in the k8s-dependencies group #4595

Closed

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Apr 29, 2024

Bumps the k8s-dependencies group with 1 update: sigs.k8s.io/controller-runtime.

Updates sigs.k8s.io/controller-runtime from 0.17.3 to 0.18.0

Release notes

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

v0.18.0

⚠️ Breaking Changes

  • Bump to k8s.io/* v1.30 (#2693 #2754 #2765 #2776 #2786)
  • Remove deprecated v1alpha1.ControllerManagerConfiguration (#2648)
  • admission.Decoder is now an interface (#2736)
  • Source, Event, Predicate, Handler: Add generics support (#2783 #2796)
  • client: Fix SubResourceCreateOptions signature in subresource client (#2766)

✨ New Features

  • cache: Add TransformStripManagedFields transform func (#2791)
  • client: Add client.WithFieldOwner to configure client-wide FieldManager (#2771 #2777)
  • controller: Add NewQueue option (#2767)
  • manager: Export HTTP server runnable implementation (#2473)
  • metrics/server: Add ListenConfig option (#2519)

🐛 Bug Fixes

  • builder/webhook: Return error if For() is used multiple times (#2740)
  • cache: Keep selectors when byObject.Namespaces is defaulted (#2747)
  • cache: Prevent race when informers are started more than once (#2758)
  • fake client: Allow fakeclient to patch CR with no resourceVersion (#2725)
  • fake client: Do not consider an apply patch to be a strategic merge patch (#2679)
  • manager: Prevent leader election when shutting down a non-elected manager (#2724)
  • manager: Runnable group should check if stopped before enqueueing (#2757)
  • restmapper: Clean restmapper cache if a version is notFound (#2663)
  • restmapper: Fix cache invalidation (#2687)

🌱 Others

  • ci: Add OSSF scorecard action (#2714)
  • ci: Improve github actions dependencies versions and permissions (#2715)
  • ci: Pin checkout action in golangci-lint action, bump checkout action, use consistent tag format (#2729)
  • ci: Update golangci-lint to v1.57.2 (#2708 #2751)
  • ci: Update scorecard github action (#2728)
  • ci: Use go-install for versioned dependencies (#2710)
  • envtest: WaitForDefaultNamespace while starting up envtest (#2668)
  • owners: Cleanup owners files (#2730)
  • owners: Remove outdated testing framework approvers (#2709)
  • predicate: Compare labels and annotations using maps.Equal (#2705)
  • typos: Fix typo of CacheReader comment (#2773)
  • typos: Fix typo in channel option (#2792)
  • typos: Minor typo fixes in docstrings (#2727)

📖 Additionally, there have been 3 contributions to our documentation and book. (#2712, #2770, #2789)

Dependencies

... (truncated)

Commits
  • ed81fa6 Merge pull request #2796 from alvaroaleman/mark
  • b35cd6b 📖 Mark Typed Handlers as experimental
  • b74908f Merge pull request #2793 from kubernetes-sigs/dependabot/github_actions/actio...
  • a9db208 Merge pull request #2794 from kubernetes-sigs/dependabot/github_actions/actio...
  • 757ae66 🌱 Bump actions/upload-artifact from 4.3.1 to 4.3.3
  • 18ae9e0 🌱 Bump actions/checkout from 4.1.2 to 4.1.3
  • 4f00207 Merge pull request #2792 from sbueringer/pr-fix-typo
  • 9fb4913 Fix typo in channel option
  • 5823d1b Merge pull request #2791 from alvaroaleman/add-default-transform
  • ae0f6ab Merge pull request #2783 from alvaroaleman/compatible-generics
  • Additional commits viewable in compare view

Dependabot compatibility score

You can trigger a rebase of this PR 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 <dependency name> major version will close this group update PR and stop Dependabot creating any more for the specific dependency's major version (unless you unignore this specific dependency's major version or upgrade to it yourself)
  • @dependabot ignore <dependency name> minor version will close this group update PR and stop Dependabot creating any more for the specific dependency's minor version (unless you unignore this specific dependency's minor version or upgrade to it yourself)
  • @dependabot ignore <dependency name> will close this group update PR and stop Dependabot creating any more for the specific dependency (unless you unignore this specific dependency or upgrade to it yourself)
  • @dependabot unignore <dependency name> will remove all of the ignore conditions of the specified dependency
  • @dependabot unignore <dependency name> <ignore condition> will remove the ignore condition of the specified dependency and ignore conditions

Bumps the k8s-dependencies group with 1 update: [sigs.k8s.io/controller-runtime](https://github.com/kubernetes-sigs/controller-runtime).


Updates `sigs.k8s.io/controller-runtime` from 0.17.3 to 0.18.0
- [Release notes](https://github.com/kubernetes-sigs/controller-runtime/releases)
- [Changelog](https://github.com/kubernetes-sigs/controller-runtime/blob/main/RELEASE.md)
- [Commits](kubernetes-sigs/controller-runtime@v0.17.3...v0.18.0)

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

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot added the rebase update the version of an external component label Apr 29, 2024
go.mod Show resolved Hide resolved
go.mod Show resolved Hide resolved
@nixpanic nixpanic force-pushed the dependabot/go_modules/k8s-dependencies-05f7cb983f branch from 298f73a to 81044ed Compare April 30, 2024 12:52
Signed-off-by: Niels de Vos <ndevos@ibm.com>
Signed-off-by: Niels de Vos <ndevos@ibm.com>
Signed-off-by: Niels de Vos <ndevos@ibm.com>
controller-runtime changed the format of the controller.Watch()
function, so that needs adjusting.

See-also: kubernetes-sigs/controller-runtime#2783
Signed-off-by: Niels de Vos <ndevos@ibm.com>
@nixpanic nixpanic force-pushed the dependabot/go_modules/k8s-dependencies-05f7cb983f branch from 81044ed to 1b7437d Compare April 30, 2024 12:59
@nixpanic nixpanic requested review from Madhu-1 and a team April 30, 2024 14:09
nixpanic
nixpanic previously approved these changes Apr 30, 2024
@nixpanic nixpanic requested a review from a team May 1, 2024 08:38
iPraveenParihar
iPraveenParihar previously approved these changes May 2, 2024
go.mod Outdated
Comment on lines 3 to 4
go 1.22

toolchain go1.22.2
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

$]cat go.mod |grep -i "1.22"
go 1.22
toolchain go1.22.2
	code.cloudfoundry.org/gofileutils => github.com/cloudfoundry/gofileutils v0.0.0-20170111115228-4d0c80011a0f
[🎩︎]mrajanna@li-2cfbef4c-22d9-11b2-a85c-a3e4a93c405f ceph-csi $]go mod tidy
[🎩︎]mrajanna@li-2cfbef4c-22d9-11b2-a85c-a3e4a93c405f ceph-csi $]git diff
diff --git a/go.mod b/go.mod
index 076fc38c7..92ce83ca3 100644
--- a/go.mod
+++ b/go.mod
@@ -1,6 +1,7 @@
 module github.com/ceph/ceph-csi
 
-go 1.22
+go 1.22.0
+
 toolchain go1.22.2
 
 require (
[🎩︎]mrajannaceph-csi $]go version
go version go1.22.2 linux/amd64

@nixpanic when i ran go mod tidy above is getting reverted back.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What version of Go do you use?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i used 1.22.2

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Strange, what do you recommend then?

Have the minor version included, as it might get added by some commands again anyway?

go 1.22.0

toolchain go1.22.2

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we can have both or drop toolchain, if we are going with both lets keep the version same in both (which is nothing both dropping toolchain internally)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Now without toolchain. I guess go mod tidy added that at one point. Maybe it is not required anymore?

Makefile Outdated Show resolved Hide resolved
Signed-off-by: Niels de Vos <ndevos@ibm.com>
@nixpanic nixpanic force-pushed the dependabot/go_modules/k8s-dependencies-05f7cb983f branch from aa8be9b to 886f5dd Compare May 2, 2024 08:50
@mergify mergify bot dismissed stale reviews from iPraveenParihar and nixpanic May 2, 2024 08:50

Pull request has been modified.

@nixpanic nixpanic requested a review from Madhu-1 May 2, 2024 08:51
@nixpanic
Copy link
Member

nixpanic commented May 2, 2024

@Mergifyio queue

Copy link
Contributor

mergify bot commented May 2, 2024

queue

🛑 The pull request has been removed from the queue default

The queue conditions cannot be satisfied due to failing checks.

You can take a look at Queue: Embarked in merge queue check runs for more details.

In case of a failure due to a flaky test, you should first retrigger the CI.
Then, re-embark the pull request into the merge queue by posting the comment
@mergifyio refresh on the pull request.

@mergify mergify bot added the ok-to-test Label to trigger E2E tests label May 2, 2024
@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/k8s-e2e-external-storage/1.29

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/k8s-e2e-external-storage/1.28

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/upgrade-tests-cephfs

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e-helm/k8s-1.29

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e-helm/k8s-1.28

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/upgrade-tests-rbd

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e/k8s-1.29

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e/k8s-1.28

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/k8s-e2e-external-storage/1.27

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e-helm/k8s-1.27

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e/k8s-1.27

@ceph-csi-bot ceph-csi-bot removed the ok-to-test Label to trigger E2E tests label May 2, 2024
@nixpanic
Copy link
Member

nixpanic commented May 2, 2024

Several e2e failures:

CephFS: deployment status: expected replica count errors seem to be cause by an existing directory on the workernode where the CephFS PV should get mounted.

RBD: expected pvcName "rbd-pvc-new" got "rbd-pvc" suggests that the creation of the new PVC was not done, the re-use of the pvcObj at e2e/rbd.go:627 may not be possible anymore. This kind of re-use is ugly in any case, and should be cleaned up.

@iPraveenParihar
Copy link
Contributor

/test ci/centos/mini-e2e/k8s-1.27

@iPraveenParihar
Copy link
Contributor

@dependabot rebase

Copy link
Contributor Author

dependabot bot commented on behalf of github May 21, 2024

Looks like this PR has been edited by someone other than Dependabot. That means Dependabot can't rebase it - sorry!

If you're happy for Dependabot to recreate it from scratch, overwriting any edits, you can request @dependabot recreate.

@iPraveenParihar
Copy link
Contributor

this can be closed, as the updates are in #4614

cc @Madhu-1

Copy link
Contributor Author

dependabot bot commented on behalf of github May 27, 2024

Looks like sigs.k8s.io/controller-runtime is updatable in another way, so this is no longer needed.

@dependabot dependabot bot closed this May 27, 2024
@dependabot dependabot bot deleted the dependabot/go_modules/k8s-dependencies-05f7cb983f branch May 27, 2024 20:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rebase update the version of an external component
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants