javascript icon indicating copy to clipboard operation
javascript copied to clipboard

fix(yaml): add support for YAML 1.1 parsing in Kubernetes manifests

Open Tanmay-008 opened this issue 2 months ago • 4 comments

##Summary This PR fixes YAML parsing inconsistencies by replacing js-yaml with the modern yaml library, which supports both YAML 1.1 and 1.2 specifications.

##What’s Changed Replaced js-yaml with yaml parser.

Added support for parsing multiple YAML documents using both YAML 1.1 and 1.2 versions.

Updated loadYaml and loadAllYaml functions to accept YamlParseOptions with version selection.

Fixes #2539

Tanmay-008 avatar Oct 14 '25 04:10 Tanmay-008

CLA Missing ID CLA Not Signed

  • :x: The email address for the commit (94ad089f76eb4d60a553deecefb33db6eae2dadf, f7f0ad39c16e6d105170342032c52fe90a0a77fa) is not linked to the GitHub account, preventing the EasyCLA check. Consult this Help Article and GitHub Help to resolve. (To view the commit's email address, add .patch at the end of this PR page's URL.) For further assistance with EasyCLA, please submit a support request ticket.

[APPROVALNOTIFIER] This PR is NOT APPROVED

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

The full list of commands accepted by this bot can be found here.

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 avatar Oct 14 '25 04:10 k8s-ci-robot

It looks like the CLA still needs to be signed.

cjihrig avatar Oct 14 '25 15:10 cjihrig

PR needs rebase.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

k8s-ci-robot avatar Nov 08 '25 12:11 k8s-ci-robot