-
Notifications
You must be signed in to change notification settings - Fork 109
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
[dhctl] feat(dhctl-for-commander): add more grpc services #8455
Conversation
ab09b80
to
a719d12
Compare
a719d12
to
381356e
Compare
⚪ Workflow details |
⚪ Workflow details |
/e2e/abort openstack;Standard;containerd;1.25 9202480859 9202480859-1-con-1-25 /sys/deckhouse-oss/install:pr8455 redos@31.129.32.110 |
🔴 Workflow details (1 job failed)🔴 |
Signed-off-by: Ilya Mayorov <ilya.mayorov@flant.com>
381356e
to
5b9f113
Compare
🟢 Workflow detailsYandex.Cloud-WithoutNAT-Containerd-1.25 - Connection string: 🟢 |
Cherry pick PR 8534 to the branch release-1.60 successful! |
) (#8534) Signed-off-by: Ilya Mayorov <ilya.mayorov@flant.com> Co-authored-by: Ilya Mayorov <37873799+faroyam@users.noreply.github.com> Co-authored-by: Timofey Kirillov <timofey.kirillov@flant.com>
Description
Add bootstrap, converge, destroy, abort, import grpc services. We need them in Deckhouse Commander.
Why do we need it, and what problem does it solve?
This PR adds operations which lacked in the current version of dhctl-server, making it fully basically implemented (in the current version only check and bootstrap implemeneted).
This PR also fixes all operations logical errors related to the rpc operation messages processing, making it more stable and actually working.
Why do we need it in the patch release (if we do)?
This is change that corrects dhctl-server which was firstly introduced in the 1.60 release. So it is preferred to include this change into 1.60 release. Because after this change dhctl-server in the 1.60 release will stabilize more and will be equipped with full set of basic operations.
What is the expected result?
It is not expected to change anything outside of pkg/server scope by this PR.