Commit Graph

19304 Commits (08f6283c3fd136c6bba09e33ead18c8c767bc3af)
 

Author SHA1 Message Date
James Cammarata 08f6283c3f Merge pull request #15747 from larsks/bug/15744
do not erroneously set gathered_facts=True
9 years ago
James Cammarata 64a738ce51 Merge branch 'zenbot-refresh-inventory-exit' into devel 9 years ago
James Cammarata 3439bcc744 Merge branch 'refresh-inventory-exit' of https://github.com/zenbot/ansible into zenbot-refresh-inventory-exit 9 years ago
James Cammarata 0d7530e6da Change error about loop variable in use to a warning 9 years ago
Toshio Kuratomi 8a184381af Strip junk after JSON return. (#15822)
Fixes #15601
9 years ago
James Cammarata 9d9a451b34 Simply being in an always block shouldn't mean a host did not fail
Previously the changed code was necessary, however it is now problematic
as we've started using the is_failed() method in other places in the code.
Additional changes at the strategy layer should make this safe to remove
now.

Fixes #15625
9 years ago
James Cammarata 37737ca6c1 Merge pull request #15466 from alikins/pr_10555_cobbler_auth_rebase
add cobbler api authentication options
9 years ago
Toshio Kuratomi 18587842c6 Exclude the sample directory from tests 9 years ago
James Cammarata 33de7707c9 Do not include params when getting role vars in certain situations
In VariableManager, we fetch the params specifically in the next step,
so including them in the prior step is unnecessary and could lead to things
being overridden in an improper order.

In Block, we should not be getting the params for the role as they are
included earlier via the VariableManager.

Fixes #14411
9 years ago
James Cammarata 161388226b Merge branch 'dagwieers-samples-to-test' into devel 9 years ago
James Cammarata 67c32ac30a Merge branch 'samples-to-test' of https://github.com/dagwieers/ansible into dagwieers-samples-to-test 9 years ago
Toshio Kuratomi 9096901c1f Update submodule refs 9 years ago
Toshio Kuratomi 99e3880181 small python3 fix so that ping will run on python3 9 years ago
Matt Martz accf40d8a8 Use httptester docker image for http tests (#15811)
* Use httptester docker image for http tests

* When not running with an httptester linked container, use public test sites
9 years ago
Brian Coca 4561be00af simplified inventory error messaging 9 years ago
Devon Harvey e33b97ed1f Fix typo in intro_dynamic_inventory.rst (#15810) 9 years ago
Tim Rupp a685fa5543 Add port argument for bigsuds (#15434)
This patch adds the port argument as a valid parameter to the f5_spec.

This argument is supported in bigsuds version 1.0.4 and greater, so
this patch uses the __version__ variable of the bigsuds module to
determine when the port value should be honored by the module.
9 years ago
Brian Coca f576082949 restore old jsonfile behaviour on key expiration
fixes #14456, now it won't expire keys in middle of a play when they
were 'valid' at 'gather time'.
9 years ago
Toshio Kuratomi dae1d4dbb7 Add changelog entry for ec2_customer_gateway (in 2.2) 9 years ago
Brian Coca 47d05e9b58 fix bad assignment, method modifies by ref already
fixes #15694
9 years ago
Toshio Kuratomi c730af5dc2 Remove reload from arg related tests. Changes to how ziploader passes args mean we don't need reload anymore. (#15782) 9 years ago
Matthew Stoltenberg ddd9f92255 add repr for hostvars (#15793)
* allows passing full hostvars to a module
9 years ago
Toshio Kuratomi 9e88fa21f5 Update submodule refs 9 years ago
Toshio Kuratomi 7ccb08cc7e Switch to a different url for testing SNI right now. (#15798) 9 years ago
Darren Birkett 30decd5e0a Fix a couple of typos in guide_rax.rst (#15795) 9 years ago
Kamjar Gerami 724e692f54 fixes-#15685-tools-that-paginate-show-spurious-less-output: less --version outputs to standard out not to standard error so this changes the redirect from 2> to > (#15720)
fixes-#15685-tools-that-paginate-show-spurious-less-output: Updated redirect to include stderr as well as stdout to not show any errors on screen
9 years ago
jctanner 781de83762 When walking through module directories, always follow symlinks. (#15784)
Fixes #15783
9 years ago
Toshio Kuratomi 409bfe4d0f Strip leading and trailing whitespace for json arg types 9 years ago
Robin Roth 30e5999812 Fix distribution_facts missing on BSD (#15780)
The previous fix in #15773 only solved MacOSX, but left other BSDs broken

fixes #15768
9 years ago
Scott Radvan 72ca3b2b5b add very minor punctuation fixes (#15763) 9 years ago
Pomin Wu 78808fc4cc Fixed `ansible_os_family` variable on OS X (#15768) (#15773)
Fixed `ansible_os_family` variable on OS X
9 years ago
Abhijit Menon-Sen 848b015ce6 Merge pull request #15764 from sradvan/2.0.2-should-be-2016
wrong year in releases file
9 years ago
Scott Radvan 48ef69982b s/2015/2016 9 years ago
nitzmahone fca5ba153e bump extras submodule ref 9 years ago
Toshio Kuratomi 0a5831e654 Update submodule refs 9 years ago
Miguel Cabrerizo a274ef9adc Update intro_dynamic_inventory.rst (#15750)
When using Cobbler with Ansible a cobbler.ini file is needed by the /etc/ansible/cobbler.py script, otherwise Python errors like ConfigParser.NoSectionError: No section: 'cobbler' are shown. Maybe this could be added to the documentation. Thanks and cheers!
9 years ago
Jiri Tyr 589f6d25bb Updating VCA module documentation (#14368) 9 years ago
James Cammarata 09c90f7f2f Fixing bugs in strategies
* Don't filter hosts remaining based on their failed state. Instead rely
  on the PlayIterator to return None/ITERATING_COMPLETE when the host is
  failed.
* In the free strategy, make sure we wait outside the host loop for all
  pending results to be processed.
* Use the internal _set_failed_state() instead of manually setting things
  when a failed child state is hit

Fixes #15623
9 years ago
Lars Kellogg-Stedman 21ac95402f do not erroneously set gathered_facts=True
In `lib/ansible/executor/play_iterator.py`, ansible sets a host's
`_gathered_facts` property to `True` without checking to see if there
are any tasks to be executed.  In the event that the entire play is
skipped, `_gathered_facts` will be `True` even though the `setup`
module was never run.

This patch modifies the logic to only set `_gathered_facts` to `True`
when there are tasks to execute.

Closes #15744.
9 years ago
Toshio Kuratomi 5c7ad654db Add some more tests for item squashing 9 years ago
nitzmahone 7708948d7d bump submodule refs 9 years ago
nitzmahone 133395db30 add jimi-c's unit test for squashed skip results, tweaked is_skipped() logic to pass 9 years ago
Andrew Taumoefolau 85868e07a9 Don't assume a task with non-dict loop results has been skipped.
This changeset addresses the issue reported here:

  ansible/ansible-modules-core#1765

The yum module (at least) includes its task results as strings, rather than
dicts, and the code this changeset replaces assumed that in that instance the
task was skipped. The updated behaviour assumes that the task has been
skipped only if:

* results exist, and
* all results are dicts that include a truthy skipped value
9 years ago
Brian Coca 478674cc57 typo fix 9 years ago
Brian Coca 3a6ca0b4a6 made ansible-doc complain on missing 'requried' 9 years ago
Brian Coca b7c874f81a enforce required 'required' in docs 9 years ago
Brian Coca 2af8e3b9d8 fix default for removing images 9 years ago
Matt Davis 7b58240654 Merge pull request #15712 from mhite/ip_type
New inventory_ip_type option in gce inventory tool
9 years ago
Brian Coca 27a1ae4732 added ability to also subset make tests 9 years ago
Brian Coca fb7940fc50 check that variable first
before using string methods to check for magic interpreter var
9 years ago