Skip to content

Replace ghodss/yaml with sigs.k8s.io/yaml - #643

Merged
pmatseykanets merged 1 commit into
rancher:mainfrom
pmatseykanets:switch-to-sigs.k8s.io-yaml
Aug 12, 2026
Merged

Replace ghodss/yaml with sigs.k8s.io/yaml#643
pmatseykanets merged 1 commit into
rancher:mainfrom
pmatseykanets:switch-to-sigs.k8s.io-yaml

Conversation

@pmatseykanets

Copy link
Copy Markdown
Contributor

Issue rancher/rancher#56162

Problem

cmd/writer.go used github.com/ghodss/yaml, unmaintained since 2017, to marshal API objects to YAML.

Solution

The import is now sigs.k8s.io/yaml, the maintained drop-in replacement backed internally by go.yaml.in/yaml. go.mod promotes it to a direct dependency and drops the ghodss/yaml direct requirement (it remains indirect via rancher/norman/types).

@pmatseykanets pmatseykanets self-assigned this Aug 11, 2026
@pmatseykanets
pmatseykanets requested a review from a team as a code owner August 11, 2026 18:58
cmd/writer.go used github.com/ghodss/yaml, unmaintained since 2017,
to marshal API objects to YAML.

The import is now sigs.k8s.io/yaml, the maintained drop-in
replacement backed internally by go.yaml.in/yaml. go.mod promotes
it to a direct dependency and drops the ghodss/yaml direct
requirement (it remains indirect via rancher/norman/types).
@pmatseykanets
pmatseykanets force-pushed the switch-to-sigs.k8s.io-yaml branch from c1e9a95 to ae6108e Compare August 11, 2026 19:06
@pmatseykanets
pmatseykanets merged commit 9245629 into rancher:main Aug 12, 2026
1 check passed
@pmatseykanets
pmatseykanets deleted the switch-to-sigs.k8s.io-yaml branch August 12, 2026 19:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants