From 2ea5f97bc5db278dd17400e86d4cb5c2c3a657a4 Mon Sep 17 00:00:00 2001 From: Tomas Tomecek Date: Feb 19 2018 13:34:20 +0000 Subject: [PATCH 1/3] no need to require root owner for artifacts dir Signed-off-by: Tomas Tomecek --- diff --git a/roles/str-common/tasks/main.yml b/roles/str-common/tasks/main.yml index f259710..ad3d9c5 100644 --- a/roles/str-common/tasks/main.yml +++ b/roles/str-common/tasks/main.yml @@ -16,7 +16,7 @@ - import_tasks: pkgs.yml - name: Make artifacts directory - file: path={{ remote_artifacts }} state=directory owner=root mode=755 recurse=yes + file: path={{ remote_artifacts }} state=directory mode=755 recurse=yes # Next task requires rsync on test environment - name: Copy tests to test environment From f97ed5bf1dfb86fd7e9a755cffdaf08cbe640abb Mon Sep 17 00:00:00 2001 From: Tomas Tomecek Date: Feb 28 2018 10:43:00 +0000 Subject: [PATCH 2/3] install packages as root Signed-off-by: Tomas Tomecek --- diff --git a/roles/str-common/tasks/pkgs.yml b/roles/str-common/tasks/pkgs.yml index 6e75ae9..2585456 100644 --- a/roles/str-common/tasks/pkgs.yml +++ b/roles/str-common/tasks/pkgs.yml @@ -27,3 +27,6 @@ verbosity: 1 - include_tasks: "pkgs-{{ pkg_mgr | trim}}.yml" + # We add become: true so that users are able to run this role locally using + # their unprivileged user. + become: true From c7634ab416a2b087bf1106d68f1884a8e37fde17 Mon Sep 17 00:00:00 2001 From: Tomas Tomecek Date: Feb 28 2018 10:43:00 +0000 Subject: [PATCH 3/3] alywas copy logs from the test env Signed-off-by: Tomas Tomecek --- diff --git a/roles/standard-test-basic/tasks/main.yml b/roles/standard-test-basic/tasks/main.yml index 049b638..3f07fc1 100644 --- a/roles/standard-test-basic/tasks/main.yml +++ b/roles/standard-test-basic/tasks/main.yml @@ -35,7 +35,7 @@ shell: grep "^FAIL" {{ remote_artifacts }}/test.log register: test_fails failed_when: test_fails.stdout or test_fails.stderr - + always: - include_role: name: str-common tasks_from: end.yml