Skip to content

tests-mbed_hal-sleep_manager: fix regression found on since PR #11721 #11956

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
Nov 27, 2019

Conversation

ABOSTM
Copy link
Contributor

@ABOSTM ABOSTM commented Nov 27, 2019

Description

Summary of change

tests-mbed_hal-sleep_manager is failed since merge of PR #11721 Bare metal greentea support
on many STM32 boards:
NUCLEO_F072RB, NUCLEO_F070RB
NUCLEO_F411RE, NUCLEO_F439ZI, NUCLEO_F446ZE, DISCO_F407VG
NUCLEO_F303RE
...

This PR solve the issue.

Documentation


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)
[] Covered by existing mbed-os tests (Greentea or Unittest)
[] Tests / results supplied as part of this PR

Test PASSED on NUCLEO_F072RB and NUCLEO_F411RE


Reviewers


Release Notes

Summary of changes

Impact of changes

Migration actions required

…d#11721

This test is failed since ARMmbed#11721 Bare metal greentea support
on many STM32 boards:
NUCLEO_F072RB, NUCLEO_F070RB
NUCLEO_F411RE, NUCLEO_F439ZI, NUCLEO_F446ZE, DISCO_F407VG
NUCLEO_F303RE
...
@jeromecoutant
Copy link
Collaborator

Thx Alexandre

@rajkan01

@0xc0170
Copy link
Contributor

0xc0170 commented Nov 27, 2019

I've checked nightly, this is there. Marked as 5.15

@0xc0170
Copy link
Contributor

0xc0170 commented Nov 27, 2019

CI started

@evedon
Copy link
Contributor

evedon commented Nov 27, 2019

Thanks for fixing.

@mbed-ci
Copy link

mbed-ci commented Nov 27, 2019

Test run: FAILED

Summary: 1 of 5 test jobs failed
Build number : 1
Build artifacts

Failed test jobs:

  • jenkins-ci/mbed-os-ci_greentea-test

@evedon
Copy link
Contributor

evedon commented Nov 27, 2019

Why was not this picked up in CI?

@0xc0170
Copy link
Contributor

0xc0170 commented Nov 27, 2019

The failure above is with one device, needs restarting in ci (@ARMmbed/mbed-os-test already notified). Will restart once fixed

Why was not this picked up in CI?

I don't know, we need to find out. I've checked the PR tests and mbed_hal-sleep_manager was tested and passed. I've checked specifically nucleo f411.

@0xc0170
Copy link
Contributor

0xc0170 commented Nov 27, 2019

CI restarted

@0xc0170
Copy link
Contributor

0xc0170 commented Nov 27, 2019

I can see nightly started failing on Nov 14, only some platforms have sleep manager failures.

mbed-os-ci_greentea-test-nightly job 41 and above should have these failures

@0xc0170 0xc0170 removed the needs: CI label Nov 27, 2019
@0xc0170 0xc0170 merged commit d5f813b into ARMmbed:master Nov 27, 2019
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.

5 participants