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

neonvm: drop generated default value in CRD #1313

Merged
merged 2 commits into from
Mar 11, 2025

Conversation

mikhail-sakhnov
Copy link
Contributor

We use kubebuilder:default annotation for scaling mode fields, which effiectively means we never allow it to be non set and that bypasses the logic with --default-cpu-scaling-mode controller argument

Copy link

github-actions bot commented Mar 11, 2025

No changes to the coverage.

HTML Report

Click to open

Copy link
Member

@sharnoff sharnoff left a comment

Choose a reason for hiding this comment

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

LGTM - maybe PR title should say neonvm: and not neonvm-controller:, as it only changes the CRD? Up to you.

@mikhail-sakhnov mikhail-sakhnov changed the title neonvm-controller: drop generated default value neonvm: drop generated default value Mar 11, 2025
@mikhail-sakhnov mikhail-sakhnov changed the title neonvm: drop generated default value neonvm: drop generated default value in CRD Mar 11, 2025
@mikhail-sakhnov mikhail-sakhnov force-pushed the misha/drop-default-scaling-mode-via-kubebuilder branch from 14f5a33 to f5d347d Compare March 11, 2025 14:08
We use kubebuilder:default annotation for scaling mode fields,
which effiectively means we never allow it to be non set and that
bypasses the logic with --default-cpu-scaling-mode controller argument
@mikhail-sakhnov mikhail-sakhnov force-pushed the misha/drop-default-scaling-mode-via-kubebuilder branch from f5d347d to f355bc8 Compare March 11, 2025 14:10
@mikhail-sakhnov mikhail-sakhnov force-pushed the misha/drop-default-scaling-mode-via-kubebuilder branch from 82cfce7 to 90a2da7 Compare March 11, 2025 14:14
@mikhail-sakhnov mikhail-sakhnov merged commit ad51dab into main Mar 11, 2025
33 checks passed
@mikhail-sakhnov mikhail-sakhnov deleted the misha/drop-default-scaling-mode-via-kubebuilder branch March 11, 2025 17:28
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