Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Summary: In order to maintain compatibility with previous chart behavior, it's necessary to quote gflag overrides in order to prevent breakages due to multi-word gflag values. This issue was introduced in 4e560f0 Test Plan: Run template containing multi-word gflag value without this change: $ helm install yugabyte ./ --set gflags.master.test1234="asdf asdf" --dry-run --debug ... --test1234=asdf asdf \ ... Run template contianing multi-word gflag value with this change: $ helm install yugabyte ./ --set gflags.master.test1234="asdf asdf" --dry-run --debug ... --test1234="asdf asdf" \ ... Verify that the argument value is properly quoted. Reviewers: arnav, sanketh Reviewed By: sanketh Subscribers: yugaware Differential Revision: https://phabricator.dev.yugabyte.com/D12055
- Loading branch information