search

kmesh-net / kmesh

High Performance ServiceMesh Data Plane Based on Programmable Kernel
https://kmesh.net
Apache License 2.0
466 stars 70 forks source link

Add UT and E2E test for `kmeshctl` #991

Open YaoZengzeng opened 3 weeks ago

YaoZengzeng commented 3 weeks ago

Task description:

Now kmeshctl lacks UT and E2E test cases, which is not good for robustness and needs to be supplemented ASAP.

Solution:

UT should be more friendly to beginners. E2E testing may require some research. It is recommended to start with UT first :)

Who can join or take the task:

The good first issue is intended for first-time contributors to get started on his/her contributor journey.

After a contributor has successfully completed 1-2 good first issue's, they should be ready to move on to help wanted items, saving the remaining good first issue for other new contributors.

How to join or take the task:

Just reply on the issue with the message /assign in a separate line.

Then, the issue will be assigned to you.

How to ask for help:

If you need help or have questions, please feel free to ask on this issue. The issue author or other members of the community will guide you through the contribution process.

codesmith25103 commented 2 weeks ago

/assign

YaoZengzeng commented 2 weeks ago

kmeshctl dump subcommand doesn't work well because rename of workload and ads mode to dual-engine and kernel-native mode.

Maybe you could start from here @codesmith25103 :)

hzxuzhonghu commented 2 weeks ago

Can you open a new issue to describe