Add jenkins e2e jobs #20
Conversation
Signed-off-by: Hang Yan <yhang@vmware.com>
Signed-off-by: Hang Yan <yhang@vmware.com>
Signed-off-by: Hang Yan <yhang@vmware.com>
|
@edwardbadboy This is the updated PR for e2e jobs settings. A few open questions:
@wsquan171 The flow records still does not match by count. Can you help take a look? you can find the paramater adjust in this PR. Also, during the test, i found out that the |
|
I think flow-aggregator.yml is still in Antrea repo, so the Jenkins job needs to download it from https://raw.githubusercontent.com/antrea-io/antrea/main/build/yamls/flow-aggregator.yml |
Signed-off-by: Hang Yan <yhang@vmware.com>
Signed-off-by: Hang Yan <yhang@vmware.com>
Signed-off-by: Hang Yan <yhang@vmware.com>
Signed-off-by: Hang Yan <yhang@vmware.com>
Signed-off-by: Hang Yan <yhang@vmware.com>
|
All updated. Including the various parameter adjust and image pre-load. Please help review again if you are free @edwardbadboy @wsquan171 |
Signed-off-by: Hang Yan <yhang@vmware.com>
|
@edwardbadboy all udpated. |
|
@dreamtalen Does the verify-doc step failed because it checks the relative file path on main branch? |
|
Hi Hang, We have PRs in Antrea suffer this dead link error of relative links too, for example, https://github.com/antrea-io/antrea/runs/6375438174?check_suite_focus=true#step:6:81. |
Signed-off-by: Hang Yan <yhang@vmware.com>
Signed-off-by: Hang Yan <yhang@vmware.com>
Signed-off-by: Hang Yan <yhang@vmware.com>
Signed-off-by: Hang Yan <yhang@vmware.com>
|
@edwardbadboy All updated. |
Signed-off-by: Hang Yan <yhang@vmware.com>
|
/theia-test-e2e |
1 similar comment
|
/theia-test-e2e |
|
The webhook are configured and this pr has a success result for jenkins e2e test. |
ci/jenkins/test-vmc.sh
Outdated
| RUN_CLEANUP_ONLY=true | ||
| shift | ||
| ;; | ||
| --coverage) |
There was a problem hiding this comment.
@hangyan Do you think we can remove all coverage related code from test-vmc.sh? There is no coverage support in Theia for now.
Signed-off-by: Hang Yan <yhang@vmware.com>
|
/theia-test-e2e |
|
@edwardbadboy All updated. |
|
This PR was for CI functions. Since Zhengsheng approved it, it can be checked in. |
| "metadata": { | ||
| "name": "CLUSTERNAMESPACE", | ||
| "labels": { | ||
| "antrea-ci": "true" |
There was a problem hiding this comment.
Let's change antrea-ci to theia-ci.
ci/jenkins/test-vmc.sh
Outdated
| echo "=== Old namespace ${cluster} has been deleted !!! ===" | ||
| done | ||
|
|
||
| kubectl get ns -l antrea-ci -o custom-columns=Name:.metadata.name,DATE:.metadata.creationTimestamp --no-headers=true | awk '{cmd="echo $(( $(date +%s) - $(date -d "$2" +%s) ))"; cmd | getline t ; print $1, t}' | awk '$1 !~ "matrix" && $1 !~ "whole-conformance" && $2 > 9000 {print $1}' | while read cluster; do |
There was a problem hiding this comment.
Let's delete line 437-450, and change antrea-ci to theia-ci in this line.
We can also change the awk expression in this line from
awk '$1 !~ "matrix" && $1 !~ "whole-conformance" && $2 > 9000 {print $1}'
to
awk '$2 > 9000 {print $1}'
ci/jenkins/jobs/projects.yaml
Outdated
| publishers: | ||
| - archive: | ||
| allow-empty: true | ||
| artifacts: theia-test-logs.tar.gz, e2e-coverage.tar.gz |
There was a problem hiding this comment.
Maybe remove e2e-coverage.tar.gz from artifacts?
Signed-off-by: Hang Yan <yhang@vmware.com>
|
/theia-test-e2e |
Signed-off-by: Hang Yan <yhang@vmware.com>
|
/theia-test-e2e |
|
@edwardbadboy Updated. |
Signed-off-by: Hang Yan yhang@vmware.com