salt icon indicating copy to clipboard operation
salt copied to clipboard

[master] Fix options parsing of highstate returner

Open Guillaume-COEUGNET opened this issue 1 year ago • 1 comments

What does this PR do?

Fix Issue #66816

What issues does this PR fix or reference?

Fixes

Previous Behavior

Highstate returner doesn't take in configuration of some "report_" options

New Behavior

Now the returner use configuration of options report_failures, report_changes and report_everything and send report accordingly.

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?

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.

Guillaume-COEUGNET avatar Aug 22 '24 13:08 Guillaume-COEUGNET

Hi there! Welcome to the Salt Community! Thank you for making your first contribution. We have a lengthy process for issues and PRs. Someone from the Core Team will follow up as soon as possible. In the meantime, here’s some information that may help as you continue your Salt journey. Please be sure to review our Code of Conduct. Also, check out some of our community resources including:

There are lots of ways to get involved in our community. Every month, there are around a dozen opportunities to meet with other contributors and the Salt Core team and collaborate in real time. The best way to keep track is by subscribing to the Salt Community Events Calendar. If you have additional questions, email us at [email protected]. We’re glad you’ve joined our community and look forward to doing awesome things with you!

welcome[bot] avatar Aug 22 '24 13:08 welcome[bot]

This should also be made against the earliest supported branch where the bug exists... probably 3006.x

twangboy avatar Jul 01 '25 19:07 twangboy