Skip to content

Jenkinsfile updates #2656

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

Closed
wants to merge 318 commits into from
Closed

Jenkinsfile updates #2656

wants to merge 318 commits into from

Conversation

miklis
Copy link
Contributor

@miklis miklis commented Sep 9, 2016

  • Enabled ARM toolchain
  • Cleaned unncessary stages
    • Stages are defined inside compile and buildTestApps functions
  • Added building of mbed-os-cliapp

stage "run mbed-os testapps"
mbed.runTestApps("${env.GIT_REVISION}")
def testApps = [
"mbed-os-cliapp"
Copy link
Contributor

Choose a reason for hiding this comment

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

perhaps this should be move behind buildTestApps() because its private..

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This cannot be after buildTestApps because it is sent as parameter to the function.

@sg-
Copy link
Contributor

sg- commented Sep 10, 2016

Why is this failing?

@jupe
Copy link
Contributor

jupe commented Sep 10, 2016

because we are not paying enough for github: GitHub API rate limit exceeded :/

@jupe
Copy link
Contributor

jupe commented Sep 10, 2016

I triggered job again, perhaps there is no rate limits this time..

@jupe
Copy link
Contributor

jupe commented Sep 10, 2016

now it is pass. First red cross here is obsolete/invalid, I think it related to known jenkins bug, @miklis probably know more about it..

@sg-
Copy link
Contributor

sg- commented Sep 10, 2016

@jupe Thanks for the update. Let me know about the comment and when ready for merge.

@miklis
Copy link
Contributor Author

miklis commented Sep 12, 2016

This fails due to errors in mbed-os-cliapp build. Cause investigation ongoing.

theotherjimmy and others added 28 commits September 12, 2016 12:48
The dict allows the user of the exporter api to specify the result directory
of particular groups of scanned dirs. This will be used by the online exporters
to spoof everything being in the same directory when they are not. It may also
be used by tests, if they would like to export something that looks exactly
like a normal project.
Required to prevent that pesky -1million something error code from
ruining our CI
The timing host tests reported success even if the total drift was
negative. This adds a check for this now.

The wait_us test now does not use a timer and just waits for 100000 us
between prints. This adds inherent drift, but it should still be well
under the limit.
In Thread::terminate() release the join semaphore before terminating
the thread. This allows the join semaphore to be properly signaled in
the case where a thread is terminating itself.
Test that thread self termination works.
Because microseconds timer can be disable during deepsleep
DELTA_DFBM_NQ620 inherit to MCU_NRF52
HW config is the same with NRF52_DK
Only change on serial pin config in PinNames.h
as suggested by adustm
the IPV4 feature was already enabled in an earlier commit but the mbed 5 ethernet initialisation was missing
The linking order of object files affects the actual code placement,
which in turn affects the size of fill section due to the number of
zeros required to maintain appropriate data/code alignment may change.
This is observed when building on Mac and Linux host.

example: mbed compile -m K64F -t GCC_ARM (build 1)
+---------------------+-------+-------+-------+
| Module              | .text | .data |  .bss |
+---------------------+-------+-------+-------+
| Fill                |   120 |     4 |  2381 |
| Misc                | 28755 |  2216 |    84 |
| features/frameworks |  4236 |    52 |   744 |
| hal/common          |  2745 |     4 |   325 |
| hal/targets         | 12116 |    12 |   200 |
| rtos/rtos           |   119 |     4 |     0 |
| rtos/rtx            |  5721 |    20 |  6786 |
| Subtotals           | 53812 |  2312 | 10520 |
+---------------------+-------+-------+-------+

example: mbed compile -m K64F -t GCC_ARM (build 2)
+---------------------+-------+-------+-------+
| Module              | .text | .data |  .bss |
+---------------------+-------+-------+-------+
| Fill                |   128 |     4 |  2381 |
| Misc                | 28755 |  2216 |    84 |
| features/frameworks |  4236 |    52 |   744 |
| hal/common          |  2745 |     4 |   325 |
| hal/targets         | 12116 |    12 |   200 |
| rtos/rtos           |   119 |     4 |     0 |
| rtos/rtx            |  5721 |    20 |  6786 |
| Subtotals           | 53820 |  2312 | 10520 |
+---------------------+-------+-------+-------+

This patch fixes fill section size variation by sorting object
files before passing to linker.

Signed-off-by: Tony Wu <[email protected]>
@miklis miklis closed this Sep 13, 2016
@miklis miklis deleted the jenkinsfile branch September 13, 2016 06:37
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.