Skip to content

Commit abf8bea

Browse files
committed
DX-46221: Deprecate 'local' distStorage option for Helm deployments.
- Deprecates support for 'local' distStorage type for Dremio release 21.0.0 or newer. Change-Id: I2e178aff5a0151d26249db05c6b97430e6404a96
1 parent 47514a4 commit abf8bea

4 files changed

Lines changed: 14 additions & 3 deletions

File tree

charts/dremio_v2/README.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ If you are upgrading from the previous Helm chart for Dremio, please see the [Mi
99
* Ensure that you have an existing Kubernetes cluster.
1010
* Ensure that Helm 3 is set up on a local machine.
1111
* Ensure that a local kubectl is configured to access your Kubernetes cluster.
12+
* Ensure that you have cloud storage available (required for 21.0.0+), see [here](./docs/Values-Reference.md#diststoragetype) for more details.
1213

1314
## Procedure
1415

charts/dremio_v2/config/dremio.conf

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,14 @@
1414
# limitations under the License.
1515
#
1616

17+
{{- if (eq $.Values.distStorage.type "local") }}
18+
{{- if eq $.Values.imageTag "latest" }}
19+
{{- fail "\n\n** ERROR: The 'local' distStorage type is no longer supported in Dremio 21.0.0 or later. Please configure another distStorage provider to deploy Dremio. (Note: 'latest' is presumed to be 21.0.0+.) **\n" -}}
20+
{{- else if ge ((semver $.Values.imageTag).Major) 21 }}
21+
{{- fail "\n\n** ERROR: The 'local' distStorage type is no longer supported in Dremio 21.0.0 or later. Please configure another distStorage provider to deploy Dremio. **\n" -}}
22+
{{- end }}
23+
{{- end }}
24+
1725
paths: {
1826
# Local path for dremio to store data.
1927
local: ${DREMIO_HOME}"/data"

charts/dremio_v2/docs/Values-Reference.md

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -997,7 +997,9 @@ Type: String
997997

998998
By default, this value is set to `local`.
999999

1000-
The valid values for `distStorage.type` are `local` (not recommended), `aws`, `azure`, `azureStorage` or `gcp`. For specific configuration values for each, see the associated sections:
1000+
***Note***: 'local' has been deprecated in Dremio 21.0.0+.
1001+
1002+
The valid values for `distStorage.type` are `local` (not recommended, not supported in Dremio 21.0.0+), `aws`, `azure`, `azureStorage` or `gcp`. For specific configuration values for each, see the associated sections:
10011003

10021004
* `aws` (S3): [AWS S3](#aws-s3)
10031005
* `azure` (Azure ADLS Gen 1): [Azure ADLS Gen 1](#azure-adls-gen-1)

charts/dremio_v2/values.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -279,9 +279,9 @@ zookeeper:
279279
# Control where uploaded files are stored for Dremio.
280280
# For more information, see https://docs.dremio.com/deployment/distributed-storage.html
281281
distStorage:
282-
# The supported distributed storage types are: local, aws, azure, gcp, or azureStorage.
282+
# The supported distributed storage types are: local (<21.0.0 only), aws, azure, gcp, or azureStorage.
283283
#
284-
# local: Not recommended for production use. When using local, dist-caching is disabled.
284+
# local: (<21.0.0 only) Not recommended for production use. When using local, dist-caching is disabled.
285285
# aws: AWS S3, additional parameters required, see "aws" section.
286286
# azure: ADLS Gen 1, additional parameters required, see "azure" section.
287287
# azureStorage: Azure Storage Gen2, additional paramters required, see "azureStorage" section.

0 commit comments

Comments
 (0)