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

Make the cache path configurable (and reside on the executor's PV!) #12

Open
wants to merge 24 commits into
base: master
Choose a base branch
from

Conversation

doernemt
Copy link

The current version of the Helm Chart does not allow you to configure the path for CCC. It is fixed to /var/lib/dremio which is unfortunate because the folder is located in the overload filesystem.

Instead, one should use a folder below /opt/dremio/data, because that is where the Persistent Volume is mounted.

The PR fixed both by introducing two variables and setting the default folder to /opt/dremio/data/cache

Best, Tim from Software AG

Nirmalya Sen and others added 24 commits April 1, 2019 11:43
Make running it conditional - either this is running or the Dremio cluster.
Controlled via helm value --set DremioAdmin=true

Change-Id: Ia6d30bcda29e5ea606f57aa6ac22b65aad8a3a91
Change-Id: Ica602f61421506708689bedb36d0d30c887c8407
Enabling support of uploads to S3 or ADLS via
config entries in values.yaml.

Change-Id: Ie3233cb7085268ecd6aef010be5f61ad67cefc3a
Change-Id: I3b6746fb8eddeff8c56ae4876453d47bfc30ef9b
Change-Id: I2e3444eccd95cdce7fab18045b43270cbbb07f79
Change-Id: I3534f554ee2168ee88e7ae19380e33696a27f2a3
Change-Id: I83a272457e977ea54dc35812a6d28b68dda666ee
1. Pull in the latest config files
2. Update memory, cpu in values.yaml so that they work with the
recommended machine types in the docs

Change-Id: I47970acda6fc776171f1970ddacb7e0d78dbff58
Plus change REAME layout and two label changes.

Change-Id: Ia5e05f13910db5c03361c6166411704b40bf4a8e
DX-16619: Fix the syntax in values.yaml

Statefulsets attribute is updateStrategy and the default for that
is RollingUpdate. So, deleting those lines would be sufficient
and not loose any functionality.

Kubernetes client libraries to v1.14 seems to be more strict than
earlier version and throws an error with those lines in there.

Dropping the lines works with earlier versions of helm.

Change-Id: I727adc50a883a4801e737aec0be6b84788f6f357
Change-Id: Ic0aa8058ffbcd46144e451621312554b72dc4a67
Certbot uses port 80 to handshake with LetsEncrypt to
generate SSL certificates. So, port 80 is being
opened up by default.

Change-Id: I565bd37b6b2657c67751efd1eee2f4cc45be4ba8
Azure VMs do not automatically do reverse DNS lookup of the VM's
public ip address. Reverse DNS needs to be specifically enabled.

Change-Id: Ie92fe9ca6c6358a3704947f201dae4e209602c66
1. The URL for path.dist values for S3, ADLS are different from
pre-3.2.0 and 3.2.0+. Handle it.
2. Add support for Azure Storage v2 for path.dist

Change-Id: I7730ed4caac22240e579e41641e720bdedf05ec0
Create a Azure Storage V2 account, create a filesystem and
directories in there and configure dremio to use that for
accelerator and uploads.

Change-Id: Ic1ce87d3ed8f2bfc6ac5a8d822af75f742b61d1e
Change-Id: I17e70a423356849c0d2368648a72241b97d5afeb
The image includes useful cloud tools. See the
Dockerfile for the list of tools/clients installed.

Change-Id: I6edd554ffc37b353813b8c351a9d505332267930
Change-Id: I90e2515ebc230fa8da5940498516f9ae313a019e
Change-Id: If8b01fcc3b7d71563a07f8115ce0c4ae03bd3f9a
- Additionally fixes deploying multiple instances of the CFT in the same
  subnet by removing the GroupName and allowing AWS to dynamically
  generate one since GroupName(s) must be unique in a subnet.

Change-Id: I1fb6101528b857a40e09f41aebd11a9843ecb257
- Dremio 4.0.0 or later required.
- Adds the concept of an imageTag to expose features that are
  introduced only in newer versions of Dremio.
- Removes the dremioVersion value that needs to be manually set
  to reference the same version that is used by the image.
- Adds optional Cloud Cache support.  Dist is split between PDFS and
  cloud storage.

Change-Id: Idccb0acbd82a8f3ed50b249599c0b04e93641406
This reverts commit d199978.

Change-Id: If484452e8608f2dbdfa2713ca33bb13b4c92c4f0
- Dremio 4.0.0 or later required.
- Adds the concept of an imageTag to expose features that are
  introduced only in newer versions of Dremio.
- Removes the dremioVersion value that needs to be manually set
  to reference the same version that is used by the image.
- Adds optional Cloud Cache support.  Dist is split between PDFS and
  cloud storage.

Change-Id: I645c53bb772c0d52362052ef77925c08b30cc494
(2) Default should be a folder below /opt/dremio/data to actually use the
executor's disk (volume size is defined in values.yaml)

Indentation fix
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants