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

Release candidate for mbed-os-5.8.6 #7109

Merged
merged 46 commits into from Jun 5, 2018
Merged

Release candidate for mbed-os-5.8.6 #7109

merged 46 commits into from Jun 5, 2018

Conversation

adbridge
Copy link
Contributor

@adbridge adbridge commented Jun 4, 2018

No description provided.

@adbridge
Copy link
Contributor Author

adbridge commented Jun 4, 2018

/morph build
/morph mbed2-release

@adbridge
Copy link
Contributor Author

adbridge commented Jun 4, 2018

pr-head failing with:

Build failures:

  * K64F::ARM::TESTS-LORAWAN-LORARADIO

        Building project loraradio (K64F, ARM)

        Scan: ARM

        Scan: loraradio

        Scan: FEATURE_LWIP

        Scan: FEATURE_STORAGE

        Compile [100.0%]: main.cpp

        [Error] main.cpp@23,0:  #5: cannot open source input file "mbed_trace.h": No such file or directory

jeromecoutant and others added 28 commits June 4, 2018 17:38
If user adds USE_USER_DEFINED_HAL_ETH_MSPINIT macro in the json config,
he can implement his own HAL_ETH_MspDeInit and HAL_ETH_MspDeInit functions.
Fixing a bug.  .mbed is ignored in the high level .gitignore file.  But .mbed is actually a needed file inside of tools/export.  Even though the file is still in the main repo, since it's ignored now, you can easily lose it or remove it without seeing the change.  Adding this .gitignore file here will add .mbed back into the tracked files for only this directory
Referencing to "UG257: EFM32 Pearl Gecko PG12 Starter Kit User's Guide - Chapter 4" (https://www.silabs.com/documents/public/user-guides/ug257-stk3402-usersguide.pdf) it should be like in this commit.
* Updated pinout of EXP header from pre-production to production
* Move assignment of PF7 to SPI_2 to be compatible with the new SPI flash routing
* Since mbed does not overwrite itself, make the flashing routines run out of flash by default
* Report a writeable size of 4 bytes (previously erroneously reported a full eraseable page as the minimum write size)
* IRQ handling got updated previously to a non-functional state when both callbacks were registered (it'd fire a fall callback for both rise and fall events). With this update, that faulty behaviour is corrected. Due to delays between the detection of the edge and the handling of the interrupt (and the fact that information about which edge you received on the pin is not stored anywhere), there is no way to be absolutely sure which edge got triggered on the pin. Therefore, we make a best-guess effort by looking at the pin state at the time of IRQ handling, and fire a callback as if that was the end state of the event. This will usually work out fine, except in cases were the signal is toggling faster than the IRQ handler's response time. In that case, a user won't get both callbacks (as expected for a pulse), but only the last event.
* Stripped some dead code.
I fixed redeclaration of type name "bool_t" for target Renesas because this typedef has been defined in rtx_core_ca.h by #6273.
@adbridge
Copy link
Contributor Author

adbridge commented Jun 4, 2018

Updated: Pushed loraradio tests to 5.9.1

@adbridge
Copy link
Contributor Author

adbridge commented Jun 4, 2018

/morph build
/morph mbed2-release

@adbridge
Copy link
Contributor Author

adbridge commented Jun 4, 2018

pr-head failed this time because of:

[ERROR] Error: C9555E: Failed to check out a license.
Unable to connect to the license server. Check that ARMLMD_LICENSE_FILE is set correctly, and the license server is available.
Information about this error is available at: http://ds.arm.com/support/lic56/m96
 General licensing information is available at: http://ds.arm.com/support/licensing/
 If you need further help, provide this complete error report to your supplier or license.support@arm.com.
 - ARMLMD_LICENSE_FILE: '8225@armcc.licenses.mbedcloudtesting.com'

@mbed-ci
Copy link

mbed-ci commented Jun 4, 2018

Build : FAILURE

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

@mbed-ci
Copy link

mbed-ci commented Jun 4, 2018

Build : FAILURE

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

@cmonr
Copy link
Contributor

cmonr commented Jun 4, 2018

The most recent failure appears to be caused by an update to the nanostack-border-router example for 5.9.0-rc1 testing, but which has broken 5.8 compatibility.

PelionIoT/nanostack-border-router#85 (comment)

@0xc0170
Copy link
Contributor

0xc0170 commented Jun 5, 2018

/morph build

@mbed-ci
Copy link

mbed-ci commented Jun 5, 2018

Build : SUCCESS

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

Triggering tests

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

@adbridge
Copy link
Contributor Author

adbridge commented Jun 5, 2018

@kjbracey-arm @OPpuolitaival Any ideas why cloud client example is failing with general SCM issues ?

https://jenkins-internal.mbed.com/job/ARMmbed/job/mbed-cloud-client-example-sources-internal/job/master/4799/console

@kjbracey
Copy link
Contributor

kjbracey commented Jun 5, 2018

@adbridge
Copy link
Contributor Author

adbridge commented Jun 5, 2018

Weird it isn't even a consistent set of boards that are failing

@OPpuolitaival
Copy link
Contributor

Seems jenkins problem. Asking support from infra team

@adbridge
Copy link
Contributor Author

adbridge commented Jun 5, 2018

@OPpuolitaival Thanks, can we stress the urgency of this as it is blocking 5.8.6 candidate and could also impact any other PRs going into the 5.9 release...

@mbed-ci
Copy link

mbed-ci commented Jun 5, 2018

@OPpuolitaival
Copy link
Contributor

@miklis said that it is most probably limitation of github API calls

@adbridge
Copy link
Contributor Author

adbridge commented Jun 5, 2018

@miklis @OPpuolitaival So how do we fix this, this was restarted twice and still failed ?

@mbed-ci
Copy link

mbed-ci commented Jun 5, 2018

@adbridge
Copy link
Contributor Author

adbridge commented Jun 5, 2018

/morph test

@adbridge
Copy link
Contributor Author

adbridge commented Jun 5, 2018

@OPpuolitaival Can we force the successful run status to this PR? I didn't realise you had kicked it off again so I restarted and then aborted and of course now the status is only showing aborted :(

@OPpuolitaival
Copy link
Contributor

@adbridge I think that we need to run that again to get that check in green

@adbridge
Copy link
Contributor Author

adbridge commented Jun 5, 2018

OK have restarted pr-head

@mbed-ci
Copy link

mbed-ci commented Jun 5, 2018

@cmonr cmonr merged commit 35fa909 into mbed-os-5.8 Jun 5, 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