You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
When the Host path is invalid we don't add it as a Volume so this should be fixed and made into an error.
Currently the validation seems to be:
switch (key) {
case "hostPath":
break;
So nothing happens. This functionality uses Cockpit's FileAutoComplete component which has no way to give back an error state, except when an error has happened it returns an empty string and set's it's own error state.
As you don't know the previous state, it's hard to determine if there was an issue.
without breaking the FileAutoComplete onChange API, by adding another callback argument we could do:
this.props.onChange('');
if (this.props.onError) {
this.props.onError(error)
};
Copied from @garrett 's comment at #1354 (review)
But this can be handled in a follow-up.
The text was updated successfully, but these errors were encountered: