-
Notifications
You must be signed in to change notification settings - Fork 38.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
test images: Use PULL_BASE_SHA for non-git image building #104393
test images: Use PULL_BASE_SHA for non-git image building #104393
Conversation
@claudiubelu: This issue is currently awaiting triage. If a SIG or subproject determines this is a relevant issue, they will accept it by applying the The 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. |
1360851
to
a5d0e48
Compare
/cc @mkumatag |
test/images/image-util.sh
Outdated
@@ -186,8 +188,7 @@ build() { | |||
docker buildx build --progress=plain --no-cache --pull --output=type="${output_type}" --platform "${os_name}/${arch}" \ | |||
--build-arg BASEIMAGE="${base_image}" --build-arg REGISTRY="${REGISTRY}" --build-arg OS_VERSION="${os_version}" \ | |||
-t "${REGISTRY}/${image}:${TAG}-${suffix}" -f "${dockerfile_name}" \ | |||
--label "image_version=${TAG}" --label "commit_id=${GIT_COMMIT_ID}" \ | |||
--label "git_url=https://github.com/kubernetes/kubernetes/tree/${GIT_COMMIT_ID}/test/images/${img_folder}" . | |||
--label "image_version=${TAG}" --label "git_tag=${GIT_TAG}" . |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I also see TAG
here but in the cloudbuild.yaml you have replaced the TAG
with GIT_TAG
, wondering if that creates any issue?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems that PULL_BASE_SHA
is more commonly used in other jobs too, and it's mentioned here as well: https://github.com/kubernetes/test-infra/blob/master/prow/jobs.md#job-environment-variables
Went ahead with PULL_BASE_SHA
instead.
In the test image build jobs, the image-util.sh script is not being run in a git repository, which causes git log to fail. In this case, we can use the PULL_BASE_SHA set in cloudbuild.yaml instead.
a5d0e48
to
00dd02f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/approve
/lgtm
/hold
For test infra PR to merge
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: claudiubelu, spiffxp 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:
Approvers can indicate their approval by writing |
/hold cancel |
What type of PR is this?
/kind bug
/sig testing
/priority important-soon
What this PR does / why we need it:
In the test image build jobs, the
image-util.sh
script is not being run in a git repository, which causes git log to fail [1]. In this case, we can use thePULL_BASE_SHA
set incloudbuild.yaml
instead.[1] https://prow.k8s.io/view/gs/kubernetes-jenkins/logs/post-kubernetes-push-e2e-agnhost-test-images/1425869767360647168
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Depends On: kubernetes/test-infra#23257
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: