Merge ~hyask/curtin:skia/disable_flaky_test into curtin:master

Proposed by Skia
Status: Merged
Approved by: Dan Bungert
Approved revision: 519f0b3fa580a37a0b32e1a450bb92f31d04e462
Merge reported by: Server Team CI bot
Merged at revision: not available
Proposed branch: ~hyask/curtin:skia/disable_flaky_test
Merge into: curtin:master
Diff against target: 17 lines (+3/-3)
1 file modified
tests/vmtests/test_mdadm_bcache.py (+3/-3)
Reviewer Review Type Date Requested Status
Server Team CI bot continuous-integration Approve
curtin developers Pending
Review via email: mp+460918@code.launchpad.net

Commit message

Disable a flaky test to reach forever green CI.

To post a comment you must log in.
Revision history for this message
Server Team CI bot (server-team-bot) wrote :

FAILED: Continuous integration, rev:519f0b3fa580a37a0b32e1a450bb92f31d04e462

No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want jenkins to rebuild you need to trigger it yourself):
https://code.launchpad.net/~hyask/curtin/+git/curtin/+merge/460918/+edit-commit-message

https://jenkins.canonical.com/server-team/job/curtin-ci/228/
Executed test runs:
    SUCCESS: https://jenkins.canonical.com/server-team/job/curtin-ci/nodes=metal-amd64/228/
    SUCCESS: https://jenkins.canonical.com/server-team/job/curtin-ci/nodes=metal-arm64/228/
    SUCCESS: https://jenkins.canonical.com/server-team/job/curtin-ci/nodes=metal-ppc64el/228/
    SUCCESS: https://jenkins.canonical.com/server-team/job/curtin-ci/nodes=metal-s390x/228/

Click here to trigger a rebuild:
https://jenkins.canonical.com/server-team/job/curtin-ci/228//rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Server Team CI bot (server-team-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Skia (hyask) wrote :

It seems I don't have the rights to push the merge to master, and there doesn't seem to be an auto-merge bot. :-)

Revision history for this message
Dan Bungert (dbungert) wrote :

There is an automerge bot but it's behind tests that take about half an hour.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/tests/vmtests/test_mdadm_bcache.py b/tests/vmtests/test_mdadm_bcache.py
2index 83be785..8de7702 100644
3--- a/tests/vmtests/test_mdadm_bcache.py
4+++ b/tests/vmtests/test_mdadm_bcache.py
5@@ -627,9 +627,9 @@ class FocalTestAllindata(relbase.focal, TestAllindataAbs):
6 __test__ = True
7
8
9-class JammyTestAllindata(relbase.jammy, TestAllindataAbs):
10- expected_failure = True # XXX Broken for now
11- __test__ = True
12+# XXX @skip doesn't work here, as the setup is triggered anyway, and is flaky
13+# class JammyTestAllindata(relbase.jammy, TestAllindataAbs):
14+# __test__ = True
15
16
17 # vi: ts=4 expandtab syntax=python

Subscribers

People subscribed via source and target branches