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

[2017.7] Merge forward from 2016.11 to 2017.7 #42363

Merged
merged 77 commits into from Jul 18, 2017

Conversation

rallytime
Copy link
Contributor

Conflicts:

  • salt/grains/core.py
  • salt/modules/git.py
  • salt/modules/pillar.py
  • salt/states/saltmod.py
  • salt/states/user.py
  • tests/integration/states/handle_iorder.py

garethgreenaway and others added 30 commits June 29, 2017 10:09
Restrict set_umask to mkstemp call only
When a command is being executed using salt.utils.timed_subprocess.TimedProc,
and that command does not exist in the PATH, it will result in an
OSError/IOError being raised. We catch this and then raise a
CommandExecutionError with some information, including the command being
run. However, when output_loglevel=quiet, we don't want any details of
the command to be visible.

This commit redacts the command details in these corner cases.
This can be used with debug logging enabled to troubleshoot which
registries this function is attempting to authenticate.
Fix saltstack#42115: parse libcloud "rc" version correctly
Prevent command from showing in exception when output_loglevel=quiet
Fix pillar.get when saltenv is passed
Fix regression in CLI pillar override for salt-call
Fixes this exception:
An exception occurred in this state: Traceback (most recent call last):
  File "/var/tmp/.syops_b1274f_salt/py2/salt/state.py", line 1733, in call
    **cdata['kwargs'])
  File "/var/tmp/.syops_b1274f_salt/py2/salt/loader.py", line 1652, in wrapper
    return f(*args, **kwargs)
  File "/var/tmp/.syops_b1274f_salt/py2/salt/states/gpg.py", line 119, in present
    ret['comment'].append('Adding {0} to GPG keychain'.format(name))
AttributeError: 'str' object has no attribute 'append'
rallytime and others added 27 commits July 11, 2017 15:57
Some minor doc fixes for dnsutil module so they'll render correctly
DOCS: describe importing custom util classes
Emptying the password was only done if the user already existed, not
when creating.

Signed-off-by: Nicolas Geniteau <nicolas@cri.epita.net>
This allows a version of 'latest' to work when just a name and version
is passed.
Multiple empty_password fixes for state.user
…dentity_files_with_user

Fixes to modules/git.py
Add clarification to salt ssh docs about key auto-generation.
Handle libcloud objects that throw RepresenterErrors with --out=yaml
Add some clarity to "multiple quotes" section of yaml docs
pkg.installed: pack name/version into pkgs argument
Update minion restart section in FAQ doc for windows
While not fatal, this could potentially cause problems running this
test on Windows.
is_windows is a function, not a propery/attribute
Conflicts:
  - salt/grains/core.py
  - salt/modules/git.py
  - salt/modules/pillar.py
  - salt/states/saltmod.py
  - salt/states/user.py
  - tests/integration/states/handle_iorder.py
@rallytime
Copy link
Contributor Author

Go Go Jenkins!

@rallytime rallytime merged commit 587138d into saltstack:2017.7 Jul 18, 2017
@rallytime rallytime deleted the merge-2017.7 branch July 18, 2017 18:40
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