From 1d75595da4a72c2ca0eb2b9b88db09a05a873e67 Mon Sep 17 00:00:00 2001 From: Andres Gomez Date: Thu, 20 Feb 2020 17:32:08 +0200 Subject: [PATCH] gitlab-ci: Change devices format to In preparation to having "vk" (Vulkan) along "gl" (OpenGL/ES). This is so it is clearer which traces belong to which API and also for the build jobs. Signed-off-by: Andres Gomez Reviewed-by: Samuel Pitoiset Reviewed-by: Alexandros Frantzis --- .gitlab-ci.yml | 2 +- .gitlab-ci/traces.yml | 6 +++--- .gitlab-ci/tracie/README.md | 12 ++++++------ .gitlab-ci/tracie/tests/test.sh | 24 +++++++++++++----------- .gitlab-ci/tracie/tests/traces.yml | 4 ++-- 5 files changed, 25 insertions(+), 23 deletions(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index 6a69e6e853b..e6891997e51 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -736,5 +736,5 @@ llvmpipe-traces: variables: LIBGL_ALWAYS_SOFTWARE: "true" GALLIUM_DRIVER: "llvmpipe" - DEVICE_NAME: "vmware-llvmpipe" + DEVICE_NAME: "gl-vmware-llvmpipe" extends: .traces-test diff --git a/.gitlab-ci/traces.yml b/.gitlab-ci/traces.yml index d390f185c20..cbb668071a0 100644 --- a/.gitlab-ci/traces.yml +++ b/.gitlab-ci/traces.yml @@ -5,13 +5,13 @@ traces-db: traces: - path: glmark2/desktop-blur-radius=5:effect=blur:passes=1:separable=true:windows=4.rdc expectations: - - device: vmware-llvmpipe + - device: gl-vmware-llvmpipe checksum: 8867f3a41f180626d0d4b7661ff5c0f4 - path: glmark2/jellyfish.rdc expectations: - - device: vmware-llvmpipe + - device: gl-vmware-llvmpipe checksum: e0fe979fee129c0ed42a3059d1a4e1c9 - path: glxgears/glxgears.trace expectations: - - device: vmware-llvmpipe + - device: gl-vmware-llvmpipe checksum: 02aca9b4b4ad6fd60331df6e4f87f2cd diff --git a/.gitlab-ci/tracie/README.md b/.gitlab-ci/tracie/README.md index 5c1f8a881ac..8ea51c6d598 100644 --- a/.gitlab-ci/tracie/README.md +++ b/.gitlab-ci/tracie/README.md @@ -17,13 +17,13 @@ traces-db: traces: - path: glmark2/jellyfish.rdc expectations: - - device: intel-0x3185 + - device: gl-intel-0x3185 checksum: 58359ea4caf6ad44c6b65526881bbd17 - - device: vmware-llvmpipe + - device: gl-vmware-llvmpipe checksum: d82267c25a0decdad7b563c56bb81106 - path: supertuxkart/supertuxkart-antediluvian-abyss.rdc expectations: - - device: intel-0x3185 + - device: gl-intel-0x3185 checksum: ff827f7eb069afd87cc305a422cba939 ``` @@ -56,9 +56,9 @@ To enable trace testing on a new device: ```yaml my-hardware-traces: + extends: .traces-test-gl variables: DEVICE_NAME: "myhardware" - extends: .traces-test ``` 2. Update the .gitlab-ci/traces.yml file with expectations for the new device. @@ -110,8 +110,8 @@ appended. Examples: - python3 dump_traces_images.py --device-name=vmware-llvmpipe mytrace.trace - python3 dump_traces_images.py --device-name=vmware-llvmpipe --calls=2075,3300 mytrace.trace + python3 dump_traces_images.py --device-name=gl-vmware-llvmpipe mytrace.trace + python3 dump_traces_images.py --device-name=gl-vmware-llvmpipe --calls=2075,3300 mytrace.trace ### Running the replay scripts locally diff --git a/.gitlab-ci/tracie/tests/test.sh b/.gitlab-ci/tracie/tests/test.sh index 3ef2144fd84..4d52578c47e 100755 --- a/.gitlab-ci/tracie/tests/test.sh +++ b/.gitlab-ci/tracie/tests/test.sh @@ -31,8 +31,10 @@ assert() { } run_tracie() { - # Run tests for the .testtrace types, using the "test-device" device name. - DEVICE_NAME=test-device CI_PROJECT_DIR="$TEST_DIR" \ + # Run tests for the .testtrace types, using the "gl-test-device" and "vk-test-device" device names. + DEVICE_NAME=gl-test-device CI_PROJECT_DIR="$TEST_DIR" \ + "$TEST_DIR/tracie.sh" "$TEST_DIR/tests/traces.yml" testtrace && \ + DEVICE_NAME=vk-test-device CI_PROJECT_DIR="$TEST_DIR" \ "$TEST_DIR/tracie.sh" "$TEST_DIR/tests/traces.yml" testtrace } @@ -102,7 +104,7 @@ tracie_ignores_unspecified_trace_types() { echo " - path: trace1/empty.trace" >> "$TEST_DIR/tests/traces.yml" echo " expectations:" >> "$TEST_DIR/tests/traces.yml" - echo " - device: test-device" >> "$TEST_DIR/tests/traces.yml" + echo " - device: gl-test-device" >> "$TEST_DIR/tests/traces.yml" echo " checksum: 000000000000000" >> "$TEST_DIR/tests/traces.yml" # For the tests we only scan for the .testtrace type, # so the .trace file added below should be ignored. @@ -160,11 +162,11 @@ tracie_stores_only_logs_on_checksum_match() { run_tracie assert "[ $? = 0 ]" - assert "[ -f "$TEST_DIR/results/trace1/test/test-device/magenta.testtrace.log" ]" - assert "[ -f "$TEST_DIR/results/trace2/test/test-device/olive.testtrace.log" ]" + assert "[ -f "$TEST_DIR/results/trace1/test/gl-test-device/magenta.testtrace.log" ]" + assert "[ -f "$TEST_DIR/results/trace2/test/vk-test-device/olive.testtrace.log" ]" - assert "[ ! -f "$TEST_DIR/results/trace1/test/test-device/magenta.testtrace-0.png" ]" - assert "[ ! -f "$TEST_DIR/results/trace2/test/test-device/olive.testtrace-0.png" ]" + assert "[ ! -f "$TEST_DIR/results/trace1/test/gl-test-device/magenta.testtrace-0.png" ]" + assert "[ ! -f "$TEST_DIR/results/trace2/test/vk-test-device/olive.testtrace-0.png" ]" ls -lR "$TEST_DIR" @@ -181,8 +183,8 @@ tracie_stores_images_on_checksum_mismatch() { run_tracie assert "[ $? != 0 ]" - assert "[ ! -f "$TEST_DIR/results/trace1/test/test-device/magenta.testtrace-0.png" ]" - assert "[ -f "$TEST_DIR/results/trace2/test/test-device/olive.testtrace-0.png" ]" + assert "[ ! -f "$TEST_DIR/results/trace1/test/gl-test-device/magenta.testtrace-0.png" ]" + assert "[ -f "$TEST_DIR/results/trace2/test/vk-test-device/olive.testtrace-0.png" ]" destroy_repo "$repo" } @@ -194,8 +196,8 @@ tracie_stores_images_on_request() { (export TRACIE_STORE_IMAGES=1; run_tracie) assert "[ $? = 0 ]" - assert "[ -f "$TEST_DIR/results/trace1/test/test-device/magenta.testtrace-0.png" ]" - assert "[ -f "$TEST_DIR/results/trace2/test/test-device/olive.testtrace-0.png" ]" + assert "[ -f "$TEST_DIR/results/trace1/test/gl-test-device/magenta.testtrace-0.png" ]" + assert "[ -f "$TEST_DIR/results/trace2/test/vk-test-device/olive.testtrace-0.png" ]" ls -lR "$TEST_DIR" diff --git a/.gitlab-ci/tracie/tests/traces.yml b/.gitlab-ci/tracie/tests/traces.yml index dc6dd04b526..64e3e4fce2b 100644 --- a/.gitlab-ci/tracie/tests/traces.yml +++ b/.gitlab-ci/tracie/tests/traces.yml @@ -1,9 +1,9 @@ traces: - path: trace1/magenta.testtrace expectations: - - device: test-device + - device: gl-test-device checksum: 8e0a801367e1714463475a824dab363b - path: trace2/olive.testtrace expectations: - - device: test-device + - device: vk-test-device checksum: 5efda83854befe0155ff8517a58d5b51 -- 2.30.2