Skip to content

DDS Gateway communication feature#2817

Merged
FScholPer merged 9 commits into
eclipse-score:mainfrom
Valeo-S-CORE-Organization:PR-DDS_GW-Valeo
Jun 11, 2026
Merged

DDS Gateway communication feature#2817
FScholPer merged 9 commits into
eclipse-score:mainfrom
Valeo-S-CORE-Organization:PR-DDS_GW-Valeo

Conversation

@Abhishek2581

@Abhishek2581 Abhishek2581 commented Apr 20, 2026

Copy link
Copy Markdown
Contributor

This PR adds the DDS Gateway feature documentation to the S-CORE communication stack.

The documentation describes the DDS Gateway as a configurable component enabling inter-ECU communication and interoperability between mw::com (LoLa) and DDS-based systems.

closes #2726

Updated copyright notice formatting and improved text clarity in the DDS Gateway documentation.

Signed-off-by: Abhishek GOYAL <abhishek.goyal@valeo.com>
Expanded the overview and architecture concept of the DDS Gateway, detailing its role in inter-ECU communication and data routing. Enhanced the motivation and key value sections to clarify the benefits and challenges addressed by the DDS Gateway.

Signed-off-by: Abhishek GOYAL <abhishek.goyal@valeo.com>
Removed redundant bullet points and improved clarity in the DDS Gateway documentation.

Signed-off-by: Abhishek GOYAL <abhishek.goyal@valeo.com>
Introduce ECU3 for DDS Application to remove confusion

Signed-off-by: Abhishek GOYAL <abhishek.goyal@valeo.com>
Updated the motivation section to clarify the focus on inter-ECU communication using DDS-based systems and added details about challenges in mixed middleware environments.

Signed-off-by: Abhishek GOYAL <abhishek.goyal@valeo.com>
Signed-off-by: Abhishek GOYAL <abhishek.goyal@valeo.com>
@github-actions

Copy link
Copy Markdown

⚠️ Docs-as-Code version mismatch detected
Please check the CI build logs for details and align the documentation version with the Bazel dependency.

Signed-off-by: Abhishek GOYAL <abhishek.goyal@valeo.com>
Signed-off-by: Abhishek GOYAL <abhishek.goyal@valeo.com>
@Abhishek2581 Abhishek2581 mentioned this pull request Apr 24, 2026
7 tasks
@4og

4og commented May 4, 2026

Copy link
Copy Markdown
Member

@crimson11, this PR has just been raised in the Architecture Community meeting. Can you please have a look?

@FScholPer

FScholPer commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

@Abhishek2581 Please fix the documentation issue so that we can merge it

@FScholPer FScholPer merged commit ad4c05a into eclipse-score:main Jun 11, 2026
5 of 6 checks passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in S-CORE Roadmap Jun 11, 2026
@LittleHuba

Copy link
Copy Markdown
Contributor

@FScholPer this was not yet agreed in the Communication FT. For the moment I will block work on this feature from our side until this agreement is reached. I'll add a comment here once this is the case.

@LittleHuba

Copy link
Copy Markdown
Contributor

@FScholPer can we please revert and reopen it for the moment to not confuse third parties?

@FScholPer

Copy link
Copy Markdown
Contributor

It was agreed in the last TL circle meeting: https://github.com/orgs/eclipse-score/discussions/516?sort=new#discussioncomment-17262045. And its really old nobody refused. If you see wrong requirements we can open a new pr to change them and what do you mean with "third parties"?

@qor-lb

qor-lb commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

@FScholPer the agreement with @Abhishek2581 in the architecture community is to get the consensus of the communication FT before merging the PR.

I am with @LittleHuba on this - we should revert the PR until that has happened.

@LittleHuba

Copy link
Copy Markdown
Contributor

@Abhishek2581 would you mind recreating the PR please?
If I do it myself, then you will not have a way to integrate changes that we request based on the discussion in the FT and architecture round.
Really sorry for the inconvenience...

@Abhishek2581

Copy link
Copy Markdown
Contributor Author

Hi @LittleHuba :

No worries, and thanks for the heads-up. I've recreated the PR:
#2997
I'll take care of any follow-up changes that come out of the FT and architecture review rounds.

Thanks!

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

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

DDS gateway

5 participants