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

run example-tls on faster K64F board (than K66F) #14797

Merged
merged 1 commit into from
Jun 17, 2021

Conversation

saheerb
Copy link
Contributor

@saheerb saheerb commented Jun 17, 2021

Summary of changes

https://github.com/ARMmbed/mbed-os-example-tls is currently tested with K66F which is slow and causing timeout in nightly builds. This PR updates the config to run run example-tls on K64F.

Impact of changes

None

Migration actions required

None

Documentation

None


Pull request type

[X] Patch update (Bug fix / Target update / Docs update / Test update / Refactor)
[] Feature update (New feature / Functionality change / New API)
[] Major update (Breaking change E.g. Return code change / API behaviour change)

Test results

[] No Tests required for this change (E.g docs only update)
[X] Covered by existing mbed-os tests (Greentea or Unittest)
[] Tests / results supplied as part of this PR

Reviewers

@LDong-Arm


Copy link
Contributor

@LDong-Arm LDong-Arm left a comment

Choose a reason for hiding this comment

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

Note: It's not yet clear why RSA-4096 key generation on K66F is way slower despite having higher hardware specs. For now, use K64F for the benchmark example to unblock CI.

@0xc0170 0xc0170 added needs: CI release-type: patch Indentifies a PR as containing just a patch labels Jun 17, 2021
@0xc0170
Copy link
Contributor

0xc0170 commented Jun 17, 2021

I'll close and open for travis to retrigger.

@0xc0170 0xc0170 closed this Jun 17, 2021
@mergify mergify bot removed needs: CI release-type: patch Indentifies a PR as containing just a patch labels Jun 17, 2021
@0xc0170 0xc0170 reopened this Jun 17, 2021
@0xc0170 0xc0170 added needs: CI release-type: patch Indentifies a PR as containing just a patch labels Jun 17, 2021
@0xc0170
Copy link
Contributor

0xc0170 commented Jun 17, 2021

Ci started

Travis as well

@mbed-ci
Copy link

mbed-ci commented Jun 17, 2021

Jenkins CI Test : ❌ FAILED

Build Number: 1 | 🔒 Jenkins CI Job | 🌐 Logs & Artifacts

CLICK for Detailed Summary

jobs Status
jenkins-ci/mbed-os-ci_unittests ✔️
jenkins-ci/mbed-os-ci_build-cloud-example-ARM ✔️
jenkins-ci/mbed-os-ci_cmake-cloud-example-ARM ✔️
jenkins-ci/mbed-os-ci_build-cloud-example-GCC_ARM ✔️
jenkins-ci/mbed-os-ci_cmake-cloud-example-GCC_ARM ✔️
jenkins-ci/mbed-os-ci_build-greentea-ARM ✔️
jenkins-ci/mbed-os-ci_build-greentea-GCC_ARM ✔️
jenkins-ci/mbed-os-ci_cmake-example-ARM ✔️
jenkins-ci/mbed-os-ci_cmake-example-GCC_ARM ✔️
jenkins-ci/mbed-os-ci_build-example-ARM ✔️
jenkins-ci/mbed-os-ci_build-example-GCC_ARM ✔️
jenkins-ci/mbed-os-ci_greentea-test

@LDong-Arm
Copy link
Contributor

CI issue?

[2021-06-17T12:47:31.276Z] Exited https://jenkins-mbedos.oss.arm.com/job/mbed-os-ci_greentea-test with result org.jenkinsci.plugins.workflow.steps.FlowInterruptedException

@saheerb
Copy link
Contributor Author

saheerb commented Jun 17, 2021

CI issue?

[2021-06-17T12:47:31.276Z] Exited https://jenkins-mbedos.oss.arm.com/job/mbed-os-ci_greentea-test with result org.jenkinsci.plugins.workflow.steps.FlowInterruptedException

Yes, fixed some scripts. Restarted CI .

@mbed-ci
Copy link

mbed-ci commented Jun 17, 2021

Jenkins CI Test : ✔️ SUCCESS

Build Number: 2 | 🔒 Jenkins CI Job | 🌐 Logs & Artifacts

CLICK for Detailed Summary

jobs Status
jenkins-ci/mbed-os-ci_unittests ✔️
jenkins-ci/mbed-os-ci_cmake-cloud-example-ARM ✔️
jenkins-ci/mbed-os-ci_build-cloud-example-ARM ✔️
jenkins-ci/mbed-os-ci_build-cloud-example-GCC_ARM ✔️
jenkins-ci/mbed-os-ci_cmake-cloud-example-GCC_ARM ✔️
jenkins-ci/mbed-os-ci_build-greentea-ARM ✔️
jenkins-ci/mbed-os-ci_build-greentea-GCC_ARM ✔️
jenkins-ci/mbed-os-ci_cmake-example-ARM ✔️
jenkins-ci/mbed-os-ci_cmake-example-GCC_ARM ✔️
jenkins-ci/mbed-os-ci_build-example-GCC_ARM ✔️
jenkins-ci/mbed-os-ci_build-example-ARM ✔️
jenkins-ci/mbed-os-ci_greentea-test ✔️

@0xc0170 0xc0170 merged commit 1a2e96b into ARMmbed:master Jun 17, 2021
@mergify mergify bot removed the ready for merge label Jun 17, 2021
@mbedmain mbedmain added release-version: 6.12.0 Release-pending and removed release-type: patch Indentifies a PR as containing just a patch Release-pending labels Jun 18, 2021
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.

None yet

5 participants