Skip to content
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

Rollup PR #8676

Merged
merged 16 commits into from Nov 8, 2018
Merged

Rollup PR #8676

merged 16 commits into from Nov 8, 2018

Conversation

0xc0170
Copy link
Contributor

@0xc0170 0xc0170 commented Nov 8, 2018

Description

All bundled PRs experienced export node failures that were unrelated to the PR.
This rollup PR is an attempt to put them through final testing at once, without severely loading the CI.

This Rollup PR contains the following bundled PRs:

Pull request type

[x] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[ ] Docs update
[ ] Test update
[ ] Breaking change

kapi90 and others added 16 commits October 29, 2018 15:30
Ethernet controller. To ensure proper operation, some methods
needed to be updated in the SMSC9220's native driver as well.
It passes all related Greentea tests, however when supervised by
the Python environment it tends to fail because of Timeout.

The current timeout is set to 1200s that seems to be a little bit short
to finish all test cases, the timeout happens towards the end of the
last test case.

Change-Id: I914608c34828b493a80e133cd132537a297bfc84
Signed-off-by: Bence Kaposzta <bence.kaposzta@arm.com>
### Description

The DS-5 exporter does not work. It exports projects which I'm 100% sure
don't build. I have been looking over supporting it for about 6 months
now and I have convinced myself that I don't think it's worth the
trouble. Shout now if you need this exporter.

### Pull request type

    [ ] Fix
    [ ] Refactor
    [ ] Target update
    [x] Functionality change
    [ ] Docs update
    [ ] Test update
    [ ] Breaking change
Reset _rtc_lp_timer only if DEVICE_LPTICKER is defined.
When DEVICE_LPTICKER is defined set_time() only works correctly on
the first call. This test calls set_time() twice and ensures the
time set by both calls is correct.  This test only runs if
DEVICE_RTC or DEVICE_LPTICKER is defined.
On platforms using both tickless and the low power ticker wrapper
so much of the background stack is used that it overflows. To
ensure the background thread's stack doesn't overflow increase this
size by 256 bytes when tickless is enabled. Worst case usage
on the NUCLEO_F476RG was recorded at 656 when tickless is
turned on so this increased size should safely prevent overflows.
@0xc0170 0xc0170 changed the title Dev rollup Rollup PR Nov 8, 2018
@0xc0170
Copy link
Contributor Author

0xc0170 commented Nov 8, 2018

/morph build

Copy link
Contributor

@NirSonnenschein NirSonnenschein left a comment

Choose a reason for hiding this comment

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

approved, but given the scope I don't have a high confidence that something wasn't missed

@0xc0170
Copy link
Contributor Author

0xc0170 commented Nov 8, 2018

/morph build

@cmonr
Copy link
Contributor

cmonr commented Nov 8, 2018

The github notify portion of the build job got stuck :(

/morph build

@mbed-ci
Copy link

mbed-ci commented Nov 8, 2018

Build : SUCCESS

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

Triggering tests

/morph test
/morph export-build
/morph mbed2-build

@mbed-ci
Copy link

mbed-ci commented Nov 8, 2018

@mbed-ci
Copy link

mbed-ci commented Nov 8, 2018

@cmonr cmonr merged commit dee3506 into ARMmbed:master Nov 8, 2018
@cmonr cmonr removed the needs: CI label Nov 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants