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

Use SnakeYAML Engine instead of Jackson to work with YAML #4754

Closed
asomov opened this issue Jan 11, 2023 · 6 comments
Closed

Use SnakeYAML Engine instead of Jackson to work with YAML #4754

asomov opened this issue Jan 11, 2023 · 6 comments
Assignees
@asomov
Milestone
6.5.0

Comments

@asomov
Copy link
Contributor

asomov commented Jan 11, 2023

Is your enhancement related to a problem? Please describe

Jackson does not support all the features of SnakeYAML Engine (anchors and aliases, security settings etc)

Describe the solution you'd like

Migrate completely to SnakeYAML Engine

Describe alternatives you've considered

No response

Additional context

I can try to contribute
@asomov asomov changed the title Use SnakeYAML Engine instead of Jackson to parese YAML Use SnakeYAML Engine instead of Jackson to work with YAML Jan 11, 2023
@asomov asomov mentioned this issue Jan 11, 2023
Closed
@asomov
Copy link
Contributor Author

asomov commented Jan 24, 2023

@sunix I have the PR already

@manusa
Copy link
Member

manusa commented Jan 24, 2023

Hi, don't worry, you were assigned the issue just for bookkeeping reasons :)

@asomov
Copy link
Contributor Author

asomov commented Jan 24, 2023

feel free to ping me when I can help

@manusa
Copy link
Member

manusa commented Feb 9, 2023

Fixed by #4836 (#4753)

@manusa manusa added this to the 6.5.0 milestone Feb 9, 2023
@manusa manusa closed this as completed Feb 9, 2023
@mswintermeyer
Copy link

Hey folks, thanks for addressing this! When do you think a release of kubernetes-client will be released that includes this fix? kubernetes-client is incompatible with snakeyaml 2.x in the meantime.

This was referenced Mar 6, 2023
Merged
Closed
@manusa
Copy link
Member

manusa commented Mar 6, 2023 •
edited

We want to release v6.5.0 by the end of this week

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

@asomov asomov

Labels
None yet
Projects
None yet
Milestone
6.5.0
Development

No branches or pull requests
3 participants
@manusa @asomov @mswintermeyer