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
Add name check logic to Toleration in ClusterSettings #536
Add name check logic to Toleration in ClusterSettings #536
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One comment on how to prevent saving a bad entry.
<HelperTextItem hasIcon variant="error"> | ||
{notebookTolerationSettings.error} | ||
</HelperTextItem> | ||
)} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you change line 419 to isDisabled={!isSettingsChanged || notebookTolerationSettings.error !== undefined} That should prevent the user from saving bad values.
eca5116
to
e0d693e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dlabaj The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
…#536) * Add name check logic to Toleration in ClusterSettings * Block saving when tolerations are in error
…#536) * Add name check logic to Toleration in ClusterSettings * Block saving when tolerations are in error
…#536) * Add name check logic to Toleration in ClusterSettings * Block saving when tolerations are in error
Resolves: #515
Description
Added the regexp k8s tells us and a similar error message so the Toleration Key is checked in real time.
Screen.Recording.2022-09-12.at.6.42.08.AM.mov
How Has This Been Tested?
Tested on the Toleration key in the Cluster Settings page.
Merge criteria: