test(e2e): add e2e tests for authz, secret, log, accesslog, version and dump
test(e2e): add end-to-end tests for kmeshctl commands
- authz: enable/disable/status subcommands
- secret: create and update IPsec secret
- log: list, get and set logger levels
- accesslog: enable/disable per-pod and cluster
- dump: kernel-native, dual-engine, and invalid-mode handling
- version: version for kmeshctl, for specific pod
Each test locates a running kmesh daemon pod, waits for readiness, invokes the CLI, and asserts expected behaviour or output.
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: Once this PR has been reviewed and has the lgtm label, please assign lec-bit for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.
Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 46.26%. Comparing base (
44253ce) to head (4cd923f). Report is 16 commits behind head on main.
Continue to review full report in Codecov by Sentry.
Legend - Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing dataPowered by Codecov. Last update 1d2122a...4cd923f. Read the comment docs.
:rocket: New features to boost your workflow:
- :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
accesslogs are reported periodically after (5 sec) for long tcp_connections, can you add a e2e test for this ?? @ravjot07
accesslogs are reported periodically after (5 sec) for long tcp_connections, can you add a e2e test for this ?? @ravjot07
ya sure
ping @ravjot07
Adding label do-not-merge/contains-merge-commits because PR contains merge commits, which are not allowed in this repository.
Use git rebase to reapply your commits on top of the target branch. Detailed instructions for doing so can be found here.
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.
Adding label
do-not-merge/contains-merge-commitsbecause PR contains merge commits, which are not allowed in this repository. Usegit rebaseto reapply your commits on top of the target branch. Detailed instructions for doing so can be found here.
@ravjot07 you should use git rebase instead of merging.