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

tests: bluetooth: tester: Add MESH_START command from upstream tester #12572

Merged
merged 2 commits into from
Oct 12, 2023

Conversation

PavelVPV
Copy link
Contributor

@PavelVPV PavelVPV commented Oct 9, 2023

Mesh initalization has been split in upstream into MESH_INIT and MESH_STAR commands in this PR:
zephyrproject-rtos/zephyr#63658

This commits alignes these changes in sdk-nrf tester.

Mesh initalization has been split in upstream into MESH_INIT and
MESH_STAR commands in this PR:
zephyrproject-rtos/zephyr#63658

This commits alignes these changes in sdk-nrf tester.

Signed-off-by: Pavel Vasilyev <[email protected]>
@PavelVPV PavelVPV added the DNM label Oct 9, 2023
@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Oct 9, 2023
@PavelVPV
Copy link
Contributor Author

PavelVPV commented Oct 9, 2023

DNM until the upstream AutoPTS is downstreamed with the patches.

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Oct 9, 2023

Test specification

CI/Jenkins/NRF

  • Integration Platforms

CI/Jenkins/integration

Test Module File based changes Manually selected West overwrite

Detailed information of selected test modules

Note: This message is automatically posted and updated by the CI

@PavelVPV PavelVPV removed the DNM label Oct 11, 2023
Copy link
Contributor

@m-alperen-sener m-alperen-sener left a comment

Choose a reason for hiding this comment

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

It looks fine. Please update the license

@PavelVPV
Copy link
Contributor Author

@carlescufi, should I change the license in these files? Seems like it was always Apache-2.0 here.

@PavelVPV PavelVPV removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Oct 11, 2023
Fixes CI issue:
nrf/tests/bluetooth/tester/src/bttester.h:
Error: "APACHE-2.0" license is not allowed for this file.
nrf/tests/bluetooth/tester/src/mesh.c:
Error: "APACHE-2.0" license is not allowed for this file.
License check failed with 2 error(s) and 0 warning(s)!

Signed-off-by: Pavel Vasilyev <[email protected]>
@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Oct 11, 2023
@PavelVPV PavelVPV added this to the 2.5.0 milestone Oct 12, 2023
@cvinayak cvinayak merged commit b866993 into nrfconnect:main Oct 12, 2023
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants