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

Enable specifying the storage class and access modes of the created PVCs #255

Conversation

elikatsis
Copy link
Member

This PR adds the following two features:

  • Specify storage class for the created PVCs
  • Specify the access modes for the creates PVCs
    It also extends the Kale CLI as well as the lab extensions to set these options.

Note: This PR, allows both a global and a per-PVC configuration for the above, giving priority to the per-PVC one. However, the per-PVC configuration is not available through the lab extension because it looks like it just complicates things.

Closes #51
Closes #161
Closes #207

StefanoFioravanzo and others added 9 commits November 18, 2020 15:04
If the Field's value is `None` don't run the validation process.
Validators are *not* supposed to check if the value is set or not,
we should control that by setting the `required` flag to True.

Signed-off-by: Stefano Fioravanzo <[email protected]>
Reviewed-by: Ilias Katsakioris <[email protected]>
Signed-off-by: Ilias Katsakioris <[email protected]>
Reviewed-by: Stefano Fioravanzo <[email protected]>
Signed-off-by: Ilias Katsakioris <[email protected]>
Reviewed-by: Stefano Fioravanzo <[email protected]>
Signed-off-by: Ilias Katsakioris <[email protected]>
Reviewed-by: Stefano Fioravanzo <[email protected]>
Signed-off-by: Ilias Katsakioris <[email protected]>
Reviewed-by: Stefano Fioravanzo <[email protected]>
Signed-off-by: Ilias Katsakioris <[email protected]>
Reviewed-by: Stefano Fioravanzo <[email protected]>
Signed-off-by: Ilias Katsakioris <[email protected]>
Reviewed-by: Stefano Fioravanzo <[email protected]>
Signed-off-by: Ilias Katsakioris <[email protected]>
Reviewed-by: Stefano Fioravanzo <[email protected]>
@StefanoFioravanzo StefanoFioravanzo force-pushed the feature-volume-select-class-n-modes branch from c27ff1c to 7406451 Compare November 18, 2020 14:05
@StefanoFioravanzo StefanoFioravanzo merged commit 4c39895 into kubeflow-kale:master Nov 18, 2020
@StefanoFioravanzo StefanoFioravanzo deleted the feature-volume-select-class-n-modes branch November 18, 2020 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants