-
Notifications
You must be signed in to change notification settings - Fork 65
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
feat: Add ability to set up Che workspace and Postgres pvc class names using arguments. #461
Conversation
…ments. Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
}), | ||
'workspace-pvc-storage-class-name': string({ | ||
description: 'persistent volume(s) storage class name to use to store Eclipse Che workspaces data', | ||
env: 'CHE_INFRA_KUBERNETES_PVC_STORAGE__CLASS__NAME', |
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.
Maybe CHE_INFRA_KUBERNETES_WORKSPACE_PVC_STORAGE__CLASS__NAME
}), | ||
'postgres-pvc-storage-class-name': string({ | ||
description: 'persistent volume storage class name to use to store Eclipse Che Postgres database', | ||
default: '' |
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.
env: 'CHE_INFRA_KUBERNETES_POSTGRESS_PVC_STORAGE__CLASS__NAME',
@@ -321,6 +321,14 @@ error: E_COMMAND_FAILED`) | |||
ctx.isDevfileRegistryDeployed = true | |||
} | |||
|
|||
if (flags['workspace-pvc-storage-class-name']) { | |||
setOptions.push(`--set global.cheWorkspacePVCStorageClassName=${flags['workspace-pvc-storage-class-name']}`) |
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.
Just wondering if we don't face any errors if storage class name is undefined
What does this PR do?
Add ability to set up Che workspace and Postgres pvc class names using arguments.
What issues does this PR fix or reference?
eclipse-che/che#15157
Signed-off-by: Oleksandr Andriienko oandriie@redhat.com