Skip to content

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

Merged
merged 1 commit into from
Oct 18, 2017
Merged

Conversation

alzix
Copy link
Contributor

@alzix alzix commented Oct 9, 2017

Description

Upgrade uVisor to v0.31.0:

  • stability enhacements
  • new static debug box

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

@0xc0170
Copy link
Contributor

0xc0170 commented Oct 9, 2017

uvisor CI fails? Is this expected and waiting for CI update or?

@Patater Patater self-assigned this Oct 9, 2017
Copy link
Contributor

@Patater Patater left a 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.

@mbed-ci
Copy link

mbed-ci commented Oct 10, 2017

Build : FAILURE

Build number : 53
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5275/

@0xc0170
Copy link
Contributor

0xc0170 commented Oct 13, 2017

Any update for this patch, regarding CI?

@alzix
Copy link
Contributor Author

alzix commented Oct 15, 2017 via email

@orenc17
Copy link
Contributor

orenc17 commented Oct 15, 2017

retest uvisor

1 similar comment
@orenc17
Copy link
Contributor

orenc17 commented Oct 15, 2017

retest uvisor

@alzix alzix force-pushed the upgrade-uvisor-v0.31.0 branch 2 times, most recently from ac62742 to 07b8474 Compare October 16, 2017 12:36
@adbridge
Copy link
Contributor

@Patater could you please re-review and let us know if you are happy with this ?

@alzix alzix force-pushed the upgrade-uvisor-v0.31.0 branch from 07b8474 to 0bf51ac Compare October 16, 2017 14:15
@alzix alzix force-pushed the upgrade-uvisor-v0.31.0 branch from 0bf51ac to fb9fd5b Compare October 16, 2017 14:27
Copy link
Contributor

@Patater Patater left a 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 :)

Copy link
Contributor

@adbridge adbridge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@adbridge
Copy link
Contributor

/morph build

@mbed-ci
Copy link

mbed-ci commented Oct 16, 2017

Build : SUCCESS

Build number : 183
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5275/

Skipping test trigger, missing label 'NEED CI'

@alzix
Copy link
Contributor Author

alzix commented Oct 16, 2017

ping - this is pretty urgent for uVisor CI - please handle ASAP

@0xc0170
Copy link
Contributor

0xc0170 commented Oct 17, 2017

Hi @alzix , the morph test was completd and green (but did not report back!), will be resolved within today

@0xc0170
Copy link
Contributor

0xc0170 commented Oct 17, 2017

/morph test

@mbed-ci
Copy link

mbed-ci commented Oct 17, 2017

Build : FAILURE

Build number : 188
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5275/

@0xc0170
Copy link
Contributor

0xc0170 commented Oct 17, 2017

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 :)

@mbed-ci
Copy link

mbed-ci commented Oct 17, 2017

@0xc0170
Copy link
Contributor

0xc0170 commented Oct 17, 2017

/morph build

@mbed-ci
Copy link

mbed-ci commented Oct 17, 2017

Build : SUCCESS

Build number : 194
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5275/

Triggering tests

/morph test

@mbed-ci
Copy link

mbed-ci commented Oct 17, 2017

@orenc17
Copy link
Contributor

orenc17 commented Oct 18, 2017

please merge this ASAP so the other PR's will succeed running uvisor requirement

@mbed-ci
Copy link

mbed-ci commented Oct 18, 2017

Build : FAILURE

Build number : 217
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5275/

@0xc0170
Copy link
Contributor

0xc0170 commented Oct 18, 2017

/morph build

@mbed-ci
Copy link

mbed-ci commented Oct 18, 2017

Build : SUCCESS

Build number : 218
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5275/

Triggering tests

/morph test
/morph uvisor-test

@mbed-ci
Copy link

mbed-ci commented Oct 18, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants