cp-docker-images icon indicating copy to clipboard operation
cp-docker-images copied to clipboard

add snapshot.trust.empty and admin.enableServer to zookeeper.properties.template

Open sawanoboly opened this issue 4 years ago • 5 comments

Hi,

I've checked upgrade guide to upgrade 5.3 to 5.4. It turns out that we need to add snapshot.trust.empty to the zookeeper config.

https://docs.confluent.io/current/installation/upgrade.html#additional-zk-upgrade-information

This PR adds snapshot.trust.empty and admin.enableServer to zookeeper.properties.

sawanoboly avatar Jan 15 '20 07:01 sawanoboly

It looks like @sawanoboly hasn't signed our Contributor License Agreement, yet.

The purpose of a CLA is to ensure that the guardian of a project's outputs has the necessary ownership or grants of rights over all contributions to allow them to distribute under the chosen licence. Wikipedia

You can read and sign our full Contributor License Agreement here.

Once you've signed reply with [clabot:check] to prove it.

Appreciation of efforts,

clabot

ghost avatar Jan 15 '20 07:01 ghost

[clabot:check]

sawanoboly avatar Jan 15 '20 08:01 sawanoboly

@confluentinc It looks like @sawanoboly just signed our Contributor License Agreement. :+1:

Always at your service,

clabot

ghost avatar Jan 15 '20 08:01 ghost

Kindly merge this. It is required.

ethicalmohit avatar Jan 30 '20 16:01 ethicalmohit

Can these not be added from Java properties?

OneCricketeer avatar Feb 20 '20 13:02 OneCricketeer