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.

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

[PR #2014/68fb0a31 backport][stable-7] Quote scalars with colons in flow style collection #2017

Commits on Mar 13, 2024

  1. Quote scalars with colons in flow style collection (#2014)

    Quote scalars with colons in flow style collection
    
    SUMMARY
    
    Older versions of libyaml and pyyaml don't support colons in scalars that appear in flow style collections. This adds quotes to the handful of cases where they are used in the integration tests.
    This was uncovered by downstream testing. I am only able to reproduce the problem on the supported ee-minimal-rhel8 image. I believe this is because even though pyyaml should be recent enough, it has likely been compiled against an older version of libyaml that does not have the fix.
    See:
    yaml/libyaml#104
    yaml/pyyaml#45
    It's worth noting that running ansible-lint --fix will undo these changes, which is how I think they were made in the first place.
    
    ISSUE TYPE
    
    Bugfix Pull Request
    
    COMPONENT NAME
    
    ADDITIONAL INFORMATION
    
    Reviewed-by: Bikouo Aubin
    Reviewed-by: GomathiselviS
    (cherry picked from commit 68fb0a3)
    gravesm authored and patchback[bot] committed Mar 13, 2024
    Configuration menu
    Copy the full SHA
    5450c01 View commit details
    Browse the repository at this point in the history