Skip to content

Conversation

@cesnietor
Copy link
Contributor

@cesnietor cesnietor commented Jun 18, 2024

fixes: #1595
fixes: #1298

Since Effect can't be other than NoExecute when Toleration Seconds are set, we need to allow Seconds to be undefined.
We address this by doing the validation in the BE and returning an error.

Test steps

  • Add a Tenant instance defining some Tolerations
  • Set one toleration without seconds
  • Set one with seconds set
  • They should show up on the Settings
  • Go to Settings and check you can also add and update tolerations with and without seconds and exposing the corresponding error.

@cesnietor cesnietor self-assigned this Jun 18, 2024
@cesnietor cesnietor merged commit da7bf70 into minio:master Jun 18, 2024
@djwfyi djwfyi mentioned this pull request Jul 18, 2024
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

3 participants