Skip to content

Conversation

@alecharp
Copy link
Member

@alecharp alecharp commented Nov 6, 2025

Testing done

The test GitLabConnectionConfigSSLTest is failing in BOM (jenkinsci/bom#5931) but also locally on the current master branch.
This is to fix it.

Submitter checklist

  • Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did
  • Link to relevant issues in GitHub or Jira
  • Link to relevant pull requests, esp. upstream and downstream changes
  • Ensure you have provided tests that demonstrate the feature works or the issue is fixed

@alecharp alecharp requested a review from a team as a code owner November 6, 2025 10:17
@github-actions github-actions bot added the tests This PR adds/removes/updates test cases label Nov 6, 2025
@alecharp
Copy link
Member Author

alecharp commented Nov 6, 2025

@krisstern as this is possibly blocking BOM release later this week.

@alecharp
Copy link
Member Author

alecharp commented Nov 6, 2025

Ok so there is more to it as Windows build still has the previous message.

@alecharp
Copy link
Member Author

alecharp commented Nov 6, 2025

Original test passes on Java 17 but not 21/25.
Patch is fixing Java 21/25 but breaks 17. Working on it.

@alecharp alecharp force-pushed the bug/certificate-not-found branch from a91199a to fb5933f Compare November 6, 2025 14:09
@alecharp
Copy link
Member Author

alecharp commented Nov 6, 2025

Sorry for the force push, I didn't want to do yet-another-spotless-fix-commit.

Copy link
Member

@krisstern krisstern left a comment

Choose a reason for hiding this comment

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

LGTM

@krisstern
Copy link
Member

Thanks so much @alecharp!

@krisstern krisstern merged commit 9ab54f9 into jenkinsci:master Nov 6, 2025
18 checks passed
@alecharp alecharp deleted the bug/certificate-not-found branch November 7, 2025 08:42
@MarkEWaite
Copy link
Contributor

@krisstern the plugin BOM test will continue to fail on this test until a new plugin release has been created that includes this fix. Are you available to create the plugin release or would you prefer that I create the plugin release?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tests This PR adds/removes/updates test cases

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants