Skip to content

Conversation

sgotti
Copy link
Member

@sgotti sgotti commented Jan 26, 2018

Add a --store-prefix option to specify the store prefix.

In the meantime, for consistency, rename StoreBasePath to StorePrefix.

Add a `--store-prefix` option to specify the store prefix.

In the meantime, for consistency, rename StoreBasePath to StorePrefix.
@kemko
Copy link

kemko commented Jan 26, 2018

It seems that keeper has ignored --store-prefix key. I changed keeper-entrypoint.sh by add --store-prefix=/stolon/cluster but got no cluster data available error and in ps stolon-keeper miss this option.

All other parts running fine and i'm seen new prefix in etcd.

@sgotti
Copy link
Member Author

sgotti commented Jan 26, 2018

@kemko I tested it and it works. Are you sure you built stolon with this PR included?

@kemko
Copy link

kemko commented Jan 29, 2018

@sgotti I forgot to add backslash while adding --store-prefix option and bash were interpreted it as another one command. Now it works.

@sgotti sgotti merged commit 0802a61 into sorintlab:master Jan 29, 2018
sgotti added a commit that referenced this pull request Jan 29, 2018
*: Make the store prefix configurable
@sgotti
Copy link
Member Author

sgotti commented Jan 29, 2018

@kemko Thanks! merged.

@sgotti sgotti added this to the v0.9.0 milestone Feb 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants