From f9e8585c5354748b2a77ed65d214adfe7c6ae5fe Mon Sep 17 00:00:00 2001 From: Chris Evich Date: Thu, 30 Nov 2023 10:13:35 -0500 Subject: [PATCH] Enable win podman-machine test failure Intended to serve as motivation to fix them. Removed from status aggregator so the failures don't block PR merging. Updated comment text to reference related open issue, #20548. Signed-off-by: Chris Evich --- .cirrus.yml | 7 +++---- contrib/cirrus/cirrus_yaml_test.py | 2 +- 2 files changed, 4 insertions(+), 5 deletions(-) diff --git a/.cirrus.yml b/.cirrus.yml index ae58bc18f5..ce84526bee 100644 --- a/.cirrus.yml +++ b/.cirrus.yml @@ -764,9 +764,6 @@ podman_machine_aarch64_task: podman_machine_windows_task: name: *std_name_fmt alias: podman_machine_windows - # TODO: These tests are new and mostly fail. Disable all failures impacting overall CI status - # until the tests, scripts, and environment stabalize. - allow_failures: $CI == $CI # Only run for non-docs/copr PRs and non-release branch builds # and never for tags. Docs: ./contrib/cirrus/CIModes.md only_if: >- @@ -1061,7 +1058,9 @@ success_task: - rootless_integration_test - podman_machine - podman_machine_aarch64 - - podman_machine_windows + # TODO: issue #20548; These tests are new and mostly fail. + # Ignore status until tests, scripts, and/or environment stabalize. + # - podman_machine_windows - local_system_test - local_system_test_aarch64 - remote_system_test diff --git a/contrib/cirrus/cirrus_yaml_test.py b/contrib/cirrus/cirrus_yaml_test.py index 262ea4d2eb..196b745a2d 100755 --- a/contrib/cirrus/cirrus_yaml_test.py +++ b/contrib/cirrus/cirrus_yaml_test.py @@ -27,7 +27,7 @@ class TestDependsOn(TestCaseBase): ALL_TASK_NAMES = None SUCCESS_DEPS_EXCLUDE = set(['success', 'bench_stuff', 'artifacts', - 'release', 'release_test']) + 'release', 'release_test', 'podman_machine_windows']) def setUp(self): super().setUp()