Skip to content

Align latest k8s modules#413

Merged
steiler merged 156 commits intomainfrom
subresources
Apr 16, 2026
Merged

Align latest k8s modules#413
steiler merged 156 commits intomainfrom
subresources

Conversation

@henderiw
Copy link
Copy Markdown
Contributor

  • Align with latest apiServer of Kubernetes and controller runtime

@henderiw henderiw requested a review from a team as a code owner February 18, 2026 04:59
@henderiw henderiw added the safe to test Apply this label for PRs to kick off CI integration testing label Feb 18, 2026
@henderiw henderiw removed safe to test Apply this label for PRs to kick off CI integration testing labels Feb 18, 2026
henderiw and others added 8 commits March 13, 2026 18:26
Co-authored-by: Alexander North <alexandernorth@users.noreply.github.com>
Co-authored-by: Alexander North <alexandernorth@users.noreply.github.com>
Co-authored-by: Alexander North <alexandernorth@users.noreply.github.com>
Co-authored-by: Alexander North <alexandernorth@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

@alexandernorth alexandernorth left a comment

Choose a reason for hiding this comment

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

lgtm

Comment on lines +391 to +396
if newConfigCond.Equal(curConfigCond) &&
newOverallCond.Equal(curOverallCond) &&
(recoverable ||
(current.Status.DeviationGeneration != nil &&
*current.Status.DeviationGeneration == 0)) {
return nil
Copy link
Copy Markdown
Contributor

@alexandernorth alexandernorth Apr 13, 2026

Choose a reason for hiding this comment

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

maybe this expression could be simplified? or explained? Otherwise LGTM

Comment thread artifacts/statefulset-data-server.yaml Outdated
Comment on lines +83 to +84
- name: KUBE_FEATURE_WatchListClient
value: "false"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@henderiw this is defined twice

Comment thread artifacts/deployment-controller.yaml Outdated
Comment on lines +81 to +82
- name: KUBE_FEATURE_WatchListClient
value: "false"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@henderiw defined twice

@steiler steiler merged commit 36ea399 into main Apr 16, 2026
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

safe to test Apply this label for PRs to kick off CI integration testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] ConfigSet, ConfigBlame and RunningConfig leaking into other namespaces

3 participants