Skip to content

fix: add jackson-annotations 2.21 and jackson-core 2.21.1 to dependencyManagement#4618

Draft
Copilot wants to merge 2 commits intodependabot/maven/com.fasterxml.jackson.core-jackson-databind-2.21.1from
copilot/sub-pr-4612
Draft

fix: add jackson-annotations 2.21 and jackson-core 2.21.1 to dependencyManagement#4618
Copilot wants to merge 2 commits intodependabot/maven/com.fasterxml.jackson.core-jackson-databind-2.21.1from
copilot/sub-pr-4612

Conversation

Copy link
Contributor

Copilot AI commented Feb 24, 2026

jackson-databind 2.21.1 requires jackson-annotations 2.21, which introduced JsonSerializeAs. WireMock 3.13.2's BOM pins jackson-annotations to 2.20 (missing that class), causing Maven to select the wrong version at runtime and throwing NoClassDefFoundError: com/fasterxml/jackson/annotation/JsonSerializeAs.

Changes

  • pom.xml: Add jackson.annotations.version=2.21 property to track the annotations module's separate versioning scheme (introduced in Jackson 2.20, where annotations dropped the patch segment)
  • pom.xml dependencyManagement: Explicitly pin jackson-annotations (2.21) and jackson-core (2.21.1) so our declarations win over WireMock's BOM during dependency mediation

No test or source changes needed — this is purely a dependency resolution fix.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

@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 Feb 24, 2026
@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Feb 24, 2026
@k8s-ci-robot k8s-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Feb 24, 2026
…cyManagement

jackson-databind 2.21.1 requires jackson-annotations 2.21 which added
the JsonSerializeAs class. WireMock's BOM was pinning jackson-annotations
to 2.20 (which lacks JsonSerializeAs), causing NoClassDefFoundError at
runtime. Explicitly managing all core Jackson modules ensures consistent
versions regardless of transitive BOM overrides.

Co-authored-by: brendandburns <5751682+brendandburns@users.noreply.github.com>
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Copilot
Once this PR has been reviewed and has the lgtm label, please ask for approval from brendandburns. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found 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 size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Feb 24, 2026
Copy link
Contributor Author

Copilot AI commented Feb 24, 2026

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • central.sonatype.com
    • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node /home/REDACTED/work/_temp/ghcca-node/node/bin/node --enable-source-maps /home/REDACTED/work/_temp/copilot-developer-action-main/dist/index.js (dns block)
  • wiremock.org
    • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node /home/REDACTED/work/_temp/ghcca-node/node/bin/node --enable-source-maps /home/REDACTED/work/_temp/copilot-developer-action-main/dist/index.js (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI changed the title [WIP] Build: Bump jackson-databind from 2.20.1 to 2.21.1 fix: add jackson-annotations 2.21 and jackson-core 2.21.1 to dependencyManagement Feb 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants