Skip to content

Conversation

@mbien
Copy link
Member

@mbien mbien commented Jan 14, 2026

workaround is no longer needed. (unclear what fixed it, likely some of the routine dependency updates)

partially reverts c0f09c9

fixes #6680

workaround is no longer needed.

partially reverts c0f09c9
@mbien mbien added this to the NB29 milestone Jan 14, 2026
@mbien mbien added Gradle [ci] enable "build tools" tests LSP [ci] enable Language Server Protocol tests tests enterprise [ci] enable enterprise job labels Jan 14, 2026
@mbien
Copy link
Member Author

mbien commented Jan 14, 2026

would be interesting to find out what fixed this issue, but bisecting would be too time consuming for me right now.

@mbien mbien requested review from lkishalmi and sdedic January 14, 2026 13:53
@matthiasblaesing
Copy link
Contributor

would be interesting to find out what fixed this issue, but bisecting would be too time consuming for me right now.

The message from c0f09c9 says, that the problem occurs when downloading. Wouldn't "they fixed the server" be the most sensible explanation? If so, you would not find the reason by bisecting.

@matthiasblaesing
Copy link
Contributor

Not a formal review, but +1 for removing ugly workaround.

@mbien
Copy link
Member Author

mbien commented Jan 14, 2026

Wouldn't "they fixed the server" be the most sensible explanation?

yeah that was also my original hypothesis #6673 (comment) (although the server is still configured to not allow tls 1.3 so that didn't change at least). But there were more weird things going on, e.g other tests were able to download gradle.

@mbien
Copy link
Member Author

mbien commented Jan 16, 2026

fwiw: still reproducible today when checking out a hash shortly after the workaround was added via #6673. The 23-rc1 tag is working while 22-rc1 fails.

Not going to narrow it down further given how time consuming the rebuild/kill daemons/clear gradle cache/run test cycle is.

Most plausible cause for the resolution would be one of the routine dep updates #7583, #7690.

Going to merge this PR. Thanks for taking a look.

@mbien mbien merged commit 4c3d774 into apache:master Jan 16, 2026
63 of 64 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enterprise [ci] enable enterprise job Gradle [ci] enable "build tools" tests LSP [ci] enable Language Server Protocol tests tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove HTTPS workaround added in #6673

2 participants