Message ID | 20241022171826.1292640-1-ross.burton@arm.com |
---|---|
State | Accepted, archived |
Commit | 177e268811c04260923ac4b16fa047315304add0 |
Headers | show |
Series | [1/2] oeqa/selftest/rust: skip on all MIPS platforms | expand |
Lgtm On Tue, Oct 22, 2024 at 10:18 AM Ross Burton via lists.openembedded.org <ross.burton=arm.com@lists.openembedded.org> wrote: > As per "The rustc book"[1], mips*-unknown-linux-* are "tier 3" targets: > > Tier 3 targets are those which the Rust codebase has support for, but > which the Rust project does not build or test automatically, so they > may or may not work. > > We already skip qemumips in this selftest, but we're now also seeing > failures with qemumips64 so refactor the test to skip all machines where > the architecture is mips or mips64. > > [1] https://doc.rust-lang.org/nightly/rustc/platform-support.html > > Signed-off-by: Ross Burton <ross.burton@arm.com> > --- > meta/lib/oeqa/selftest/cases/rust.py | 8 +++----- > 1 file changed, 3 insertions(+), 5 deletions(-) > > diff --git a/meta/lib/oeqa/selftest/cases/rust.py > b/meta/lib/oeqa/selftest/cases/rust.py > index 88d7bb95189..fab6267b155 100644 > --- a/meta/lib/oeqa/selftest/cases/rust.py > +++ b/meta/lib/oeqa/selftest/cases/rust.py > @@ -3,6 +3,7 @@ import os > import subprocess > import time > from oeqa.core.decorator import OETestTag > +from oeqa.core.decorator.data import skipIfArch > from oeqa.core.case import OEPTestResultTestCase > from oeqa.selftest.case import OESelftestTestCase > from oeqa.utils.commands import runCmd, bitbake, get_bb_var, get_bb_vars, > runqemu, Command > @@ -38,15 +39,12 @@ def parse_results(filename): > @OETestTag("toolchain-user") > @OETestTag("runqemu") > class RustSelfTestSystemEmulated(OESelftestTestCase, > OEPTestResultTestCase): > + > + @skipIfArch(['mips', 'mips64']) > def test_rust(self, *args, **kwargs): > # Disable Rust Oe-selftest > #self.skipTest("The Rust Oe-selftest is disabled.") > > - # Skip mips32 target since it is unstable with rust tests > - machine = get_bb_var('MACHINE') > - if machine == "qemumips": > - self.skipTest("The mips32 target is skipped for Rust > Oe-selftest.") > - > # build remote-test-server before image build > recipe = "rust" > start_time = time.time() > -- > 2.34.1 > > > -=-=-=-=-=-=-=-=-=-=-=- > Links: You receive all messages sent to this group. > View/Reply Online (#206167): > https://lists.openembedded.org/g/openembedded-core/message/206167 > Mute This Topic: https://lists.openembedded.org/mt/109156055/1997914 > Group Owner: openembedded-core+owner@lists.openembedded.org > Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [ > raj.khem@gmail.com] > -=-=-=-=-=-=-=-=-=-=-=- > >
diff --git a/meta/lib/oeqa/selftest/cases/rust.py b/meta/lib/oeqa/selftest/cases/rust.py index 88d7bb95189..fab6267b155 100644 --- a/meta/lib/oeqa/selftest/cases/rust.py +++ b/meta/lib/oeqa/selftest/cases/rust.py @@ -3,6 +3,7 @@ import os import subprocess import time from oeqa.core.decorator import OETestTag +from oeqa.core.decorator.data import skipIfArch from oeqa.core.case import OEPTestResultTestCase from oeqa.selftest.case import OESelftestTestCase from oeqa.utils.commands import runCmd, bitbake, get_bb_var, get_bb_vars, runqemu, Command @@ -38,15 +39,12 @@ def parse_results(filename): @OETestTag("toolchain-user") @OETestTag("runqemu") class RustSelfTestSystemEmulated(OESelftestTestCase, OEPTestResultTestCase): + + @skipIfArch(['mips', 'mips64']) def test_rust(self, *args, **kwargs): # Disable Rust Oe-selftest #self.skipTest("The Rust Oe-selftest is disabled.") - # Skip mips32 target since it is unstable with rust tests - machine = get_bb_var('MACHINE') - if machine == "qemumips": - self.skipTest("The mips32 target is skipped for Rust Oe-selftest.") - # build remote-test-server before image build recipe = "rust" start_time = time.time()
As per "The rustc book"[1], mips*-unknown-linux-* are "tier 3" targets: Tier 3 targets are those which the Rust codebase has support for, but which the Rust project does not build or test automatically, so they may or may not work. We already skip qemumips in this selftest, but we're now also seeing failures with qemumips64 so refactor the test to skip all machines where the architecture is mips or mips64. [1] https://doc.rust-lang.org/nightly/rustc/platform-support.html Signed-off-by: Ross Burton <ross.burton@arm.com> --- meta/lib/oeqa/selftest/cases/rust.py | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-)