Commit Graph

62091 Commits

Author SHA1 Message Date
Mike Place
11e39e7203 Merge pull request #33712 from meaksh/fix-for-groupadd-module-failures-in-SLE11-2015.8
Fix for groupadd execution module failures in SLES11 systems
2016-06-03 09:13:06 -07:00
Mike Place
2c450a7494 Merge pull request #33718 from rallytime/bp-33700
Back-port #33700 to 2015.8
2016-06-03 09:10:44 -07:00
Mike Place
b9133326c8 Merge pull request #33726 from jtand/sysmod_skip_valid_docs_glance
glance.warn_until shouldn't be checked for a doc string
2016-06-03 09:06:08 -07:00
Mike Place
b07701f0a0 Merge pull request #33727 from terminalmage/issue33725
Fix git_pillar edge case for remote repos without a master branch
2016-06-03 09:03:59 -07:00
Mike Place
015e50cec8 Merge pull request #33728 from jfindlay/test_state_test
Make `configurable_test_state` configurable in test mode
2016-06-03 09:02:57 -07:00
Mike Place
1cf8fe3f1d Merge pull request #33729 from twangboy/fix_win_servermanager
Add exclude option to win_servermanager
2016-06-03 08:53:13 -07:00
Mike Place
6c150d840d Merge pull request #33743 from vutny/drop-debian-community-repo-doc
Debian installation docs: drop section about community-maintained repo
2016-06-03 08:29:45 -07:00
Justin Anderson
ff2dae103d ubuntu doesn't install default gems when ruby is installed 2016-06-03 08:54:51 -06:00
Pablo Suárez Hernández
ab738416ba pylint fix 2016-06-03 13:57:13 +01:00
Pablo Suárez Hernández
bf27e5d36e test_members cleanup 2016-06-03 13:25:05 +01:00
Pablo Suárez Hernández
ba815dbf76 improvements on groupadd unit tests 2016-06-03 13:17:13 +01:00
Justin
504df9a65a Fixed lint error 2016-06-03 04:28:42 -06:00
Pablo Suárez Hernández
3bbc5ae0d9 one line is better 2016-06-03 10:08:32 +01:00
Justin Findlay
87e018af2a states.test.configurable_test_state: add unit tests 2016-06-02 17:20:36 -06:00
twangboy
2de91d166f Fix docstring 2016-06-02 16:41:13 -06:00
Justin Anderson
0cb1bfa0d3 Removed extra : 2016-06-02 16:35:40 -06:00
Erik Johnson
d8ba7ed5a5 Fix git_pillar edge case for remote repos without a master branch
When the remote repository does not have a master branch, on the initial
checkout there will be no ref in refs/remotes/origin for this branch,
leading to an error being logged and the checkout failing.

Instead of failing the checkout, in these cases we'll create the initial
local HEAD based on the ref we will eventually check out, since we know
for a fact that this ref exists.

Resolves #33725
2016-06-02 17:33:07 -05:00
Justin Anderson
ebee8a89af glance.warn_until shouldn't be checked for a doc string 2016-06-02 16:30:36 -06:00
twangboy
9870479d99 Add exclude option to state 2016-06-02 16:04:19 -06:00
twangboy
50bd76e206 Add exclude option 2016-06-02 15:49:56 -06:00
Justin Findlay
c2d0679c4b states.test.configurable_test_state: refactor change_data
Also adjust continued string indentation.
2016-06-02 15:23:32 -06:00
Justin Findlay
f06ff1af1f states.test.configurable_test_state test mode
Make `configurable_test_state` configurable even in test mode and return
the correct result in test mode as per
https://docs.saltstack.com/en/latest/ref/states/writing.html#return-data.
2016-06-02 15:01:19 -06:00
Justin Anderson
86f59b3e80 Made more pythonic 2016-06-02 14:37:05 -06:00
Mike Place
635efa248b Change to just surround the mkdir 2016-06-02 14:36:56 -06:00
Justin Anderson
2f36f34981 Fixed salt.util import. Added status check to make sure external resource is available 2016-06-02 14:32:08 -06:00
Mike Place
21b7123a60 Catch oserror for race condition
fixup #33653
2016-06-02 14:27:05 -06:00
Mike Place
56c0a42e12 Create missing jid dir if it doesn't exist (#33653)
Closes #33554
2016-06-02 16:20:33 -04:00
Jεan Sacren
a6a446121a Fix speed issue
Erik Johnson pointed out the speed issue with compelling numbers:

