Skip to content

Release candidate for mbed-os-5.13.0-rc4 #10889

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Jun 25, 2019
Merged

Release candidate for mbed-os-5.13.0-rc4 #10889

merged 3 commits into from
Jun 25, 2019

Conversation

adbridge
Copy link
Contributor

No description provided.

Mirela Chirica added 3 commits June 24, 2019 15:15
@adbridge
Cellular: Added IPV6 and IPV4V6 as supported properties for BG96
04d8c70
@adbridge
Cellular: Fix requested socket id in socket_connect routines of BG96 …
bfbf3a7 
…and M26
@adbridge
Cellular: Remove IPV6 and IPV4V6 as supported properties for BG96
1533d24 
IPV6 and IPV6V4 support is also network dependent not only modem.

Having these properties enabled for a modem requires a fallback
mechanism during PDP context activation. This mechanism is missing
at the moment and that can result in imposibility to establish
succesfull connection when network only supports IPV4 contexts.
@adbridge
Copy link
Contributor Author

CI started

@mbed-ci
Copy link

mbed-ci commented Jun 24, 2019

Test run: FAILED

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

Failed test jobs:

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

@0xc0170
Copy link
Contributor

0xc0170 commented Jun 24, 2019

@ARMmbed/mbed-os-test Can you help with the client failure?

@0xc0170
Copy link
Contributor

0xc0170 commented Jun 24, 2019

or is this on the client side (error status) ? cc @teetak01 @JanneKiiskila

@JanneKiiskila
Copy link
Contributor

Execution mode for the files is being changed; 100644 → 100755 - that should not happen.

@0xc0170
Copy link
Contributor

0xc0170 commented Jun 24, 2019

Execution mode for the files is being changed; 100644 → 100755 - that should not happen.

Good catch, fixing now and restarting tests once client completes

@0xc0170
Copy link
Contributor

0xc0170 commented Jun 24, 2019 •
edited
Loading

This was taken to master, not the release script -> it picked up the file already changed :/ Need to catch this in travis or somewhere else.

@0xc0170
Copy link
Contributor

0xc0170 commented Jun 24, 2019

client test passed, I restarted also greentea tests.

I would propose to get this rc4 in. 5.13.1 will contain the fix to revert file permission changes for 2 target files. cc @adbridge @bulislaw

@0xc0170

This comment has been minimized.

Sign in to view
@0xc0170
Copy link
Contributor

0xc0170 commented Jun 25, 2019

The failure is being tracked in #10849

@0xc0170
Copy link
Contributor

0xc0170 commented Jun 25, 2019 •
edited
Loading

All fine from client side (there are failures on their side of things, will be picked up separately - will recheck now). We will continue with 10849 issue as well to have this addressed on master.

@0xc0170 0xc0170 removed the needs: CI label Jun 25, 2019
@0xc0170 0xc0170 merged commit 7482462 into mbed-os-5.13 Jun 25, 2019
@ifyall ifyall mentioned this pull request Jun 28, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@0xc0170 0xc0170 0xc0170 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@adbridge @mbed-ci @0xc0170 @JanneKiiskila