fix: remove trailing slash in --test-package-url URLs causing CI 404 errors#36535
Conversation
A follow up to kubernetes#36528 as the job got further but still broke afterwards. Remove trailing slashes from --test-package-url URLs in kubetest2 ec2 job configurations. When combined in these spots - https://github.com/kubernetes-sigs/kubetest2/blob/beaf896881019d46bdf961228a117993c1d813db/pkg/testers/ginkgo/package.go#L42 - https://github.com/kubernetes-sigs/kubetest2/blob/beaf896881019d46bdf961228a117993c1d813db/pkg/testers/ginkgo/package.go#L183-L200 these trailing slashes resulted in double slashes in download URLs (e.g. `https://dl.k8s.io//ci/fast/latest-fast.txt`). Causing the following error: ``` I0226 13:35:18.842783 26057 package.go:51] Test package version was not specified. Defaulting to version from latest-fast.txt: <?xml version='1.0' encoding='UTF-8'?><Error><Code>NoSuchKey</Code><Message>The specified key does not exist.</Message><Details>No such object: 767373bbdcb8270361b96548387bf2a9ad0d48758c35//ci/fast/latest-fast.txt</Details></Error> I0226 13:35:18.842890 26057 package.go:201] Downloading test tar ball from: https://dl.k8s.io//ci/fast/NoSuchKeyThe specified key does not exist. Details No such object: 767373bbdcb8270361b96548387bf2a9ad0d48758c35//ci/fast/latest-fast.txt /kubernetes-test-linux-amd64.tar.gz F0226 13:35:18.854134 26057 ginkgo.go:259] failed to run ginkgo tester: failed to get ginkgo test package from published releases: failed to create gzip reader: EOF ``` This previously worked because the dl.k8s.io CDN would redirect malformed URLs, but as of Feb 16, 2026, the CDN serves directly from the origin, causing 404 errors for URLs with double slashes. Affected jobs: - cloud-provider-aws periodics and presubmits - sig-node presubmits Fixes CI failures observed since mid-February in pull-cloud-provider-aws-e2e-kubetest2-quick and related jobs. Context on thread here: https://kubernetes.slack.com/archives/C09QZ4DQB/p1772041962062339
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: damdo, upodroid The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
@damdo: Updated the
DetailsIn response to this:
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-sigs/prow repository. |
A follow up to #36528 as the job got further but still broke afterwards.
--
Remove trailing slashes from --test-package-url URLs in kubetest2 ec2 job configurations. When combined in these spots
these trailing slashes resulted in double slashes in download URLs (e.g.
https://dl.k8s.io//ci/fast/latest-fast.txt).Causing the following error:
This previously worked because the dl.k8s.io CDN would redirect malformed URLs, but as of Feb 16, 2026, the CDN serves directly from the origin, causing 404 errors for URLs with double slashes.
Affected jobs:
Fixes CI failures observed since mid-February in pull-cloud-provider-aws-e2e-kubetest2-quick and related jobs.
Context on thread here: https://kubernetes.slack.com/archives/C09QZ4DQB/p1772041962062339