-
Notifications
You must be signed in to change notification settings - Fork 3k
uVisor: Upgrade to v0.31.0 #5275
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
Conversation
uvisor CI fails? Is this expected and waiting for CI update or? |
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.
We also need to update the test pointer, features/FEATURE_UVISOR/uvisor-tests.txt
from the uVisor repo.
Build : FAILUREBuild number : 53 |
Any update for this patch, regarding CI? |
It should be fixed now.
Currently there are some instabilities with our CI due to resource exhaustion.
We will fix this and re-trigger builds.
|
retest uvisor |
1 similar comment
retest uvisor |
ac62742
to
07b8474
Compare
@Patater could you please re-review and let us know if you are happy with this ? |
07b8474
to
0bf51ac
Compare
0bf51ac
to
fb9fd5b
Compare
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.
LGTM. Let’s see CI pass :)
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.
LGTM
/morph build |
Build : SUCCESSBuild number : 183 Skipping test trigger, missing label 'NEED CI' |
ping - this is pretty urgent for uVisor CI - please handle ASAP |
Hi @alzix , the morph test was completd and green (but did not report back!), will be resolved within today |
/morph test |
Build : FAILUREBuild number : 188 |
The above failure (java) , as on master we need one more fix to make tests green, we are waiting for that one to get in before this one, will keep you updated. I hope there is no cross dependency there :) |
Test : FAILUREBuild number : 89 |
/morph build |
Build : SUCCESSBuild number : 194 Triggering tests/morph test |
Test : SUCCESSBuild number : 93 |
please merge this ASAP so the other PR's will succeed running uvisor requirement |
Build : FAILUREBuild number : 217 |
/morph build |
Build : SUCCESSBuild number : 218 Triggering tests/morph test |
Test : SUCCESSBuild number : 107 |
Description
Upgrade uVisor to v0.31.0:
Status
READY
Migrations
debug box APIs have changed. Refer to https://github.com/ARMmbed/uvisor/blob/master/docs/lib/DEBUGGING.md
YES
@Patater - please also review