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

Create cluster documentation is not clear #197

Closed
fbalak opened this issue Jun 1, 2017 · 5 comments
Closed

Create cluster documentation is not clear #197

fbalak opened this issue Jun 1, 2017 · 5 comments
Assignees

Comments

@fbalak
Copy link
Contributor

fbalak commented Jun 1, 2017

I do not understand what exactly do all the parameters for CreateCluster API call. There are some parameters that apply only for ceph and I am not sure if anything from those parameters I can use for gluster.

I also do not understand what provisioning_ip parameter stand for. I thought that it should point to node where is set provisioner tag but in documentation there is set node ip of each node in the node_configuration dictionary.

How does node_identifier parameter work and how it affect other parameters?

Could there be some list of parameters with information like what those parameters do and what parameters are mandatory for each sds type?

@fbalak
Copy link
Contributor Author

fbalak commented Jun 2, 2017

What are possible values for conf_overrides parameter?

@mbukatov
Copy link
Contributor

mbukatov commented Jun 8, 2017

@fbalak try to ping @anivargi via irc or slack

@r0h4n
Copy link
Contributor

r0h4n commented Jun 9, 2017

@fbalak conf_overrides is not used right now in Tendrl CreateCluster, the definitions 0 need to be edited to remove the same.

conf_overrides is a way in which you can pass sds specific configuration (global) which will be added to the sds conf files (eg: ceph config file or gluster conf). But we have decided to keep it simple and not allow conf_overrides till we get the basics right.

@anivargi
Copy link
Contributor

anivargi commented Jun 9, 2017

The node_identifier takes two possible inputs ip or uuid.

  • ip address is the ip of the host machine
  • uuid is the node_id which you can retrieve form GetNodeList

The reason for adding the support for the ip parameter is making the GetNodeList call optional.

Regarding the provisioning_ip, is the same as the above mentioned ip option.

conf_overrides is supported from the API, but these overrides are used only for Ceph.

I will be updating the CreateCluster API doc soon with these optional parameters.

@mkudlej
Copy link

mkudlej commented Jun 9, 2017

@r0h4n @anivargi Thank you for responses. Could you please change documentation according these comments?

fbalak added a commit to fbalak/usmqe-tests that referenced this issue Jun 9, 2017
use insted `node_identifier` of from node list
based on:
Tendrl/api#197 (comment)
@anivargi anivargi self-assigned this Jun 29, 2017
@r0h4n r0h4n closed this as completed Jul 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants