Skip to content

Gemalto Crinterion UDP test adjustments #9288

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

Conversation

michalpasztamobica
Copy link
Contributor

Description

Adjusting the tests to match the cellular modems ELS61-E2 and BGS2-W requirements.

Pull request type

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

Reviewers

@SeppoTakalo
@VeijoPesonen
@AriParkkila

@ciarmcom
Copy link
Member

ciarmcom commented Jan 8, 2019

@michalpasztamobica, thank you for your changes.
@AriParkkila @VeijoPesonen @SeppoTakalo @ARMmbed/mbed-os-test @ARMmbed/mbed-os-ipcore @ARMmbed/mbed-os-maintainers please review.

Copy link
Contributor

@0xc0170 0xc0170 left a comment

Choose a reason for hiding this comment

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

Adjusting the tests to match the cellular modems ELS61-E2 and BGS2-W requirements.

CAn you add this to the commit msg (explaning the change from 100 to 1000 and setting the timeout)

@michalpasztamobica michalpasztamobica force-pushed the gemalto_crinterion_test_updates branch from 0555cd5 to a2e5ae3 Compare January 8, 2019 14:16
@michalpasztamobica
Copy link
Contributor Author

Extended commit message.

@0xc0170
Copy link
Contributor

0xc0170 commented Jan 14, 2019

CI started

@mbed-ci
Copy link

mbed-ci commented Jan 14, 2019

Test run: SUCCESS

Summary: 9 of 9 test jobs passed
Build number : 1
Build artifacts

@0xc0170
Copy link
Contributor

0xc0170 commented Jan 15, 2019

I restarted travis, did not report status back

@0xc0170
Copy link
Contributor

0xc0170 commented Jan 15, 2019

@michalpasztamobica Can you rebase this on top of latest master ? one python module fix is on master (toolspy should be fixed by that)

Adjusting the tests to match the cellular modems ELS61-E2 and BGS2-W requirements.
@michalpasztamobica michalpasztamobica force-pushed the gemalto_crinterion_test_updates branch from a2e5ae3 to 6e629e1 Compare January 15, 2019 11:39
@michalpasztamobica
Copy link
Contributor Author

@0xc0170 , rebased

@0xc0170
Copy link
Contributor

0xc0170 commented Jan 15, 2019

CI started

@mbed-ci
Copy link

mbed-ci commented Jan 15, 2019

Test run: FAILED

Summary: 1 of 6 test jobs failed
Build number : 2
Build artifacts

Failed test jobs:

  • jenkins-ci/mbed-os-ci_build-ARM

@0xc0170
Copy link
Contributor

0xc0170 commented Jan 15, 2019

CI job was aborted, 5.11.2 RC candidate is in the CI. We will restart it as soon as 5.11.2 RC is ready

@cmonr
Copy link
Contributor

cmonr commented Jan 16, 2019

CI started

@mbed-ci
Copy link

mbed-ci commented Jan 16, 2019

Test run: SUCCESS

Summary: 9 of 9 test jobs passed
Build number : 3
Build artifacts

@cmonr cmonr merged commit bfc4646 into ARMmbed:master Jan 17, 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.

8 participants