Closed wuhuizuo closed 2 weeks 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 are made to fix image building jobs in the ti-community-infra/prow
repository. The changes introduced are modifying the run_if_changed
regex patterns in the push-prow-images
job and build-images
command in the prow
job. The serviceAccountName
is also changed to prow-pusher
in both jobs.
However, there are some potential problems I noticed in this pull request. Firstly, the run_if_changed
regex pattern in the push-prow-images
job seems incorrect. The closing bracket is misplaced, which may cause the pattern to match unintended files. Secondly, the build-images
command in the prow
job is missing the -C prow
argument which specifies the working directory. This may cause the command to fail or build images in the wrong directory.
To fix these issues, the run_if_changed
regex pattern in the push-prow-images
job should be modified to:
- run_if_changed: '^(\.ko\.yaml|hack/(make-rules|prowimagebuilder))|.+\.go'
+ run_if_changed: '^(\.ko\.yaml|hack/(make-rules|prowimagebuilder)|.+\.go)'
And the build-images
command in the prow
job should be modified to:
- - make
- - -C
- - prow
+ - make build-images REGISTRY=ticommunityinfra
This will ensure that the correct directory is used and the images are built with the correct registry.
I have already done a preliminary review for you, and I hope to help you do a better job.
Key Changes:
postsubmits.yaml
and presubmits.yaml
files.pusher
to prow-pusher
.cpu
value from "4"
to "1"
in both files.REGISTRY=ticommunityinfra
in the args
list in presubmits.yaml
file.Potential Problems:
cpu
value was reduced from "4"
to "1"
. This might lead to slower builds or longer build times.Fixing Suggestions:
cpu
value.cpu
value was reduced to save resources, then it might be better to consider other approaches like horizontal pod autoscaling or limiting the number of concurrent builds instead of reducing the cpu
value./approve
[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
Signed-off-by: wuhuizuo wuhuizuo@126.com