PingCAP-QE / ci

Continue intergration tests
Apache License 2.0
19 stars 97 forks source link

Revert "refactor(jenkins/pipelines,pipelines): use the new dockerfile url location" #2959

Closed wuhuizuo closed 1 month ago

wuhuizuo commented 1 month ago

Reverts PingCAP-QE/ci#2958

It broken the dev-builds since of the issue, we must solve it atomic:

[2024-05-10T10:29:43.850Z] + docker build --pull -t 325168c9-8528-454b-9e0f-2041080ee440 . --build-arg 'BASE_IMG='
ti-chi-bot[bot] commented 1 month ago

I have already done a preliminary review for you, and I hope to help you do a better job.

Based on the pull request title and description, it seems that the changes made in PingCAP-QE/ci#2958 are being reverted. The diff shows that the Dockerfile URLs are being changed from the new location to the old one.

Potential problems with this pull request could be:

Some suggestions to fix these problems could be:

wuhuizuo commented 1 month ago

/review default

ti-chi-bot[bot] commented 1 month ago

@wuhuizuo:

I have already done a preliminary review for you, and I hope to help you do a better job.

It seems this pull request is reverting a previous change that broke the dev-builds due to a problem with the Docker build command. The change that is being reverted is a refactor of the Jenkins pipelines that changed the location of the Dockerfile URLs.

There are no potential problems with the code changes since the pull request is just reverting a previous change. However, there might be issues with the dev builds if the previous problem is not resolved.

My suggestion is to investigate the issue with the Docker build command and fix it before attempting to refactor the Jenkins pipelines. Once the issue is fixed, the previous refactor can be revisited and implemented again.

In response to [this](https://github.com/PingCAP-QE/ci/pull/2959#issuecomment-2105424143): >/review default Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.
wuhuizuo commented 1 month ago

/approve

ti-chi-bot[bot] commented 1 month ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: wuhuizuo

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: - ~~[jenkins/OWNERS](https://github.com/PingCAP-QE/ci/blob/main/jenkins/OWNERS)~~ [wuhuizuo] - ~~[pipelines/OWNERS](https://github.com/PingCAP-QE/ci/blob/main/pipelines/OWNERS)~~ [wuhuizuo] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment