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.

Sign up for GitHub

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

Jump to bottom

add emctl proposal #27

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

add emctl proposal #27

wants to merge 1 commit into from

Conversation

Poorunga
Copy link
Member

@Poorunga Poorunga commented Aug 2, 2021 •
edited
Loading

emctl is a CLI utility that allows users to easily use EdgeMesh.

@kubeedge-bot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please assign after the PR has been reviewed.
You can assign the PR to them by writing /assign in a comment when ready.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@Poorunga
add emctl proposal
5560c2d 
Signed-off-by: Poorunga <[email protected]>
@kubeedge-bot kubeedge-bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Aug 2, 2021
@Poorunga
Copy link
Member Author

Poorunga commented Aug 2, 2021

/hold

@kubeedge-bot kubeedge-bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Aug 2, 2021
@Poorunga
Copy link
Member Author

Poorunga commented Aug 2, 2021

/assgin

@kubeedge-bot
Copy link
Collaborator

@Poorunga: PR needs rebase.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@kubeedge-bot kubeedge-bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Sep 16, 2021
@JiaweiGithub
Copy link
Contributor

JiaweiGithub commented Jul 10, 2023 •
edited
Loading

Hi Poorunga. Do you have any plans for emctl? I am interested in this and may be able to make some contributions. @Poorunga

@Poorunga
Copy link
Member Author

@JiaweiGithub Your attention to emctl brings me great joy, and I extend a warm welcome to your willingness to engage in further discussions on this feature with us.
Initially, we envisioned that emctl could enhance the user experience for edgemesh users, primarily by focusing on streamlining usability aspects.
Currently, we have made some progress, such as the ability to swiftly deploy EdgeMesh using the helm install approach. However, I believe there is ample room for further enhancements, such as facilitating rapid configuration changes for kubeedge (cloudcore and edgecore) to accommodate Edgemesh operations. Currently, this process involves manual modifications, which has been reported as a point of feedback. Additionally, offline installation of EdgeMesh is another area of consideration. All of these ideas can be implemented within the realm of emctl.

@JiaweiGithub
Copy link
Contributor

I have seen the ctl code of calico before and found it very powerful. I think emctl can do more than that, especially when Edgemesh supports podip.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Poorunga @kubeedge-bot @JiaweiGithub