From b610a0e126130b4d81d5961f81104313292542e1 Mon Sep 17 00:00:00 2001 From: Douglas Reis Date: Fri, 15 Dec 2023 11:57:07 +0000 Subject: [PATCH 1/2] [sival, ci] Filter out test tagged as broken Signed-off-by: Douglas Reis --- azure-pipelines.yml | 4 ++-- ci/azure-pipelines-nightly.yml | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/azure-pipelines.yml b/azure-pipelines.yml index 10647ee9603943..282766edc4bba4 100644 --- a/azure-pipelines.yml +++ b/azure-pipelines.yml @@ -612,7 +612,7 @@ jobs: set -e . util/build_consts.sh module load "xilinx/vivado/$(VIVADO_VERSION)" - ci/scripts/run-fpga-tests.sh hyper310 cw310_sival || { res=$?; echo "To reproduce failures locally, follow the instructions at https://opentitan.org/book/doc/getting_started/setup_fpga.html#reproducing-fpga-ci-failures-locally"; exit "${res}"; } + ci/scripts/run-fpga-tests.sh hyper310 cw310_sival,-broken || { res=$?; echo "To reproduce failures locally, follow the instructions at https://opentitan.org/book/doc/getting_started/setup_fpga.html#reproducing-fpga-ci-failures-locally"; exit "${res}"; } displayName: Execute tests - template: ci/publish-bazel-test-results.yml @@ -703,7 +703,7 @@ jobs: set -e . util/build_consts.sh module load "xilinx/vivado/$(VIVADO_VERSION)" - ci/scripts/run-fpga-tests.sh cw310 manuf,-cw310_sival || { res=$?; echo "To reproduce failures locally, follow the instructions at https://opentitan.org/book/doc/getting_started/setup_fpga.html#reproducing-fpga-ci-failures-locally"; exit "${res}"; } + ci/scripts/run-fpga-tests.sh cw310 manuf,-cw310_sival,-broken || { res=$?; echo "To reproduce failures locally, follow the instructions at https://opentitan.org/book/doc/getting_started/setup_fpga.html#reproducing-fpga-ci-failures-locally"; exit "${res}"; } displayName: Execute tests - template: ci/publish-bazel-test-results.yml diff --git a/ci/azure-pipelines-nightly.yml b/ci/azure-pipelines-nightly.yml index 7d83b294f7087d..4fab0153391f86 100644 --- a/ci/azure-pipelines-nightly.yml +++ b/ci/azure-pipelines-nightly.yml @@ -154,7 +154,7 @@ jobs: --define bitstream=gcp_splice \ --build_tests_only \ --test_output=errors \ - --test_tag_filters="cw310_sival" \ + --test_tag_filters=cw310_sival,-broken \ $(bash ./bazelisk.sh query 'attr("tags", "[\[ ]pmod[,\]]", tests(//...))') displayName: "Run the PMOD tests for Sival" - template: ../ci/publish-bazel-test-results.yml From b683ab440fe63f3ca786049d48254101a3410678 Mon Sep 17 00:00:00 2001 From: Guillermo Maturana Date: Wed, 3 Jan 2024 22:34:35 -0800 Subject: [PATCH 2/2] [sival,tests] Enable clkmgr_off_otbn_trans_test in silicon The fpga_cw310_sival_rom_ext env is already part of EARLGREY_TEST_ENVS. Signed-off-by: Guillermo Maturana --- sw/device/tests/BUILD | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sw/device/tests/BUILD b/sw/device/tests/BUILD index 3ee4925071abd0..a06d2d10885d32 100644 --- a/sw/device/tests/BUILD +++ b/sw/device/tests/BUILD @@ -806,7 +806,7 @@ opentitan_test( EARLGREY_TEST_ENVS, { "//hw/top_earlgrey:fpga_cw310_sival": None, - "//hw/top_earlgrey:fpga_cw310_sival_rom_ext": None, + "//hw/top_earlgrey:silicon_owner_sival_rom_ext": None, }, ), deps = ["clkmgr_off_trans_impl"],