Commit graph

8954 commits

Author SHA1 Message Date
Robin Roth
2256ae0793 fix git switch branches in combination with depth on old git versions (#3492)
* fix git switch branches in combination with depth

* the old implementation is correct, but relies on git working correctly
  (as only newer versions do)
* with some older git version (e.g. 1.8.3), git fetch remote branch does
  not work, if the branch does not yet exist locally
* this patch works around that without explicitly checking the git version
* future refactoring is needed to reduce the number of fetches to a
  minimum; but this patch makes the code "correct"

* add git version requirements to doc

* replace set-branches with git 1.7.1 compatible version
2016-04-21 07:40:54 -07:00
GGabriele
0f57bb1513 Fixing bugs 2016-04-21 16:01:04 +02:00
ThomasSteinbach
8dc0276533 docker: fix type issue (#3491)
* fix type issue

* fix type issue for 'volumes-from'
2016-04-21 13:28:26 +02:00
GGabriele
c25610777f Adding nxos_ip_interface 2016-04-21 10:48:45 +02:00
Rene Moser
1f92e77650 git: fix status to use porcelain, ignore user configuration
From the git status doc:

    --porcelain

        Give the output in an easy-to-parse format for scripts. This is similar to the short output, but will remain stable across Git versions and regardless of user configuration. See below for details.

User configuration can break the git module from working as expected.
2016-04-21 10:41:03 +02:00
Matt Davis
c6f5475afe Merge pull request #3501 from dreamcat4/fix/win-modules-strict-mode-fixes
Fix/win modules strict mode fixes
2016-04-20 16:06:01 -07:00
Dreamcat4
b3584c5ffe fix: win_get_url - fixes for strict-mode 2016-04-20 20:26:44 +01:00
Dreamcat4
bb229c8fa2 fix: win_stat - fixes for strict-mode 2016-04-20 20:26:27 +01:00
Adrian Likins
d83c269b82 improve redhat_subscription documentation (#3348)
* add subscription-manager's --type, --consumerid, --name option support

* redhat_subscription: More docs on consumerid
2016-04-20 17:33:08 +02:00
Michael Scherer
a144663953 Fix error reporting for fetch_key (#2662)
Since fetch_url already take care of the exception, the try/except
clause is no longer working, so replace it with proper status
checking, thus permitting to remove urlib2 from the import list.
2016-04-19 18:03:14 -04:00
Constantin
b3bcd5a433 Added improvements and documented return structure (#2320) 2016-04-19 18:01:27 -04:00
jjshoe
bfa0a94c27 Fix an issue where by a call to ec2.get_image wasn't wrapped in a try/except, and would occasionally throw an exception in the wild. Also greatly simplifies two of the ugliest known loops to man down into a single loop. (#2554) 2016-04-19 18:00:16 -04:00
Brian Coca
c64b7a71fb removed usless default doc 2016-04-19 17:48:11 -04:00
Justin Kennedy
7a2b00b625 Updating examples to include required fields (#3242)
Updated examples to include required fields (host, username).  Also updated src option to be listed as required.
2016-04-19 17:46:34 -04:00
Brian Coca
53c4827da5 added missing version_added 2016-04-19 17:45:36 -04:00
Pål Nilsen
eb6192c050 Add optional force parameter for apache2_module (#3251)
* Add optional force parameter for apache2_module

Fixes #2499

* Add documentation for the force parameter

* Clarify in description that -f(orce) is a Debian parameter
2016-04-19 17:44:08 -04:00
Jan Hutař
9ab4780b25 Add support for Satellite/Spacewalk which have slightly different API to Hosted (#3289) 2016-04-19 17:43:28 -04:00
GGabriele
1977be60b2 Adding nxos_vrrp 2016-04-19 21:00:36 +02:00
GGabriele
e7fbafce14 Adding nxos_ping 2016-04-19 20:55:25 +02:00
Michael Scherer
b5ef21ef89 Suggest using uri if curl is detected
Some people use the command module to run curl not to
download file, but to interact with remote HTTP api, and the
get_url suggestion is not suitable. So adding uri in the warning
would give a more meaningful warning.

Suggested by Tahvok on #ansible irc channel.
2016-04-19 20:00:02 +02:00
Dennis Conrad
c1787da7ff Fix UnboundLocalError if 'access_key_ids' not supplied
'key_ids' is referenced before it is assigned, causing the module to fail with a UnboundLocalError instead of failing gracefully with a helpful error message.  This very small patch moves the assignment of 'key_ids' to before the variable is referenced.
2016-04-19 19:53:21 +02:00
jedelman8
ef904f67a6 fix index for portchannel 2016-04-19 10:01:18 -07:00
Toshio Kuratomi
98322a777f Merge pull request #3466 from robinro/git-full-clone-with-hash
git: fall back to full clone if version is hash
2016-04-19 07:42:06 -07:00
Toshio Kuratomi
f39e2343c4 Fix apt module needing another format character 2016-04-19 07:33:55 -07:00
Toshio Kuratomi
609d4cf84b Merge pull request #3464 from robinro/fix-pull-git-branch-#3456
git: with depth only fetch actual version, don't use refs/heads/*
2016-04-19 07:00:09 -07:00
GGabriele
bbe85bf1fa fix conditional for py24 2016-04-19 14:59:01 +02:00
GGabriele
2466e407f4 fix conditional for py24 2016-04-19 14:57:06 +02:00
GGabriele
14e334285f fix conditional for py24 2016-04-19 14:53:49 +02:00
GGabriele
67d416576f fix conditional for py24 2016-04-19 14:48:53 +02:00
jedelman8
1e5dd9674e fix docstring 2016-04-19 05:48:50 -07:00
jedelman8
8a8025ba2d fix conditional for py24 2016-04-19 05:43:23 -07:00
jedelman8
30a6984983 force travis build 2016-04-19 05:40:28 -07:00
jedelman8
f9f894ef95 fix conditional for py24 2016-04-19 05:34:59 -07:00
jedelman8
8a53755a61 minor test 2016-04-19 05:12:33 -07:00
Robin Roth
b67d15f385 fall back to full clone if version is hash
if version is not a branch or tag (i.e. a hash), we need a full checkout to be able to switch to it
2016-04-19 13:45:43 +02:00
Robin Roth
c398eb5399 with depth only fetch actual version
* don't use refs/heads/branchname for branches
* for tags it's needed thou
* fixes #3456
2016-04-19 12:47:36 +02:00
Jeroen Geusebroek
766671f1e1 Allow an empty password when adding a new MySQL user. (#3445) 2016-04-19 10:44:52 +02:00
Kyle Dyroff
2b003a2773 Log git error stdout/stderr to fail_json (#3022) 2016-04-19 08:56:48 +02:00
Dag Wieers
983cdd00bc Making unarchive idempotent (#3307)
* WIP: Making unarchive idempotent

Currently unarchive is not idempotent and has many rough edges and bugs.
The current release is a workable improvement on many fronts:

- zip support is now idempotent (but gtar lacks check-mode)
- New option `exclude` to exclude specific paths/files
- New option `keep_newer` to exclude newer files on target
- New option `extra_opts` to influence unzip/gtar (like synchronize module)

The following items are still ongoing:

- Implement CRC32 support for .zip files
- Re-implement the zip support using native zipfile module
- Re-implement the gtar support using native tarfile/gzip/bz2 modules (lzma external)
- Implement check-mode (works in gzip, but fails using gtar)
- Implement diff-mode (discuss an appropriate output model, like synchronize module)

The re-implementation of unzip/gtar support using native python modules will not only simplify the codebase, additional functionality can be implemented correctly and identically, which is currently not possible. (Other archives could be implemented using native modules equally, incl. options)

* Assorted fixes to zip support (during quality checks)

- Support both rw---- and rwx--- permstr
- Better file type support (more qa needed)
- Symlink support
- Include fix from #3229

* Implement zip diff-mode (itemized change) and avoid changes permissions every time (!)

This commit implements:
- rsync-compatible itemized-change output in diff-mode (using zip)
- no longer changing permissions unconditionally (when idempotent)

* Small fixes to itemized change output

* Fixes to user/group ownership changes

- The implementation of user/group ownership is a bit more complex for idempotency
- We report when a ZIP file incorrectly tags a directory as a file/link
- We only offer diff output when there is a change

* Fix the handling of includes and excludes for unzip

* Remove test output from output (confuses easily)

* Logic and performance improvements to ownership handling, and umask fix

* Handle special files (type '?')

* Make exceptions compatible with python 2.4

* Implement CRC32 support

* Revert some unintended/unknown changes ?

* Taking over maintenance as offered by current maintainer

* Fix support for white-spaces in filenames

* Remove/rename incorrect regex

* Ensure that fat executables end up with execute permission

* Remove check_result from output when unchanged

* When unarchiving as a user, or when owner/group/mode is supplied --diff is insufficient

Only way to be sure is to check request with what is on disk (as we do for zip).
Leave this up to set_fs_attributes_if_different() instead of inducing a (false) change

* By default, don't send confusing check_results in verbose output

This fixes #74.
2016-04-18 16:31:06 -04:00
jedelman8
ed421149e6 update return doc string 2016-04-18 13:18:30 -07:00
jedelman8
ea5ed26e04 update return doc string 2016-04-18 13:17:06 -07:00
GGabriele
091c5b6a4e Fixed RETURN string 2016-04-18 18:00:47 +02:00
NielsH
ab699a0953 Resolve idempotency issue with virtual apt package (#3449)
Virtual packages were always marked as upgradable which caused the changed-state even though nothing changed.
2016-04-18 11:14:57 -04:00
GGabriele
026ddfcc46 Fixed RETURN string 2016-04-18 16:50:26 +02:00
GGabriele
6afeb471ad Fixed RETURN string and added notes 2016-04-18 16:48:39 +02:00
GGabriele
36942ec24b Fixed RETURN string 2016-04-18 16:28:15 +02:00
Tobias Wolf
dba53b5543 Add diff to apt.py (#2944)
for install, remove, deb_install, and upgrade.

Since apt has very commonly familiar output, just use the normal output
from apt-get or aptitude -- trimmed to the interesting parts -- to show
to the user if she specified --diff on the CLI.

Uses the recent support for the `diff['prepared]` key.

Fixes ansible/ansible#10239
2016-04-18 10:18:07 -04:00
GGabriele
fff9c9ed44 Fixing nxos_vrf 2016-04-18 15:48:37 +02:00
GGabriele
6a412e55e9 Adding nxos_vrf_interface 2016-04-18 15:27:21 +02:00
GGabriele
3c51d11244 Adding nxos_vrf 2016-04-18 15:25:35 +02:00