kmesh icon indicating copy to clipboard operation
kmesh copied to clipboard

test(e2e): add e2e tests for authz, secret, log, accesslog, version and dump

Open ravjot07 opened this issue 8 months ago • 7 comments

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.

ravjot07 avatar Apr 18 '25 21:04 ravjot07

[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.

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

kmesh-bot avatar Apr 18 '25 21:04 kmesh-bot

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 data Powered 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.

codecov[bot] avatar Apr 18 '25 21:04 codecov[bot]

accesslogs are reported periodically after (5 sec) for long tcp_connections, can you add a e2e test for this ?? @ravjot07

yp969803 avatar Apr 20 '25 01:04 yp969803

accesslogs are reported periodically after (5 sec) for long tcp_connections, can you add a e2e test for this ?? @ravjot07

ya sure

ravjot07 avatar Apr 20 '25 18:04 ravjot07

ping @ravjot07

YaoZengzeng avatar Apr 24 '25 02:04 YaoZengzeng

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.

kmesh-bot avatar May 04 '25 05:05 kmesh-bot

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.

@ravjot07 you should use git rebase instead of merging.

YaoZengzeng avatar May 06 '25 07:05 YaoZengzeng