Skip to content
This repository was archived by the owner on Feb 8, 2024. It is now read-only.
Merged
Show file tree
Hide file tree
Changes from 3 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -143,7 +143,7 @@ If you have direct access to the underlying Kubernetes Nodes in your cluster, CO

3. Update the solution configuration file to reflect your environment. The most common and expected updates are reflected below:

- Update the namespace you want to deploy CORTX into. The default is "cortx". If the namespace does not exist then it will be created for you. **There is currently a limitation on the maximum length of the namespace to 8 characters.**
- Update the namespace you want to deploy CORTX into. The default is "cortx". If the namespace does not exist then it will be created for you. **There is currently a limitation on the maximum length of the namespace to 20 characters.**

- Update the `deployment_type` with the desired deployment mode. See under [Global Parameters](#global-parameters) for more details.

Expand Down Expand Up @@ -252,7 +252,7 @@ All paths below are prefixed with `solution.` for fully-qualified naming and are

| Name | Description | Default Value |
| ----------------- | ---------------------------------------------------------------------------------------------------------------------------- | ------------- |
| `namespace` | The Kubernetes namespace that all CORTX-related resources will be deployed into. Currently limited to a maximum of 8 characters.
| `namespace` | The Kubernetes namespace that all CORTX-related resources will be deployed into. Currently limited to a maximum of 20 characters.
| `deployment_type` | The type of deployment. This determines which Kubernetes resources are created. Valid values are `standard` and `data-only`. | `standard` |

### Secret parameters
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ fi
### CORTX-29861 Temporary namespace length limitation enforced
### This can be removed once namespaces of nominal length (20+ characters) have been validated repeatedly.
observed_namespace_length=$(yq '.solution.namespace | length' "${solution_yaml}")
maximum_namespace_length=8
maximum_namespace_length=20

if [[ "${observed_namespace_length}" -gt "${maximum_namespace_length}" ]]; then
result_str="The maximum length of the targeted Kubernetes namespace is currently limited to ${maximum_namespace_length} characters. The specified namespace in ${solution_yaml} has a length of ${observed_namespace_length} characters."
Expand Down