Skip to content

Controller dep bump - #229

Open
gshikhar2021 wants to merge 2 commits into
redhat-data-and-ai:mainfrom
gshikhar2021:controller-dep-bump
Open

Controller dep bump#229
gshikhar2021 wants to merge 2 commits into
redhat-data-and-ai:mainfrom
gshikhar2021:controller-dep-bump

Conversation

@gshikhar2021

Copy link
Copy Markdown
Contributor
  • Upgraded controller runtime from 0.23.3 to 0.24.1
  • Fix: Deprecated package and golangci linting

- Upgraded controller runtime from 0.23.3 to 0.24.1

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request refactors the Kubernetes API scheme registration by consolidating individual CRD registrations from separate _types.go files into a centralized addKnownTypes function within api/v1alpha1/groupversion_info.go. Additionally, the Go runtime is upgraded to version 1.26.0, and several dependencies are updated, including Kubernetes packages to v0.36.0, controller-runtime to v0.24.1, and OpenTelemetry and Zap logging libraries. No review comments were provided, so there is no feedback to address.

@piyush-garg piyush-garg added this to the v0.8.0 milestone May 25, 2026
piyush-garg
piyush-garg previously approved these changes May 29, 2026
Comment thread api/v1alpha1/chunksgenerator_types.go
Comment thread api/v1alpha1/groupversion_info.go Outdated
Comment thread api/v1alpha1/groupversion_info.go Outdated
@piyush-garg

Copy link
Copy Markdown
Contributor

also should we update all comments mentioning old version like this https://github.com/redhat-data-and-ai/unstructured-data-controller/blob/main/cmd/main.go#L141

@piyush-garg

Copy link
Copy Markdown
Contributor

and along with this, should we bump operator-sdk, kubebuilder etc versions too

@piyush-garg
piyush-garg self-requested a review May 29, 2026 09:24
@piyush-garg

Copy link
Copy Markdown
Contributor

@concaf MR looks good to me, if you want to include in 0.7 we can merge, if you want on 0.8 we can hold it

@concaf

concaf commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

let's see how to handle this in context of operator-sdk / kubebuilder migrations because they have a separate migration guide and support only certain versions of certain libraries like controller-runtime

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants