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

Additional configuration should apply to start server. #71

Open
caoxianfei1 opened this issue Jun 16, 2023 · 2 comments
Open

Additional configuration should apply to start server. #71

caoxianfei1 opened this issue Jun 16, 2023 · 2 comments
Assignees
Labels
good first issue Good for newcomers

Comments

@caoxianfei1
Copy link
Collaborator

caoxianfei1 commented Jun 16, 2023

All services need to specify the machine ip and port to start, and there are some other configuration items that allow users to configure, so these configuration items also need to read the user's configuration before starting the service and then start the corresponding service.

CRD config file may look like this:

etcd:
    # Port for listening to partner communication. 
    # Etcd member accept incoming requests from its peers on a specific scheme://IP:port combination and the IP is host ip because we use hostnetwork:true.
    peerPort: 23800
    # clientPort for listening server port.
    clientPort: 23790
    configure:
        *
        *
        ...
mds:
    port: 6700
    dummyPort: 7700
    configure:
        *
        *
        ...
storage:
    ...
snapshotClone:
    ....

所有的服务都需要在yaml文件中指定Ip和端口,但是并不是只有这来个选项可以配置,一些其他的选项也可以在部署的时候指定,因此我们的curve-operator需要支持配置额外的参数。

@Daz-3ux
Copy link

Daz-3ux commented Aug 2, 2023

please assign me :)
already signed up for Curve Summer Coding Camp

@caoxianfei1
Copy link
Collaborator Author

please assign me :) already signed up for Curve Summer Coding Camp

done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants