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

[2016.11] Merge forward from 2016.3 to 2016.11 #40240

Merged
merged 23 commits into from Mar 23, 2017

Conversation

rallytime
Copy link
Contributor

Conflicts:

  • pkg/windows/req.txt

Replaces #40237

Foxlik and others added 23 commits March 6, 2017 18:36
In some cases (e.g. Ubuntu cloud-init) authorized_keys may contain quoted spaces as part of options:
`no-port-forwarding,no-agent-forwarding,no-X11-forwarding,com‌​mand="echo 'Please login as the user \"ubuntu\" rather than the user \"root\".';echo;sleep 10" ssh-rsa xxxxxx`
which confused the split in _replace_auth_key() in modules/ssh.py.
Needed for lint
This provides a natural solution to the chicken-and-egg problem
identified in the FAQ. It also changes the name of the event in the
reactor example, as I think we may have stopped sending the legacy
(i.e. not-namespaced) events, so the "minion_start" event in the example
would be incorrect.
…mple

Link to minion start reactor example from FAQ.
Leave sign_pub_messages off by default.
…uthorized_keys

Use regular expression instead of split when replacing authorized_keys
Conflicts:
  - pkg/windows/req.txt
@rallytime rallytime merged commit 720a362 into saltstack:2016.11 Mar 23, 2017
@rallytime rallytime deleted the merge-2016.11 branch March 23, 2017 14:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants