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

fix-35384, fix cmd.run unless #35545

Merged
merged 1 commit into from
Aug 18, 2016
Merged

Conversation

hu-dabao
Copy link
Contributor

What does this PR do?

Fix #35384

What issues does this PR fix or reference?

Previous Behavior

Only one command in the unless list will be executed.

New Behavior

All commands in the unless list will be executed.

Tests written?

No

Please review Salt's Contributing Guide for best practices.

@hu-dabao hu-dabao changed the title fix-35384, fix the logic caused by wrong indent fix-35384, fix cmd.run unless Aug 18, 2016
@rallytime
Copy link
Contributor

@hu-dabao Thanks for fixing this!

@rallytime rallytime merged commit e4e3315 into saltstack:2016.3 Aug 18, 2016
@rallytime rallytime added the ZZZ[Done]-back-ported-bf RETIRED The pull request has been back-ported to an older branch. label Aug 18, 2016
rallytime pushed a commit to rallytime/salt that referenced this pull request Aug 18, 2016
…fails

This is a regression integration test for saltstack#35384, which was fixed by saltstack#35545,
and back-ported to the 2015.8 branch in saltstack#35566.
@hu-dabao hu-dabao deleted the fix-35384 branch August 21, 2016 05:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ZZZ[Done]-back-ported-bf RETIRED The pull request has been back-ported to an older branch.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants