upgrade dns coredns image url to ghcr.io

[YangKeao]

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

What problem does this PR solve?

DNSChaos supports arm64 now!

What's changed and how it works?

1. Use the coredns image build on our own CI and published on ghcr.io
2. Replace tons of pingcap project into chaos-mesh project.

[ti-chi-bot]

[REVIEW NOTIFICATION]

This pull request has been approved by:

• STRRL
• 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.

[codecov]

Codecov Report

Merging #3488 (e34b303) into master (43fa669) will increase coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3488      +/-   ##
==========================================
+ Coverage   38.50%   38.56%   +0.05%     
==========================================
  Files         165      165              
  Lines       13698    13698              
==========================================
+ Hits         5275     5283       +8     
+ Misses       7992     7983       -9     
- Partials      431      432       +1     

see 3 files with indirect coverage changes

---

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 43fa669...e34b303. Read the comment docs.

[STRRL]

LGTM!

[cwen0]

LGTM

[cwen0]

/merge

[cwen0]

/merge

[ti-chi-bot]

@YangKeao: Your PR was out of date, I have automatically updated it for you.

At the same time I will also trigger all tests for you:

/run-all-tests

If the CI test fails, you just re-trigger the test that failed and the bot will merge the PR for you after the CI passes.

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 ti-community-infra/tichi repository.

[YangKeao]

/hold

This PR cannot pass the CI. I'll fix it later.

[STRRL]

/hold

This PR cannot pass the CI. I'll fix it later.

I found

/hold

This PR cannot pass the CI. I'll fix it later.

I found that the ghcr.io/chaos-mesh/chaos-coredns:v0.2.1-r1 might not work well:

I download logs from artifacts: https://github.com/chaos-mesh/chaos-mesh/actions/runs/2751672468

the container is not ready, with status:

  containerStatuses:
  - containerID: docker://ba34fa66001a517c68cb40a05e8074bc283389b0d01b19fa16be1aa9b3de1df0
    image: ghcr.io/chaos-mesh/chaos-coredns:v0.2.1-r1
    imageID: docker-pullable://ghcr.io/chaos-mesh/chaos-coredns@sha256:f4d9e55d9bb11c120d038f84b52dd01bc96124b8bcb2c31891cf27c5f325fdfa
    lastState:
      terminated:
        containerID: docker://ba34fa66001a517c68cb40a05e8074bc283389b0d01b19fa16be1aa9b3de1df0
        exitCode: 126
        finishedAt: "2022-07-28T06:18:19Z"
        message: 'OCI runtime create failed: container_linux.go:380: starting container
          process caused: exec: "/coredns": permission denied: unknown'
        reason: ContainerCannotRun
        startedAt: "2022-07-28T06:18:19Z"
    name: chaos-dns-server
    ready: false
    restartCount: 2
    started: false
    state:
      waiting:
        message: 'failed to start container "ba34fa66001a517c68cb40a05e8074bc283389b0d01b19fa16be1aa9b3de1df0":
          Error response from daemon: OCI runtime create failed: container_linux.go:380:
          starting container process caused: exec: "/coredns": permission denied:
          unknown'
        reason: RunContainerError

[STRRL]

/merge cancel

[STRRL]

@cwen0 @YangKeao @STRRL @xlgao-zju After debugging, I found that the coredns version is the reason for the failed e2e tests.

So in chaos-mesh/k8s_dns_chaos@07dc8bd, I revert the version to the specific commit and upload the latest version to test in this PR. All related DNSChaos tests are passed.

If there are no other questions, we can release a new version (v0.2.5) of chaos-coredns and update this PR.

hmm it still does not work, I would take a look later..

[STRRL]

@cwen0 @YangKeao @STRRL @xlgao-zju After debugging, I found that the coredns version is the reason for the failed e2e tests.
So in chaos-mesh/k8s_dns_chaos@07dc8bd, I revert the version to the specific commit and upload the latest version to test in this PR. All related DNSChaos tests are passed.
If there are no other questions, we can release a new version (v0.2.5) of chaos-coredns and update this PR.

hmm it still does not work, I would take a look later..

Hi, I think I resolved the issue and am releasing a new version of chaos-coredns: chaos-mesh/k8s_dns_chaos#24

This PR would get merged if it could all the testcases.

[g1eny0ung]

Great!

[sonatype-lift]

SC2068: Double quote array expansions to avoid re-splitting elements.

---

ℹ️ Expand to see all @sonatype-lift commands

You can reply with the following commands. For example, reply with @sonatype-lift ignoreall to leave out all findings.

Command	Usage
@sonatype-lift ignore	Leave out the above finding from this PR
@sonatype-lift ignoreall	Leave out all the existing findings from this PR
@sonatype-lift exclude <file|issue|path|tool>	Exclude specified file|issue|path|tool from Lift findings by updating your config.toml file

Note: When talking to LiftBot, you need to refresh the page to see its response.
_{Click here to add LiftBot to another repo.}

---

Help us improve LIFT! (Sonatype LiftBot external survey)

Was this a good recommendation for you? _{Answering this survey will not impact your Lift settings.}

[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]

[STRRL]

@sonatype-lift ignore

[sonatype-lift]

I've recorded this as ignored for this pull request.
If you change your mind, just comment @sonatype-lift unignore.

[xlgao-zju]

coooool!!!!

[STRRL]

/unhold

[STRRL]

/merge

[ti-chi-bot]

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

Commit hash: e34b303