21 Commits (37052a983a7f37e5f7e8c03e1ed75fe89d4a58bd)

Author	SHA1	Message	Date
Toshio Kuratomi	2e87c1f74e	Two fixes to action plugins ... * Fix the task_vars parameter to not default to a mutable type (dict) * Implement invocation in the base class's run() method have each action module call the run() method's implemention in the base class. * Return values from the action plugins' run() method takes the return value from the base class run() method into account so that invocation makes its way to the output. Fixes #12869	9 years ago
Toshio Kuratomi	a1428d6bed	Remove tmp as a parameter to the connection plugins ... There doesn't appear to be anything that actually uses tmp_path in the connection plugins so we don't need to pass that in to exec_command. That change also means that we don't need to pass tmp_path around in many places in the action plugins any more. there may be more cleanup that can be done there as well (the action plugin's public run() method takes tmp as a keyword arg but that may not be necessary). As a sideeffect of this patch, some potential problems with chmod and the patch, assemble, copy, and template modules has been fixed (those modules called _remote_chmod() with the wrong order for their parameters. Removing the tmp parameter fixed them.)	9 years ago
Toshio Kuratomi	03127dcfae	remove the stdin return value from connection plugin exec_command() methods ... The value was useless -- unused by the callers and always hardcoded to the empty string.	9 years ago
Florian Apolloner	d9f873495e	Ported over #7158 to support SELinux context switches.	9 years ago
Florian Apolloner	8548690ca3	Added a test to ensure that sudo is only used if remote and become user difer.	9 years ago
Brian Coca	b6d6c2e4db	corrected all missing paths changes	9 years ago
Abhijit Menon-Sen	065bb52109	Be systematic about parsing and validating hostnames and addresses ... This adds a parse_address(pattern) utility function that returns (host,port), and uses it wherever where we accept IPv4 and IPv6 addresses and hostnames (or host patterns): the inventory parser the the add_host action plugin. It also introduces a more extensive set of unit tests that supersedes the old add_host unit tests (which didn't actually test add_host, but only the parsing function).	9 years ago
James Cammarata	444987cde6	Fix unit test for plugins after basedir/push_basedir removal	9 years ago
Toshio Kuratomi	e2c49b4ef4	Fix problem with "=" in the initial file path.	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
Ryan Petrello	1886307845	Fix a parsing bug that prevents IPv6 addresses from being used with `add_host` ... Closes #8682	9 years ago
James Cammarata	8146a193eb	Fix strategy base unit tests after 88e1ef8	9 years ago
James Cammarata	e64989beb4	Moving ConnectionInformation -> PlayContext ... Also making PlayContext a child class of the Playbook Base class, which gives it access to all of the FieldAttribute code to ensure field values are correctly typed after post_validation Fixes #11381	9 years ago
James Cammarata	fadd8f23f1	Fix broken strategy unit test from earlier change	9 years ago
James Cammarata	3b913943b2	Updating base strategy unit test regarding bad file loads based on earlier change	9 years ago
James Cammarata	bfbb88b4a9	Fix strategy plugin unit tests related to earlier changes	9 years ago
Matt Martz	49d19e82ab	Get tests passing ... The largest failure in the tests was due to selinux not being installed. The tests don't require it to be installed, so mock the import.	10 years ago
James Cammarata	0828028c71	Fixing unit test for included file changes	10 years ago
James Cammarata	9921a1d2be	Unit tests for base strategy class (v2)	10 years ago
James Cammarata	21fa385ce7	Reorganizing plugin unit tests and adding start of strategy tests (v2)	10 years ago
James Cammarata	ce3ef7f4c1	Making the switch to v2	10 years ago