Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update jackson to 2.14.1 #92990

Merged
merged 1 commit into from
Jan 17, 2023
Merged

Update jackson to 2.14.1 #92990

merged 1 commit into from
Jan 17, 2023

Conversation

pgomulka
Copy link
Contributor

@pgomulka pgomulka commented Jan 17, 2023 •
edited by grcevski
Loading

Closes #92341

@pgomulka pgomulka added >non-issue :Core/Infra/Core Core issues without another label v8.6.1 v8.7.0 labels Jan 17, 2023
@pgomulka pgomulka self-assigned this Jan 17, 2023
@elasticsearchmachine elasticsearchmachine added the Team:Core/Infra Meta label for core/infra team label Jan 17, 2023
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (Team:Core/Infra)

@pgomulka pgomulka mentioned this pull request Jan 17, 2023
Merged
stu-elastic
stu-elastic approved these changes Jan 17, 2023
View reviewed changes
Copy link
Contributor

@stu-elastic stu-elastic left a comment

Choose a reason for hiding this comment

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

lgtm

grcevski
grcevski approved these changes Jan 17, 2023
View reviewed changes
Copy link
Contributor

@grcevski grcevski left a comment

Choose a reason for hiding this comment

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

LGTM!

@grcevski
Copy link
Contributor

I updated the comment, since we already had an issue opened for this upgrade. Thanks!

@pgomulka pgomulka merged commit d19721b into elastic:main Jan 17, 2023
ChrisHegarty
ChrisHegarty reviewed Jan 17, 2023
View reviewed changes
Copy link
Contributor

@ChrisHegarty ChrisHegarty left a comment

Choose a reason for hiding this comment

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

LGTM

@elasticsearchmachine
Copy link
Collaborator

💔 Backport failed

Status Branch Result
❌ 8.6 Commit could not be cherrypicked due to conflicts

You can use sqren/backport to manually backport by running backport --upstream elastic/elasticsearch --pr 92990

This was referenced Jan 17, 2023
Merged
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ChrisHegarty ChrisHegarty ChrisHegarty left review comments

@grcevski grcevski grcevski approved these changes

@stu-elastic stu-elastic stu-elastic approved these changes

Assignees

@pgomulka pgomulka

Labels
backport pending :Core/Infra/Core Core issues without another label >non-issue Team:Core/Infra Meta label for core/infra team v8.6.1 v8.7.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Upgrade to jackson 2.14.1
5 participants
@pgomulka @elasticsearchmachine @grcevski @stu-elastic @ChrisHegarty