Skip to content

Conversation

@tsuzu
Copy link
Contributor

@tsuzu tsuzu commented Dec 21, 2025

While I'm executing tests, I've got an error like:

=== RUN   TestLoaderHTTPMalformedURL
    fileloader_test.go:689: 
        	Error Trace:	/home/runner/work/kustomize/kustomize/api/internal/loader/fileloader_test.go:689
        	Error:      	Not equal: 
        	            	expected: "HTTP Error: status code 500 (Internal Server Error)"
        	            	actual  : "HTTP Error: status code 400 (Bad Request)"
        	            	
        	            	Diff:
        	            	--- Expected
        	            	+++ Actual
        	            	@@ -1 +1 @@
        	            	-HTTP Error: status code 500 (Internal Server Error)
        	            	+HTTP Error: status code 400 (Bad Request)
        	Test:       	TestLoaderHTTPMalformedURL

I'm not sure why is's caused, but it would be due to example.com-side changes.

@k8s-ci-robot
Copy link
Contributor

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@k8s-ci-robot k8s-ci-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Dec 21, 2025
@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Dec 21, 2025
@tsuzu tsuzu marked this pull request as ready for review December 21, 2025 15:26
@k8s-ci-robot k8s-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Dec 21, 2025
@tsuzu
Copy link
Contributor Author

tsuzu commented Dec 21, 2025

/kind bug

@k8s-ci-robot k8s-ci-robot added the kind/bug Categorizes issue or PR as related to a bug. label Dec 21, 2025
Copy link
Member

@TerryHowe TerryHowe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The 400 response makes more sense although this shouldn't really hit example.com

@koba1t
Copy link
Member

koba1t commented Dec 28, 2025

It seems to reproduce on the master branch.
I agree with the opinion that this error is caused by changes on the part of example.com.
Thank you!

/lgtm
/approve

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Dec 28, 2025
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: koba1t, TerryHowe, tsuzu

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Dec 28, 2025
@k8s-ci-robot k8s-ci-robot merged commit 008b7a0 into kubernetes-sigs:master Dec 28, 2025
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. kind/bug Categorizes issue or PR as related to a bug. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants