Skip to content

Adapt actuator-provider to the latest zenoh-pico version#11

Open
max-grzanna wants to merge 1 commit intoeclipse-sdv-blueprints:mainfrom
boschglobal:update-actuator-provider
Open

Adapt actuator-provider to the latest zenoh-pico version#11
max-grzanna wants to merge 1 commit intoeclipse-sdv-blueprints:mainfrom
boschglobal:update-actuator-provider

Conversation

@max-grzanna
Copy link
Contributor

This PR implements compatibility updates for zenoh-pico version 1.3.4.

Key changes:

  • Removed redundant build flags due to library improvements
  • Updated zenoh-kuksa-provider configuration to establish connection with the zenoh-router, ensuring proper update delivery to the actuator-provider
  • adapted the actuator-provider to the major API changes in zenoh-pico

@sophokles73
Copy link
Contributor

@eriksven are you in a position to also review this PR? Or should we simply merge it?

@eriksven
Copy link
Contributor

eriksven commented Dec 4, 2025

Good question and thanks for the reminder. @max-grzanna What do you think?

The thing I am a bit concerned about is that this PR uses a different Zenoh version (1.3.4) compared to the rest of the blueprint at the moment (1.6.2).

@sophokles73 do you have insights of how many potential breaking changes we should expect here? I know that following semantic versioning these version should be compatible but based on previous experience I would like to at least test it before merging.

@sophokles73
Copy link
Contributor

@sophokles73 do you have insights of how many potential breaking changes we should expect here? I know that following semantic versioning these version should be compatible but based on previous experience I would like to at least test it before merging.

Maybe @max-grzanna can update the PR to use zenoh 1.6.x as well?

@max-grzanna
Copy link
Contributor Author

@sophokles73 do you have insights of how many potential breaking changes we should expect here? I know that following semantic versioning these version should be compatible but based on previous experience I would like to at least test it before merging.

Maybe @max-grzanna can update the PR to use zenoh 1.6.x as well?

Unfortunately, I don't have any hardware available for testing at the moment. However, I will have some next week and will be able to take a look at the version upgrade.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants