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

[feat] add http deployment method #351

Closed
wants to merge 57 commits into from

Conversation

liuminjian
Copy link

@liuminjian liuminjian commented Nov 21, 2023

Issue Number: #328

1.增加http daemon的部署方式,host增加protocol和http port字段
image
image

2.在每个host节点上部署http daemon,可以使用ansible playbook部署,或者在k8s环境下用daemonset部署
image

3.部署http daemon后能正常部署curve集群
image

image image

遗留问题:目前enter指令不支持http方式

chengyu-l and others added 3 commits April 23, 2023 18:13
@liuminjian liuminjian changed the title 1.add http deployment method [feat] add http deployment method Nov 21, 2023
@liuminjian
Copy link
Author

@caoxianfei1 PTAL

@caoxianfei1
Copy link
Contributor

It seems to only support deploy cluster now, but how to implement other maintenance operations?

Wine93 and others added 23 commits November 22, 2023 17:13
Signed-off-by: Wine93 <[email protected]>
Signed-off-by: fengshunli <[email protected]>
Signed-off-by: Wine93 <[email protected]>
Signed-off-by: fengshunli <[email protected]>
Signed-off-by: Wine93 <[email protected]>
Wine93 and others added 20 commits November 22, 2023 17:13
solution: execute the ss command in a temporary container to precheck for ports in use instead

Signed-off-by: jyf111 <[email protected]>
Signed-off-by: Wine93 <[email protected]>
} else if t.httpConfig != nil {
client, err := module.NewHttpClient(*t.httpConfig)
if err != nil {
return errno.ERR_SSH_CONNECT_FAILED.E(err)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ERR_HTTP...

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

@caoxianfei1
Copy link
Contributor

LGTM!

@aspirer
Copy link
Collaborator

aspirer commented Nov 23, 2023

the branch you pushed should be develop

Signed-off-by: liuminjian <[email protected]>
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.