Hello Kudu Jenkins, Bankim Bhavsar,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/17739
to look at the new patch set (#3).
Change subject: [helm] fix usage of multiple directories
......................................................................
[helm] fix usage of multiple directories
This patch updates the /helm implementation to use the new FS_WAL_DIR and
FS_DATA_DIRS environment variables. This patch also updates the README with
steps I used to test the changes.
I verified that I could update the number of directories in kudu/values.yaml,
confirming that the servers used multiple directories. Below is a snippet of
the logs from one tserver with multiple directories (5, set via
kudu/values.yaml). Previously, even with multiple storage devices set,
each server would use a single directory /var/lib/kudu for storage.
$ kubectl logs kudu-tserver-1
I0730 00:36:07.404302 1 tablet_server_runner.cc:78] Tablet server
non-default flags:
--use_hybrid_clock=false
--fs_data_dirs=/mnt/disk1,/mnt/disk2,/mnt/disk3,/mnt/disk4
--fs_wal_dir=/mnt/disk0
--webserver_doc_root=/opt/kudu/www
--tserver_master_addrs=kudu-master-0.kudu-masters.default.svc.cluster.local,kudu-master-1.kudu-masters.default.svc.cluster.local,kudu-master-2.kudu-masters.default.svc.cluster.local
--heap_profile_path=/tmp/kudu.1
--stderrthreshold=0
Tablet server version:
kudu 1.16.0-SNAPSHOT
revision 2ceec7749
build type RELEASE
...
A rendered README can be found here:
https://github.com/andrwng/kudu/blob/helm_multidir/kubernetes/helm/README.adoc
Change-Id: I583dff69c809e0851052c98267740a3e70c60efa
---
M kubernetes/helm/README.adoc
M kubernetes/helm/kudu/templates/_helmutils.tpl
M kubernetes/helm/kudu/templates/kudu-service.yaml
M kubernetes/helm/kudu/values.yaml
4 files changed, 109 insertions(+), 23 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/39/17739/3
--
To view, visit http://gerrit.cloudera.org:8080/17739
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I583dff69c809e0851052c98267740a3e70c60efa
Gerrit-Change-Number: 17739
Gerrit-PatchSet: 3
Gerrit-Owner: Andrew Wong <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Bankim Bhavsar <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
