Skip to content

fix: Validate the AdminServer.EnableAPIs[] field properly.#263

Merged
hessjcg merged 2 commits intomainfrom
fix-enum-bug
Mar 14, 2023
Merged

fix: Validate the AdminServer.EnableAPIs[] field properly.#263
hessjcg merged 2 commits intomainfrom
fix-enum-bug

Conversation

@hessjcg
Copy link
Copy Markdown
Collaborator

@hessjcg hessjcg commented Mar 10, 2023

Validation generated by kubebuilder:Enum does not work with []string fields because it applies to the entire value, not the elements of the slice.

Validation generated by kubebuilder:Enum does not work with []string fields because
it applies to the entire value, not the elements of the slice.
@hessjcg hessjcg requested a review from a team as a code owner March 10, 2023 16:53
@hessjcg hessjcg changed the title fix: Write code to validate the AdminServer.EnableAPIs[] field. fix: Validate the AdminServer.EnableAPIs[] field properly. Mar 10, 2023
@hessjcg hessjcg requested a review from enocom March 10, 2023 17:30
@hessjcg hessjcg merged commit 115ac32 into main Mar 14, 2023
@hessjcg hessjcg deleted the fix-enum-bug branch March 14, 2023 15:38
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.

2 participants