run ci and integration test task on arm #3177

YangKeao · 2022-04-22T07:08:25Z

Signed-off-by: YangKeao yangkeao@chunibyo.icu

What problem does this PR solve?

Run ci workflow on arm

Signed-off-by: YangKeao <yangkeao@chunibyo.icu>

ti-chi-bot · 2022-04-22T07:08:26Z

[REVIEW NOTIFICATION]

This pull request has been approved by:

WangXiangUSTC
cwen0

To complete the pull request process, please ask the reviewers in the list to review by filling /cc @reviewer in the comment.
After your PR has acquired the required number of LGTMs, you can assign this pull request to the committer in the list by filling /assign @committer in the comment to help you merge this pull request.

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

Reviewer can indicate their review by submitting an approval review.
Reviewer can cancel approval by submitting a request changes review.

Signed-off-by: YangKeao <yangkeao@chunibyo.icu>

codecov · 2022-04-22T07:22:52Z

Codecov Report

Merging #3177 (4b36baa) into master (c743fc2) will increase coverage by 0.13%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3177      +/-   ##
==========================================
+ Coverage   40.81%   40.95%   +0.13%     
==========================================
  Files         164      164              
  Lines       13811    13811              
==========================================
+ Hits         5637     5656      +19     
+ Misses       7743     7728      -15     
+ Partials      431      427       -4

Impacted Files	Coverage Δ
.../workflow/controllers/workflow_entry_reconciler.go	`49.18% <0.00%> (+2.16%)`	⬆️
pkg/workflow/controllers/chaos_node_reconciler.go	`61.31% <0.00%> (+3.15%)`	⬆️
controllers/statuscheck/controller.go	`84.14% <0.00%> (+4.87%)`	⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c743fc2...4b36baa. Read the comment docs.

Signed-off-by: YangKeao <yangkeao@chunibyo.icu>

WangXiangUSTC

rest LGTM

.github/workflows/integration_test.yml

ti-chi-bot · 2022-04-27T04:51:10Z

@YangKeao: PR needs rebase.

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.

.github/workflows/ci.yml

…on-arm Signed-off-by: YangKeao <yangkeao@chunibyo.icu>

Signed-off-by: YangKeao <yangkeao@chunibyo.icu>

…on-arm Signed-off-by: YangKeao <yangkeao@chunibyo.icu>

Signed-off-by: YangKeao <yangkeao@chunibyo.icu>

YangKeao · 2022-05-17T07:32:10Z

Some problems found in this PR is also recorded in #3173

YangKeao · 2022-05-17T07:44:06Z

.github/workflows/integration_test.yml

+      - name: Magic Kind DNS Fix
+        if: ${{ matrix.arch == 'arm64' }}
+        run: |
+          docker exec chart-testing-control-plane /bin/bash -c "sed -e 's/nameserver \(.*\)/nameserver 8.8.8.8/g' /etc/resolv.conf > /etc/resolv.conf.sed"


The network problem in kind is all about DNS.

kind has some magic scripts to fix the DNS loop in coredns when running inside a docker and the DNS server is a local address, see https://github.com/kubernetes-sigs/kind/blob/main/images/base/files/usr/local/bin/entrypoint#L375, and so does minikube. The awful thing is that: these scripts run, iptables rules are also applied successfully, but the DNS just don't work.

I have entered the environment manually, and dig/nslookup/tcpdump/... the DNS request with a lot of tools, but both the redirection source/target doesn't respond the DNS request. To further understand the problem, we could trace with the iptables, enable debug log of dockerd... It will take a lot more time to find the root, so I choose a much simpler way to solve: replace the dns server to 8.8.8.8 ...

WangXiangUSTC

rest LGTM

test/integration_test/run.sh

Signed-off-by: YangKeao <yangkeao@chunibyo.icu>

WangXiangUSTC · 2022-05-20T05:37:37Z

LGTM

cwen0 · 2022-05-24T02:40:19Z

@YangKeao Remember to update the configuration of ti-chi-bot before merging this pr. https://github.com/ti-community-infra/configs/blob/main/prow/config/config.yaml

cwen0

LGTM

cwen0 · 2022-05-24T09:37:40Z

/merge

ti-chi-bot · 2022-05-24T09:37:43Z

This pull request has been accepted and is ready to merge.

Commit hash: 4b36baa

add integration test and ci on arm

b60cbbd

Signed-off-by: YangKeao <yangkeao@chunibyo.icu>

ti-chi-bot requested review from WangXiangUSTC and yeya24 April 22, 2022 07:08

YangKeao mentioned this pull request Apr 22, 2022

Native ARM64 CI/CD #3173

Open

10 tasks

ti-chi-bot added the size/S label Apr 22, 2022

add changelog

18681a5

Signed-off-by: YangKeao <yangkeao@chunibyo.icu>

YangKeao force-pushed the integration-test-on-arm branch from faea049 to 18681a5 Compare April 22, 2022 07:15

fix fake_image for arm

51b3b0c

Signed-off-by: YangKeao <yangkeao@chunibyo.icu>

YangKeao added the rebuild-dev-env-image rebuild dev-env image in this PR label Apr 22, 2022

modify skywalking-eyes source

e5726a9

Signed-off-by: YangKeao <yangkeao@chunibyo.icu>

YangKeao changed the title ~~add integration test and ci on arm~~ run ci task on arm Apr 22, 2022

don't setup python for arm machine

faf3d31

Signed-off-by: YangKeao <yangkeao@chunibyo.icu>

ti-chi-bot added size/M and removed size/S labels Apr 22, 2022

