Commit Graph

19592 Commits (b5daf2286cfc4239a06c0a1b9b4ca5ea3888e4f3)
 

Author SHA1 Message Date
James Cammarata 29f2f26278 New release v2.1.2.0-1 8 years ago
James Cammarata 5f23d09e7c Removing mount from CHANGELOG due to reversion 8 years ago
Toshio Kuratomi b989004a0b Pick up core reversion of mount changes 8 years ago
James Cammarata fd256a79a1 Updating CHANGELOG for 2.1 8 years ago
James Cammarata c3257027b9 New release v2.1.2.0-0.5.rc5 8 years ago
Toshio Kuratomi 4c845b4ef0 Update submodule ref for fix to mount backport 8 years ago
James Cammarata 3d8dac2084 New release v2.1.2.0-0.4.rc4 8 years ago
Toshio Kuratomi 2b7761cd37 Update core to pick up more mount changes 8 years ago
Toshio Kuratomi 60a6c83abc Add ini_file whitespace change to the CHANGELOG 8 years ago
Toshio Kuratomi ee4ba3ceca Update submodule refs 8 years ago
Toshio Kuratomi 66ffe199e5 Disable mount tests for now. Mount is buggy on too many platforms 8 years ago
Toshio Kuratomi 5635670191 Update submodule refs for mount fixes 8 years ago
Toshio Kuratomi a4611ff509 Add tests for the mount module (#17718)
* Add tests for the mount module

* Switch from unmounted to absent...

the code for mounting always modifies fstab so we need to always modify
fstab to cleanup as well.

* Fix comments and copyright
8 years ago
Abhijit Menon-Sen cc71765d9e Use loop_control.loop_var directly
6eefc11c converted task.loop_control into an object, but while the other
callers were updated to use .loop_var instead of .get('loop_var'), this
site was overlooked.

This can be reproduced by including with loop_control a file that does
set_fact; a simple regression test along these lines is included.

(cherry picked from commit 950cc26aab)
8 years ago
Peter Sprygada b5b8a756fc fixes issue with net_template failing unless src argument provided. (#17726)
This fixes an issue where the net_template action will fail if a non
required argument (src) is not provided.

fixes ansible/ansible-modules-core#4978
8 years ago
nitzmahone 4c9ed1f4fb bump extras submodule ref for win_chocolatey fix 8 years ago
Toshio Kuratomi e917a0bd7a Fix for cherry-pick e8dddc3679,
The cherry-pick allows module parameters to convert int to float, fixing
bugs where a float type is required.  However, it used types from the
six library for python3 compatibility as well.  Since we don't have
six in 2.1.0 this was buggy.  Merging the unittests for the problem
here: adc158a499 made this buggy behaviour
apparent.
8 years ago
Toshio Kuratomi 9de21e1eaa Update submodule refs 8 years ago
Toshio Kuratomi eaba2152f3 Remove _load_hosts() from Play initialization as it's no longer needed and it breaks using extra_vars defining a list for hosts (#17699)
Thanks to @jimi-c for the solution

Fixes #16583
8 years ago
Christoph adc158a499 Add a test for int/float parameter type checking (#16741)
A parameter of type int should accept int and string, but not float.
A parameter of type float should accept float, int, and string.

Also reset the arguments in another test so that it runs cleanly.  This
agrees with what all the other tests are doing.
8 years ago
James Cammarata 01439aafaf Create a raw lookup for hostvars that does not template the data
When using hostvars to get extra connection-specific vars for connection
plugins, use this raw lookup to avoid prematurely templating all of the
hostvar data (triggering unnecessary lookups).

Fixes #17024

(cherry picked from commit ac5ddf4aa092e12f9e1c85c6b74aa30b7ef0a382)
8 years ago
Matt Clay b7168d2ac8 Remove Travis config/badge and add Shippable. (#17680) 8 years ago
James Cammarata c004ae578d Take ITERATING_ALWAYS into account when setting failed state in PlayIterator
Fixes #15963

(cherry picked from commit 4dc2bf4815)
8 years ago
James Cammarata efe5bb122e Rework the way params are assigned to TaskIncludes when they're dynamic
Copying the TaskInclude task (which is the parent) before loading the blocks
makes the code much more simple and clean, and fixes a bug introduced during
the performance improvement changes (and specifically the change which moved
things to a single-parent model).

Fixes #17064

(cherry picked from commit f4237b2151)
8 years ago
René Moser 27c621950c cloudstack: fix has_changed dict values comparsion (#17632)
In some rare situations, the CloudStack API returns string for numbers
when we expected int.

With this fix, we ensure we compare the types expected.
8 years ago
Andrea Tartaglia 7bfa36eb8b Pass the absolute path to dirname when assigning basedir (#17457)
* Pass the absolute path to dirname when assigning basedir

If no path is specified when calling the playbook, os.path.dirname(playbook_path) returns ''
This will cause failure when creating the retry file.

Fixes #17456

* Updated to use os.pathdirname(os.path.abspath())
8 years ago
James Cammarata fb71a9dfd9 New release v2.1.2.0-0.3.rc3 8 years ago
Jay d86b2ec225 Fix issue with proxy setting leaving temp files (#17073)
When using the no_proxy option, the temp ca files that are created for
setting up an ssl context were not being deleted. 

This small patch fixes that issue by removing the temp file.
8 years ago
Ryan S. Brown 7ea342d8ec bump core submodule ref 8 years ago
Toshio Kuratomi 6ba8926889 Update submodule refs 8 years ago
nitzmahone 2e06f0b427 add win_user regression tests
Ensure https://github.com/ansible/ansible-modules-core/issues/4369 doesn't occur again, also adds tests for password set when expired.

(cherry picked from commit 69880f1640)
8 years ago
nitzmahone a900a0b4e3 bump core submodule ref 8 years ago
Monty Taylor c536ca399a Fix galaxy import command for 2.1 (#17417)
The g_connect decorator is required on create_import_task or else
self.baseurl is None. This is fixed in devel already.
8 years ago
Matt Clay 2587d2aaf9 Add partially backwards compatible version of _fixup_perms. (#17427)
Also added a deprecation notice for _fixup_perms.

Resolves issue #17352 (assumes custom actions use recursive=False).

(cherry picked from commit 94a0d2afb4)
8 years ago
James Cammarata e83840c3fd New release v2.1.2.0-0.2.rc2 8 years ago
Toshio Kuratomi e8dddc3679 For module parameter checking, allow conversion of int to float (#17325)
Fixes https://github.com/ansible/ansible-modules-core/issues/4084
8 years ago
James Cammarata da3fd2d588 Several fixes for includes
* when including statically, make sure that all parents were also included
  statically (issue #16990)
* properly resolve nested static include paths
* print a message when a file is statically included

Fixes #16990

(cherry picked from commit 1c7e0c73c9)
8 years ago
Will Thames c9b212c5bd task_result _check_key should handle empty results (#16766)
When a task result has an empty results list, the
list should be ignored when determining the results
of `_check_key`. Here the empty list is treated the
same as a non-existent list.

This fixes a bug that manifests itself with squashed
items - namely the task result contains the correct
value for the key, but an empty results list. The
empty results list was treated as zero failures
when deciding which handler to call - so the task
show as a success in the output, but is deemed to
have failed when deciding whether to continue.

This also demonstrates a mismatch between task
result processing and play iteration.

A test is also added for this case, but it would not
have caught the bug - because the bug is really in
the display, and not the success/failure of the
task (visually the test is more accurate).

Fixes ansible/ansible-modules-core#4214
(cherry picked from commit eb2a3a91a8)
8 years ago
Michael Scherer 478283f571 Do not convert Nonetype to "None" (#17261)
If someone use a task with a empty name like this:

  - name:
    command: true

This will result in displaying 'None' as a task name instead of
'command'.
8 years ago
Remi Ferrand 5035b8a8bf Properly template task names in free strategy
* Fixes #16295
8 years ago
Toshio Kuratomi 559fcbe531 Add fetch fix to changelog 8 years ago
Toshio Kuratomi 0cce86cac9 Fix fetch idempotence (#17255)
Fetch always follows symlinks when downloading so it needs to always
follow symlinks when getting the checksum of the file as well.
8 years ago
James Cammarata f80c981ef6 Clean up PlaybookExecutor logic for batches and errors
The calculation for max_fail_percentage was moved into the linear
strategy a while back, and works better there in the stategy layer
rather than at the PBE layer. This patch removes it from the PBE layer
and tweaks the logic controlling whether or not the next batch is run.

Fixes #15954

(cherry picked from commit 890e096b2b)
8 years ago
Ansible Test Runner fc3efdb057 Changelog entry for https://github.com/ansible/ansible/pull/17225 8 years ago
Toshio Kuratomi 7d6df50e0e The former match for a section header included hosts that began with a range. (#17225)
Checking that the line ends with "]" narrows that window somewhat,

Fixes #15331
8 years ago
James Cammarata b2fe1b39df New release v2.1.2.0-0.1.rc1 8 years ago
Toshio Kuratomi 719c73afa2 Fix tmpfile misspelled as tmplfile (#17183) 8 years ago
James Cammarata bf929ac532 Backport of f722d41 to the stable-2.1 branch
Related to #15915
8 years ago
nitzmahone 9478e41394 bump extras submodule ref for another win_chocolatey fix 8 years ago
nitzmahone 29dd1635fe bump submodule refs 8 years ago