summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* Use a if/else instead to avoid loading possibly invalid values for MemoryMicheal Waltz2015-08-061-3/+4
| | | | | Conflicts: cloud/docker/docker.py
* Set the API version when checking differences in containers and useMicheal Waltz2015-08-061-0/+7
| | | | | | | this to determine the location of the Memory value depending on the version used. In v1.18 and earlier it was ['Config']['Memory'], but in v1.19 it changed to ['HostConfig']['Memory'].
* Use proper HostConfig element which contians the proper Memory value - fixes ↵Micheal Waltz2015-08-061-1/+1
| | | | | | | #1766 Conflicts: cloud/docker/docker.py
* restore mem_limitLars Kellogg-Stedman2015-08-061-0/+6
| | | | | mem_limit got lost in the #1744; this restores it. Thanks to @dgromov for the report.
* Use HostConfig object when creating container with Docker Remote API > 1.15Maksim Losev2015-08-061-32/+53
| | | | | | | | | This is mlosev's patch (from #1208), rebased against devel as of 2790af2. It resolves #1707, which was caused by an API incompatibility between the docker module and server API version 1.19. Conflicts: cloud/docker/docker.py
* Merge pull request #1847 from amenonsen/1842-backport-bisToshio Kuratomi2015-07-311-2/+2
|\ | | | | #1842 backport fixup: module is now self.module
| * Fix dangling parameter references: module is now self.moduleAbhijit Menon-Sen2015-07-311-2/+2
|/
* Simplify distribution testAbhijit Menon-Sen2015-07-301-1/+1
| | | | | If it's Ubuntu, use UbuntuSourcesList; if it's any other apt-friendly distribution, use SourcesList; otherwise, fail.
* fix error occurring with DebianPierre-Louis Bonicoli2015-07-301-6/+5
| | | | Error was: AttributeError: 'SourcesList' object has no attribute 'repos_urls'
* Make SourcesList __init__ method also set self.moduleAbhijit Menon-Sen2015-07-301-7/+8
| | | | | | | | | This was originally required to allow other methods in SourcesList to fail, but subsequent changes rendered that unnecessary, and it's just a cleanup now, and avoids passing in module separately to save(). Conflicts: packaging/os/apt_repository.py
* Clarify HAVE_PYTHON_APT/install_python_apt handling in apt_repositoryAbhijit Menon-Sen2015-07-301-12/+12
| | | | | | | | | | | | | | | 1. Don't test check_mode in both the caller and in the callee. 2. Don't test HAVE_PYTHON_APT inside an if that tests HAVE_PYTHON_APT 3. Don't be irritatingly vague about why the module fails ("You may be seeing this because…"). Note that if «apt-get -y install python-apt» succeeds with rc==0, but for some reason python_apt is not usable afterwards, this will break because the imports in install_python_apt aren't wrapped inside a try/except. In other words, we assume that install_python_apt either succeeds or fails with a traceback. This commit doesn't affect that behaviour.
* Fix call to _expand_ppaAbhijit Menon-Sen2015-07-301-1/+1
|
* docker: fix parsing of docker __version__ stringLars Kellogg-Stedman2015-07-291-0/+1
| | | | | | | | | | | | | | | | | If `docker.__version__` contains non-digit characters, such as: >>> import docker >>> docker.__version__ '1.4.0-dev' Then `get_docker_py_versioninfo` will fail with: ValueError: invalid literal for int() with base 10: '0-de' This patch corrects the parsing of the version string so that `get_docker_py_versioninfo` in this example would return: (1, 4, 0, '-dev')
* Remove validate_certs as the url is not user settable so we always want to ↵Toshio Kuratomi2015-07-221-10/+0
| | | | validate the certificate
* Deprecated _ec2_ami_search now verifies SSL certificatesToshio Kuratomi2015-07-211-6/+19
| | | | | Conflicts: cloud/amazon/ec2_ami_search.py
* Do not erroneously mask exceptionsLars Kellogg-Stedman2015-07-181-1/+5
| | | | | | | | | | | | | | | There was a catch-all `except` statement in `create_containers`: try: containers = do_create(count, params) except: self.pull_image() containers = do_create(count, params) This would mask a variety of errors that should be exposed, including API compatability errors (as in #1707) and common Python exceptions (KeyError, ValueError, etc) that could result from errors in the code. This change makes the `except` statement more specific, and only attempts to pull the image and start a container if the original create attempt failed due to a 404 error from the docker API.
* Check if the gid is setbambou2015-07-181-3/+5
|
* Add notes about loop squashing and 1.9.2 change to install packages in one ↵Toshio Kuratomi2015-07-091-2/+16
| | | | | | | yum transaction Conflicts: packaging/os/yum.py
* Fixes regression introduced by edf1ac1ea6ffd5d44bb2bb62ad320364baf310a3Bobby Calderwood2015-07-081-8/+9
|
* Fix group mod and group add for FreeBSDSean Chittenden2015-07-071-2/+4
|
* Fix problem writing binary content to a temporary file in the uri module.Toshio Kuratomi2015-07-061-11/+18
| | | | | Fixes https://github.com/ansible/ansible/issues/10938 Fixes https://github.com/ansible/ansible/issues/7606
* Restore travis test removed incorrectly by:Toshio Kuratomi2015-07-051-0/+1
| | | | https://github.com/ansible/ansible-modules-core/pull/1671
* Merge pull request #1671 from msabramo/stable-1.9_GH-133Brian Coca2015-07-042-4/+31
|\ | | | | Backport issue #133 fix to stable-1.9 branch
| * Python2.4 fixesMatt Martz2015-07-042-3/+6
| | | | | | | | | | | | | | | | * Ignore accelerate.py * Don't use a lambda or a ternary Conflicts: .travis.yml
| * This change is in response to issue #133.verm6662015-07-041-1/+25
|/ | | | | | | The original problem is: apt_repository.py connect to launchpad on every playbook run. In this patch apt_repository.py checks if required repository already exists or not. If no - paa will be added, if yes - just skip actions.
* updated upgrade to a more sensible default as the previous was prone to ↵Brian Coca2015-07-031-3/+7
| | | | | | confusion fixes #1667
* Merge pull request #1622 from dresden-weekly/fix_win_file_touchBrian Coca2015-06-301-1/+1
|\ | | | | fixed win_file state=touch
| * fixed win_file state=touchAndreas Reischuck2015-06-271-1/+1
| |
* | Merge pull request #1618 from vroetman/stable-1.9Brian Coca2015-06-301-0/+1
|\ \ | | | | | | apache2_module documetation update
| * | apache2_module documetation updateRoetman, Victor2015-06-261-0/+1
| |/ | | | | | | requires a2enmod and a2dismod
* | minor doc fixes on win_templateBrian Coca2015-06-291-13/+8
|/
* Updating version for 1.9.2-1 releaseJames Cammarata2015-06-241-1/+1
|
* document file size limit for win_copy moduleJon Hawkesworth2015-06-191-3/+9
|
* Restore setting cachedir when non-root but don't take a useless cachedir ↵Toshio Kuratomi2015-06-161-0/+8
| | | | | | | | parameter to the function * Revert "Remove unused code" This reverts commit bcfba0c05098696b6e770335870a9c22792fec38. * Re-add the changes to remove cachedir as a parameter
* Renamed previous pkgs variable to installed_pkgs as spotted by @strahinjaEdward Torbett2015-06-151-3/+3
|
* Corrected pkg to pkgs as noted by @abadgerEdward Torbett2015-06-151-1/+1
|
* Added multi package operation to remove as suggested by @abadger. Adding to ↵Edward Torbett2015-06-151-9/+14
| | | | latest is a little more complex due to '*' support.
* Comments by @abadgerEdward Torbett2015-06-151-1/+3
|
* Rather than executing yum once per package, execute yum once for all ↵Edward Torbett2015-06-151-13/+17
| | | | supplied packages. This is necessary when performing a yum upgrade involving multiple dependent packages installed from RPM, for example when upgrading from PostgreSQL 9.0.11 to 9.0.21 on a Red Hat server.
* corrected version added which I got wrong while rebasingJon Hawkesworth2015-06-102-2/+2
|
* Fix win_copy problems described here: ↵Jon Hawkesworth2015-06-105-70/+144
| | | | | | https://github.com/ansible/ansible-modules-core/issues/1404 and update documentation.
* Use a list comprehension instead of map and lambdaToshio Kuratomi2015-05-291-1/+1
|
* Strip spaces around permsDennis Rowe2015-05-291-1/+1
|
* Import ansible module_utils at bottom of file to not mess with line numbers ↵Toshio Kuratomi2015-05-291-2/+3
| | | | in tracebacks
* add :// url support for EL 5Jonathan Mainguy2015-05-291-0/+32
|
* - Fixed annoying bug that disablerepo was essentially broken if a package ↵Zoltan Kozma2015-05-291-9/+22
| | | | | | needed updating by state latest. - Replaced some unsafe practice with default parameters. However looking at the code this does not seem to matter much as the calling functions always seem to supply these parameters anyway.
* stat doc fixBrian Coca2015-05-281-4/+4
| | | | fixes #1371
* sleep when only doing a time delay to avoid cpu churnBrian Coca2015-05-281-1/+8
|
* value should be string as that is what it compares againstBrian Coca2015-05-281-1/+1
|
* Properly flip default for verifying server cert. Add nice error messages ↵Toshio Kuratomi2015-05-281-1/+5
| | | | when the cert is invalid