openyurt icon indicating copy to clipboard operation
openyurt copied to clipboard

[Propalsal] [WIP] propalsal to add edge-autonomy-e2e-tests

Open lorrielau opened this issue 3 years ago • 12 comments

What type of PR is this?

/kind documentation

What this PR does / why we need it:

Add edge-autonomy-e2e-tests for Openyurt.

Special notes for your reviewer:

@Congrool

Does this PR introduce a user-facing change?

None

other Note

lorrielau avatar Sep 01 '22 12:09 lorrielau

@lorrielau: GitHub didn't allow me to assign the following users: your_reviewer.

Note that only openyurtio members, repo collaborators and people who have commented on this issue/PR can be assigned. Additionally, issues/PRs can only have 10 assignees at the same time. For more information please see the contributor guide

In response to this:

What type of PR is this?

/kind documentation

What this PR does / why we need it:

Add edge-autonomy-e2e-tests for Openyurt.

Special notes for your reviewer:

@Congrool

Does this PR introduce a user-facing change?

None

other Note

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.

openyurt-bot avatar Sep 01 '22 12:09 openyurt-bot

Codecov Report

Merging #981 (56ba9c1) into master (b9f6bd7) will decrease coverage by 0.03%. The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #981      +/-   ##
==========================================
- Coverage   47.46%   47.42%   -0.04%     
==========================================
  Files          85       85              
  Lines       11799    11791       -8     
==========================================
- Hits         5600     5592       -8     
  Misses       5738     5738              
  Partials      461      461              
Flag Coverage Δ
unittests 47.42% <ø> (-0.04%) :arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
pkg/yurthub/cachemanager/cache_agent.go 49.27% <0.00%> (-5.28%) :arrow_down:
pkg/yurtctl/cmd/yurttest/kindinit/init.go 50.71% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

codecov[bot] avatar Sep 01 '22 12:09 codecov[bot]

E2e-tests timed out. How can I re-run the checks? :>

lorrielau avatar Sep 02 '22 01:09 lorrielau

E2e-tests timed out. How can I re-run the checks? :>

Currently, we do not have something like /retest😢. You can git commit --amend --no-edit --signoff and then push to your repo again, e2e-test then will be re-triggered.

Congrool avatar Sep 02 '22 01:09 Congrool

Could you please also describe a little more about how do we do autonomy test, including

  1. how could we know if test succeeds on the host, since these tests are running in the kind node which is a container?
  2. how to distinguish original test and autonomy test, and how to run them respectively?
  3. which way do these tests run in, in parallel, in serial or in any other special order?

And also add details about how to run each autonomy test on kubelet, kube-proxy, flannel, coreDNS and yurthub. We can describe them in steps.

Congrool avatar Sep 02 '22 02:09 Congrool

Done, please take a look.

lorrielau avatar Sep 13 '22 12:09 lorrielau

@Congrool: changing LGTM is restricted to collaborators

In response to this:

/lgtm

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.

openyurt-bot avatar Sep 15 '22 02:09 openyurt-bot

/lgtm

Congrool avatar Sep 16 '22 06:09 Congrool

/lgtm cancel

Congrool avatar Sep 16 '22 06:09 Congrool

Modified

lorrielau avatar Sep 18 '22 07:09 lorrielau

/approve

rambohe-ch avatar Sep 27 '22 07:09 rambohe-ch

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: lorrielau, rambohe-ch

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

The pull request process is described 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

openyurt-bot avatar Sep 27 '22 07:09 openyurt-bot