ansible

Commit Graph

Author	SHA1	Message	Date
Brian Coca	ac9278ff0f	remove action plugin only fields from 'file' calls (#31047 ) * remove action plugin only fields from 'file' calls fixes #30556 * Add a test for #30556	7 years ago
Toshio Kuratomi	bc66faa328	Add more tests for copy/file/template with harlinks	7 years ago
Mehran Kholdi	ecfa7f696d	Enforce `state='file'` in `copy` module This was causing wrong behaviour when `prev_state` was `hard`-link, since the `file` module tried to apply the same `state` on the new file, causing unexpected errors. Particularly, both `overlay` and `devicemapper` storage drivers in docker use hardlinks to share files between layers. This causes most ansible playbooks to fail when working with files from layers below.	7 years ago
Sam Doran	a8e4c9be7a	Cause copy module to fail on empty string as source (#27975 ) * Fail if an empty string is set as src for copy module Fixes #27363 * Cleanup task formatting on copy tests Use multi-line YAML Add debug statements with verbosity: 1 rather than leave them in there commented out. * Add test for empty string as source * Do more checks in order to add more specific errors messages Add more integration tests for the various failure scenarios. Cleanup some syntax on existing integration test tasks.	7 years ago
Pierre-Louis Bonicoli	022d45cb52	copy: clearer parameter values (tests)	7 years ago
Pierre-Louis Bonicoli	efd17915f1	copy: clearer comments, filenames, task names (tests)	7 years ago
Pierre-Louis Bonicoli	6f47b3e9e8	copy: check non-existing one level directory too	7 years ago
Pierre-Louis Bonicoli	470989bff9	copy: check behavior related to dest creation when src is a file	7 years ago
Pilou	ade593da52	Copy module: improve tests allowing to use a managed host which isn't the controller host (#25672 ) * set output_dir_expanded using module result 'path' values are expanded using 'expandvars' too * foo.txt is located in 'files' directory * Use 'role_path' and 'connection: local' for local paths '{{ role_path }}/tmp' is used for generated paths * Use local connection with local paths /tmp/ansible-test-abs-link and /tmp/ansible-test-abs-link-dir are defined by targets/copy/files/subdir/subdir1/ansible-test-abs-link and targets/copy/files/subdir/subdir1/ansible-test-abs-link-dir links. * task names: add a suffix when same name is reused * Check that item exists before checking file mode then error message is more explicit when item doesn't exist * Use output_dir_expanded only when necessary * Enforce remote_user when root is required * Fix remote path * Use different local & remote user this is useful when controller and managed hosts are identical * Checks must not expect output of tested module to be right * Use a temporary directory on the controller * Use sha1 & md5 filters instead of hardcoded values * Use 'remote_dir' for directory on managed host * Workaround tempfile error on OS X Error was: temp_path = tempfile.mkdtemp(prefix='ansible_') AttributeError: 'module' object has no attribute 'mkdtemp'"	7 years ago
Toshio Kuratomi	0a2cdb2585	New tests for copy recursive with absolute paths Absolute path trailing slash handling in absolute directories find_needle() isn't passing a trailing slash through verbatim. Since copy uses that to determine if it should copy a directory or just the files inside of it, we have to detect that and restore it after calling find_needle() Fixes #27439	7 years ago
Toshio Kuratomi	f86ce0975d	Add a directory walker to copy * We need a directory walker that can handle symlinks, empty directories, and some other odd needs. This commit contains a directory walker that can do all that. The walker returns information about the files in the directories that we can then use to implement different strategies for copying the files to the remote machines. * Add local_follow parameter to copy that follows local symlinks (follow is for remote symlinks) * Refactor the copying of files out of run into its own method * Add new integration tests for copy Fixes #24949 Fixes #21513	7 years ago
Toshio Kuratomi	753a3a03d0	Revert "Fix for recursive copy slowness" This reverts commit `78ced5318f`. The fix for copy slowness did not handle circular symlinks.	7 years ago
Brian Coca	4594bee65a	keep unsafe .. unsafe (#23742 ) * keep unsafe .. unsafe fixes #23734, which was broken in previous fix that allowed non string types to be templated use new 'is_template' function vs bastardizing others refactored clean_data to allow for arbitrary data structures to clean fixed/removed some tests * deal with complex data for is_template * typos	8 years ago
Toshio Kuratomi	78ced5318f	Fix for recursive copy slowness Copy module was walking over files in subdirectories repeatedly (a directory tree a few levels deep could bring the time spent into the tens of minutes) This was traced to the fix for this bug report: https://github.com/ansible/ansible/issues/13013 Fixed #13013 a different way and added an integration test to check for regressions of #13013 as we optimize this code. Fixes #21513	8 years ago
Matt Clay	6bbd92e422	Initial ansible-test implementation. (#18556 )	8 years ago
Adrien Vergé	0e834fc9e4	Fix cosmetic problems in YAML source This change corrects problems reported by the `yamllint` linter. Since key duplication problems were removed in `4d48711`, this commit mainly fixes trailing spaces and extra empty lines at beginning/end of files.	8 years ago
Matt Clay	75e4645ee7	Migrate Linux CI roles to test targets. (#17997 )	8 years ago

17 Commits (ded023bb3b54a157a55cfc0d865cc0af5a0b4afa)