salt icon indicating copy to clipboard operation
salt copied to clipboard

[master] fix yaml output

Open tomdoherty opened this issue 6 months ago • 0 comments

In b9be2de, OrderedDict was replaced with HashableOrderedDict. Add logic to yamldumper to handle the new type.

What does this PR do?

What issues does this PR fix or reference?

Fixes #66594

Previous Behavior

% sudo salt-call state.show_sls salt.netbox-download --out=yaml
local: NULL
%

New Behavior

% sudo salt-call state.show_sls salt.netbox-download --out=yaml
local:
  salt.netbox-download_manage_file_netbox-download.sh:
    file:
    - name: /bootstrap/netbox-download.sh
    - source: salt://salt/netbox-download/files/netbox-download.sh
    - mode: '0755'
    - managed
    - order: 10002
...

Merge requirements satisfied?

[NOTICE] Bug fixes or features added to Salt require tests.

  • [ ] Docs
  • [ ] Changelog - https://docs.saltproject.io/en/master/topics/development/changelog.html
  • [ ] Tests written/updated

Commits signed with GPG?

Yes/No

Please review Salt's Contributing Guide for best practices, including the PR Guidelines.

See GitHub's page on GPG signing for more information about signing commits with GPG.

tomdoherty avatar Aug 07 '24 14:08 tomdoherty