Merge pull request #10269 from danrue/unarchive_creates_behavior

Make behavior consistent when using 'creates' option with modules

lib/ansible/runner/action_plugins/script.py

@@ -66,7 +66,7 @@ class ActionModule(object):
                     conn=conn,
                     comm_ok=True,
                     result=dict(
-                        skipped=True,
+                        changed=False,
                         msg=("skipped, since %s exists" % creates)
                     )
                 )
@@ -83,7 +83,7 @@ class ActionModule(object):
                     conn=conn,
                     comm_ok=True,
                     result=dict(
-                        skipped=True,
+                        changed=False,
                         msg=("skipped, since %s does not exist" % removes)
                     )
                 )

lib/ansible/runner/action_plugins/unarchive.py

@@ -69,7 +69,6 @@ class ActionModule(object):
                     conn=conn,
                     comm_ok=True,
                     result=dict(
-                        skipped=True,
                         changed=False,
                         msg=("skipped, since %s exists" % creates)
                     )

test/integration/roles/test_unarchive/tasks/main.yml

@@ -96,7 +96,6 @@
   assert:
     that:
       - "unarchive02c.changed == false"
-      - "unarchive02c.skipped == true"
 
 - name: unarchive a tar.gz file with creates over an existing file using complex_args
   unarchive:
@@ -110,7 +109,6 @@
   assert:
     that:
       - "unarchive02d.changed == false"
-      - "unarchive02d.skipped == true"
 
 - name: remove our tar.gz unarchive destination
   file: path={{output_dir}}/test-unarchive-tar-gz state=absent