download kind with ARCH specified

dc719e9

Signed-off-by: YangKeao <yangkeao@chunibyo.icu>

YangKeao changed the title ~~run ci task on arm~~ run ci and integration test task on arm Apr 22, 2022

YangKeao added 3 commits April 22, 2022 16:22

upgrade kubectl version to v1.23.4

b2429c9

Signed-off-by: YangKeao <yangkeao@chunibyo.icu>

remove install kubectl

2945b46

Signed-off-by: YangKeao <yangkeao@chunibyo.icu>

use kubectl version v1.23.1

8926277

Signed-off-by: YangKeao <yangkeao@chunibyo.icu>

YangKeao force-pushed the integration-test-on-arm branch from c4d928e to 8926277 Compare April 22, 2022 08:43

setup helm

a4b635b

Signed-off-by: YangKeao <yangkeao@chunibyo.icu>

WangXiangUSTC reviewed Apr 25, 2022

View reviewed changes

.github/workflows/integration_test.yml Show resolved Hide resolved

ti-chi-bot added the needs-rebase label Apr 27, 2022

STRRL reviewed May 7, 2022

View reviewed changes

.github/workflows/ci.yml Outdated Show resolved Hide resolved

ti-chi-bot added size/XXL and removed size/M labels May 10, 2022

Merge remote-tracking branch 'upstream/master' into integration-test-…

1a86e57

…on-arm Signed-off-by: YangKeao <yangkeao@chunibyo.icu>

YangKeao added 7 commits May 16, 2022 15:38

add magic commands to fix the kind dns

4d10a92

Signed-off-by: YangKeao <yangkeao@chunibyo.icu>

Merge remote-tracking branch 'upstream/master' into integration-test-…

e2d795e

…on-arm Signed-off-by: YangKeao <yangkeao@chunibyo.icu>

fix yaml quote

fecc3ac

Signed-off-by: YangKeao <yangkeao@chunibyo.icu>

don't pull image and preload

181b3ab

Signed-off-by: YangKeao <yangkeao@chunibyo.icu>

fix magic dns fix

453c2f7

Signed-off-by: YangKeao <yangkeao@chunibyo.icu>

use web-show on ghcr

a47e9dc

Signed-off-by: YangKeao <yangkeao@chunibyo.icu>

remove IOChaos test in chaosctl

074a187

Signed-off-by: YangKeao <yangkeao@chunibyo.icu>

ti-chi-bot added size/L and removed size/M labels May 17, 2022

skip JVMChaos in integration test

4268046

Signed-off-by: YangKeao <yangkeao@chunibyo.icu>

YangKeao force-pushed the integration-test-on-arm branch from 9051773 to 4268046 Compare May 17, 2022 06:40

remove physicalmachinechaos in integration test

9c746fd

Signed-off-by: YangKeao <yangkeao@chunibyo.icu>

YangKeao commented May 17, 2022

View reviewed changes

STRRL self-requested a review May 17, 2022 16:39

YangKeao requested a review from a team May 19, 2022 07:23

WangXiangUSTC approved these changes May 19, 2022

View reviewed changes

test/integration_test/run.sh Outdated Show resolved Hide resolved

test/integration_test/run.sh Show resolved Hide resolved

ti-chi-bot added the status/LGT1 label May 19, 2022

modify tab with spaces

d63be32

Signed-off-by: YangKeao <yangkeao@chunibyo.icu>

YangKeao mentioned this pull request May 24, 2022

add arm64 test for Chaos Mesh ti-community-infra/configs#579

Merged

cwen0 approved these changes May 24, 2022

View reviewed changes

ti-chi-bot added status/LGT2 and removed status/LGT1 labels May 24, 2022

Merge branch 'master' into integration-test-on-arm

4b36baa

ti-chi-bot added the status/can-merge label May 24, 2022

ti-chi-bot merged commit b65fd22 into chaos-mesh:master May 25, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

run ci and integration test task on arm #3177

run ci and integration test task on arm #3177

YangKeao commented Apr 22, 2022 •

edited

ti-chi-bot commented Apr 22, 2022 •

edited

codecov bot commented Apr 22, 2022 •

edited

WangXiangUSTC left a comment

ti-chi-bot commented Apr 27, 2022

YangKeao commented May 17, 2022

YangKeao May 17, 2022 •

edited

WangXiangUSTC left a comment

WangXiangUSTC commented May 20, 2022

cwen0 commented May 24, 2022

cwen0 left a comment

cwen0 commented May 24, 2022

ti-chi-bot commented May 24, 2022

run ci and integration test task on arm #3177

run ci and integration test task on arm #3177

Conversation

YangKeao commented Apr 22, 2022 • edited

What problem does this PR solve?

ti-chi-bot commented Apr 22, 2022 • edited

codecov bot commented Apr 22, 2022 • edited

Codecov Report

WangXiangUSTC left a comment

Choose a reason for hiding this comment

ti-chi-bot commented Apr 27, 2022

YangKeao commented May 17, 2022

YangKeao May 17, 2022 • edited

Choose a reason for hiding this comment

WangXiangUSTC left a comment

Choose a reason for hiding this comment

WangXiangUSTC commented May 20, 2022

cwen0 commented May 24, 2022

cwen0 left a comment

Choose a reason for hiding this comment

cwen0 commented May 24, 2022

ti-chi-bot commented May 24, 2022

YangKeao commented Apr 22, 2022 •

edited

ti-chi-bot commented Apr 22, 2022 •

edited

codecov bot commented Apr 22, 2022 •

edited

YangKeao May 17, 2022 •

edited