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

Leave sign_pub_messages off by default. #40206

Merged
merged 2 commits into from Mar 22, 2017

Conversation

Projects
None yet
4 participants
@cro
Copy link
Member

commented Mar 21, 2017

What does this PR do?

Turns sign_pub_messages off by default

Fixes #40203

@salt-jenkins

This comment has been minimized.

Copy link
Contributor

commented Mar 21, 2017

@cro, thanks for your PR! By analyzing the history of the files in this pull request, we identified @rallytime, @s0undt3ch and @cachedout to be potential reviewers.

@cachedout cachedout requested a review from thatch45 Mar 22, 2017

@cachedout cachedout merged commit a27a2cc into saltstack:2016.3 Mar 22, 2017

6 checks passed

default Pull Requests » Salt PR - Main Build #2796
Details
jenkins/PR/salt-pr-clone Pull Requests » Salt PR - Clone #12345 — SUCCESS
Details
jenkins/PR/salt-pr-docs-n Pull Requests » salt-pr-docs-n #5180 — SUCCESS
Details
jenkins/PR/salt-pr-linode-ubuntu14-n Pull Requests » Salt Linode Ubuntu14.04 #9657 — SUCCESS
Details
jenkins/PR/salt-pr-lint-n Pull Requests » Salt PR - Code Lint #9715 — SUCCESS
Details
jenkins/PR/salt-pr-rs-cent7-n Pull Requests » Salt PR - RS CentOS 7 #9859 — SUCCESS
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.