Commit Graph

16312 Commits (696cf32d635f5aa85170da16697c3bf2c6f3e86a)
 

Author SHA1 Message Date
James Cammarata c16961db60 Fix two PlayIterator bugs
* When iterating over a child state, a failure should be propagated
  up so parent blocks don't continue iterating
* Make sure a child state exists before trying to search it

Fixes #12210
9 years ago
Praveen Kumar 6e1f7cd503 Fix typo to avoid traceback during testing 9 years ago
Toshio Kuratomi 7f0cd14e0f Merge pull request #12230 from mgedmin/py3k
Python 3: avoid iteritems() in a template
9 years ago
Marius Gedminas 38a96d7e8f Python 3: avoid iteritems() in a template
I don't think six.iteritems is available here, but I also don't expect
there to be enough platforms to ever make the speed difference between
.items() and .iteritems() noticeable.
9 years ago
Toshio Kuratomi 364313c01f Simplify and speed up _split_role_params() by moving an invariant outside of a loop. 9 years ago
Toshio Kuratomi 1ae28f606b Refactor to replace filter and lambda with a generator expression 9 years ago
Toshio Kuratomi dac562a132 Add note about the trailing newlines change 9 years ago
Toshio Kuratomi 9e023fedf2 Merge pull request #12228 from mgedmin/py3k
Simplify FactCache.copy()
9 years ago
Marius Gedminas a32bf1ec18 Simplify FactCache.copy()
Also fix the bug (missing from six import iteritems) I introduced in
823677b490.
9 years ago
Toshio Kuratomi ce97874523 Merge pull request #12226 from mgedmin/py3k
Replace .iteritems() with six.iteritems()
9 years ago
Arbab Nazar d2e547c31f Update developing_modules.rst 9 years ago
Marius Gedminas 823677b490 Replace .iteritems() with six.iteritems()
Replace .iteritems() with six.iteritems() everywhere except in
module_utils (because there's no 'six' on the remote host).  And except
in lib/ansible/galaxy/data/metadata_template.j2, because I'm not sure
six is available there.
9 years ago
James Cammarata 9e1bc434c7 Further cleanup from locking changes 9 years ago
James Cammarata b93f27e260 Move the lockfile back to tqm to make sure it stays unique 9 years ago
James Cammarata ba658ff3a9 Merge branch 'amenonsen-connection-locking' into devel 9 years ago
James Cammarata b9afbf0ee4 Reorganizing the way the connection lockfile is created 9 years ago
James Cammarata 7034bbef30 Merge branch 'connection-locking' of https://github.com/amenonsen/ansible into amenonsen-connection-locking 9 years ago
James Cammarata 3e5b90c6fb Merge pull request #12195 from amenonsen/ssh-lesswork
Make lock_host_keys a real noop, instead of half-a-noop
9 years ago
Abhijit Menon-Sen 9378c8e2da Make the paramiko plugin use locking 9 years ago
Abhijit Menon-Sen 5887e96b27 Introduce a connection locking infrastructure
The lock file is (a temporary file) opened in the parent process, whose
open fd is inherited by the workers after fork, and passed down through
the PlayContext. Connection grows lock/unlock methods which can be used
by individual connection plugins.
9 years ago
Abhijit Menon-Sen 12d8f9132e Make lock_host_keys a real noop, instead of half-a-noop
Right now, we don't do any locking, but we still scan known_hosts files
twice per connection. That's completely unnecessary, and the proposed
solutions to the locking problem wouldn't need known_hosts scanning
anyway, so this code can go away.
9 years ago
Brian Coca 88d3751c28 Merge pull request #12217 from thyming/hacking-remove-pyc
Delete all compiled python files when running hacking/env-setup
9 years ago
Luke Rohde cba4642d8d Use find ... -delete instead of non-portable globbing 9 years ago
Brian Coca 099d1797f2 added new path config setting handling 9 years ago
Brian Coca a006e85947 removed printing module args as it violates no_log settings 9 years ago
Toshio Kuratomi 1b64702b9a Fix templar unittests for change in preserve_trailing_newline parameter's default value 9 years ago
Toshio Kuratomi 1998eddf8d Keep newlines by default now.
In v1, a trailing newline was kept if the parameter was passed as key=value.  If
the parameter was passed as yaml dict the trailing newline was
discarded.  Since key-value and yaml dict were unified in v2 we have to
make a choice as to which behaviour we want.  Decided that keeping trailing
newlines by default made the most sense.

Fixes #12200
Fixes #12199
9 years ago
Luke Rohde 647e48f776 Delete all compiled python files when running hacking/env-setup 9 years ago
James Cammarata 45aac6a739 Adding unit tests for safe_eval 9 years ago
Toshio Kuratomi d0d40dff9e Enable the var precedence test in travis 9 years ago
Toshio Kuratomi 41da8de094 Speedup for counting newlines 9 years ago
James Cammarata 843138e7d9 Merge pull request #12202 from mgedmin/py3k
Use rich comparisons for Attribute and FieldAttribute
9 years ago
James Cammarata 5434ed973b Make sure locals in safe_eval is a plain dict when running eval
Fixes #12206
9 years ago
Toshio Kuratomi 9ecfc30f9a * Mark all strings as unicode -- shouldn't matter for this dataset but
ansible is passing unicode arond internally so we should test the same
  data.
* Add a zero length test for _count_newlines and fix the zero newlines
  test to have no newlines.
9 years ago
Toshio Kuratomi 417bf1c805 Unittest the _count_trailing_newlines function 9 years ago
Toshio Kuratomi 7ed746ad45 Fix preserve_trailing_newlines (broken by 7f5080f64a )
Fix for one half of hte bug reported in #12198
9 years ago
Brian Coca a7231c2203 actually implemented flags correctly for all priv escalation methods 9 years ago
Brian Coca 14f061d5ea fixed become tests 9 years ago
Brian Coca dfb94f0def added ec2_vpc_route_table to changelog 9 years ago
Brian Coca b9642585bd added missing become_exe from config 9 years ago
Brian Coca e156d9b677 fixed and generalized privilege escalation exe settings 9 years ago
Marius Gedminas 33a374edde Use rich comparisons for Attribute and FieldAttribute
Because __cmp__ is not supported on Python 3.
9 years ago
Marius Gedminas ca908f2e12 Fix indentation to be a multiple of 4 9 years ago
James Cammarata e8d7fafc52 Merge pull request #12194 from amenonsen/dupe-fix
Remove duplicated function definition (fallout from earlier repeated merge)
9 years ago
Abhijit Menon-Sen 147b04b682 Remove duplicated function definition (fallout from earlier repeated merge) 9 years ago
James Cammarata f162990cb3 Properly assign the searchpath for templates to the environment loader
dbd755e0 previously assigned the value to self._templar.environment.searchpath,
which is incorrect - it needs to be assigned to the environment.loader.searchpath
value instead.

Fixes #11931
9 years ago
James Cammarata 2761df232e Small tweak to 7551b75 to correct the ordering of tests 9 years ago
James Cammarata 7551b75e61 Add ispath type for constants and make sure all local paths are ispath=True
Fixes #12180
9 years ago
Toshio Kuratomi aeff960d02 Cleanup combine_vars
* Dedupe combine_vars() code (removed from VariableManager)
* Fix merge_hash algorithm to preserve the type
* unittest combine_vars and merge_hash
9 years ago
James Cammarata 7fe495d619 Merge pull request #12187 from caphrim007/add-utf-8-encoding-to-f5-module-utils
Adds utf-8 encoding to f5 module_utils
9 years ago