-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
trunner: add armv8r52-mps3an536-qemu target #366
base: master
Are you sure you want to change the base?
Conversation
JIRA: CI-468
6b5b105
to
2f73bcb
Compare
rootfs = False | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If most of the tests haven't been run yet, and if we only want to run specific ones for now, I would assume for example, psh tests, I would set experimental to True and "include" the given target in given test yaml.
rootfs = False | |
rootfs = False | |
experimental = True |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree on the suggestion, we should set is as experimental for now and enable only psh tests (in yaml)
@@ -94,3 +94,15 @@ def __init__(self): | |||
@classmethod | |||
def from_context(cls, _: TestContext): | |||
return cls() | |||
|
|||
|
|||
class ARMv8R52_MPS3_AN536QemuTarget(QemuTarget): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
From what I can see, we don't use _
in other target class names. I would stick to one consistent naming convention.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@adamdebek please apply the suggestion
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mateusz-bloch I'm aware of this issue, it's related to toolchain - I've found the root cause but now have no time to try to fix it. I will get back to it. Also: any application will fail on exit - so no point in running other tests on this target now. |
@@ -94,3 +94,15 @@ def __init__(self): | |||
@classmethod | |||
def from_context(cls, _: TestContext): | |||
return cls() | |||
|
|||
|
|||
class ARMv8R52_MPS3_AN536QemuTarget(QemuTarget): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@adamdebek please apply the suggestion
rootfs = False | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree on the suggestion, we should set is as experimental for now and enable only psh tests (in yaml)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please enable this target to CI in a separate commit
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Due to resolution of issue #1171, you can check if psh tests are works correctly now
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The issue that was stopping these changes have been fixed, please prepare it to merge - apply the suggestions and run CI on this target on PR in phoenix-rtos-project
JIRA: CI-468
Description
Motivation and Context
Types of changes
How Has This Been Tested?
Checklist:
Special treatment