>>> import timeit
>>> t1 = timeit.Timer('clock == "localtime"', 'clock = "localtime"')
>>> t2 = timeit.Timer('clock.__eq__("localtime")', 'clock = "localtime"')
>>> t1.repeat()
[0.061421871185302734, 0.0285952091217041, 0.026736021041870117]
>>> t2.repeat()
[0.1310901641845703, 0.09536910057067871, 0.09479284286499023]

I'll absolutely fix it by using the simple '==' operator.

Thank you!
2016-06-02 16:10:35 -04:00
Jεan Sacren
a41146730a Fix incorrect args passed to timezone.set_hwclock
On Gentoo, we should check the value passed in to timezone.set_hwclock
and only allow 'UTC' or 'localtime' for argument. If incorrect value is
passed in, it could subsequently fry the important config file.

There is another closely related issue on Gentoo. The value to set
'clock' variable in the config file is either 'UTC' or 'local'. In case
of localtime arg, we should convert the value of 'localtime' to 'local'
when setting that 'clock' variable.
2016-06-02 16:10:29 -04:00
Mike Place
137f0b19f3 Merge pull request #33611 from TargetHolding/2015.5
2015.5
2016-06-02 12:32:58 -07:00
Mike Place
8a566ff4b9 Merge pull request #33654 from twangboy/fix_win_servermanager
Fix win servermanager
2016-06-02 10:55:45 -07:00
Mike Place
996ff56dd4 Merge pull request #33679 from terminalmage/issue33424
Only compile the template contents if they evaluate to True
2016-06-02 10:20:00 -07:00
Justin Anderson
400a71ec33 Removed redundancies 2016-06-02 11:16:23 -06:00
Justin Anderson
91db411bea A couple lint fixes 2016-06-02 09:03:10 -06:00
Justin Anderson
c97f3319b9 Add check for gem binary 2016-06-02 08:56:09 -06:00
Denys Havrysh
8621f5be54 Debian installation docs: drop section about community-maintained repository 2016-06-02 17:50:47 +03:00
Pablo Suárez Hernández
a53dc192c9 fix groupadd module for sles11 systems 2016-06-02 15:22:54 +01:00
twangboy
6c7b21676a Fix lint and tests 2016-06-01 17:10:55 -06:00
twangboy
4775e6bdf0 Add additional params to state 2016-06-01 17:10:55 -06:00
twangboy
b0af32346d Add additional params to install and remove 2016-06-01 17:10:55 -06:00
Justin Anderson
210aceb402 Refactored tests to not use return messages 2016-06-01 16:57:37 -06:00
Erik Johnson
9da40c4437 Append empty dictionaries for saltenvs with no top file
Rather than excluding them from the get_tops() return data at all, just
populate them with empty dictionaries. This makes it easier to unit test
things like merge order, while still being functionally the same as
exlcluding the saltenv from get_tops() altogether.
2016-06-01 17:31:29 -05:00
Mike Place
c8dc70b96a Merge pull request #33685 from jfindlay/get_url_test
modules.cp.get_url: add test for https://
2016-06-01 15:25:41 -07:00
Mike Place
5e022ff29c Merge pull request #33581 from dincamihai/2015.8
Call zypper refresh after adding/modifying a repository
2016-06-01 15:25:11 -07:00
Justin Anderson
9d437bd45d Removed artifact from testing 2016-06-01 15:32:41 -06:00
Justin Anderson
134e1fa888 Fixed typos, and added destructiveTest decorator 2016-06-01 15:25:18 -06:00
Justin Anderson
37bc3ad8fd Fixed typo, uninstalled to uninstall 2016-06-01 15:16:15 -06:00
Mike Place
069ee15b7c Merge pull request #33681 from rallytime/bp-33599
Back-port #33599 to 2015.8
2016-06-01 14:14:29 -07:00
Mike Place
13537c4891 Merge pull request #33396 from babilen/issue-33393
Issue 33393
2016-06-01 14:12:03 -07:00
Justin Findlay
2b5035fdc0 modules.cp.get_url: add test for https:// 2016-06-01 15:10:12 -06:00