Skip to content

Feature Request for DDS Gateway#2997

Open
Abhishek2581 wants to merge 1 commit into
eclipse-score:mainfrom
Valeo-S-CORE-Organization:DDS_GW_FR-Valeo
Open

Feature Request for DDS Gateway#2997
Abhishek2581 wants to merge 1 commit into
eclipse-score:mainfrom
Valeo-S-CORE-Organization:DDS_GW_FR-Valeo

Conversation

@Abhishek2581

Copy link
Copy Markdown
Contributor

DDS Gateway: Recreated PR following accidental merge/closure of the previous PR. No functional changes beyond the requested modifications.

@github-actions

Copy link
Copy Markdown

The created documentation from the pull request is available at: docu-html

@LittleHuba LittleHuba left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Blocking this PR formally until agreement is achieved in Communication FT.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I would like to place the same requests as we agreed on with the SOME/IP gateway.

  1. How a translation from intra-ecu to inter-ecu communication is integrated in the system is very much up to the integrator.
  2. The DDS gateway implementation that S-CORE provides is only a showcase on how this problem can be solved. Users must be able to implement/integrate a different DDS translation concept (e.g. one without explicit gateway process).
  3. The DDS logic must be a binding beneath mw::com (e.g. the gateway internally uses mw::com to interact with the DDS stack) This is the foundation of supporting different integration concepts.

Comment on lines +107 to +110
- End-to-End (E2E) protection:

- Centralized handling of Counter, CRC, and DataID
- Validation and protection configurable per route

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

E2E protection handling might be impossible to achieve in a centralized handling. In some of our supported operating systems the network stack is not qualified.
The SOME/IP gateway works around this with a split of the "gateway process" into a QM and an ASIL-B part.


- Configurable routing:

- Mapping between ``mw::com`` events(TBD for fields and methods) and DDS topics

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'd be interested in more detail on this mapping.
What are the supported mapping cases? What are the corner cases where support is not provided?
It is hard to judge feasibility without further details.

- Configurable routing:

- Mapping between ``mw::com`` events(TBD for fields and methods) and DDS topics
- Support for DDS domain-based routing

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Same here, please add more details / link to references.
I'm not a DDS expert so I need some more background to judge.


- Dynamic Type handling:

- Runtime type definition via configuration or via dynamic library

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This sounds like we have a huge overlap with the SOME/IP efforts. The format might differ, but the general task is the same. I'd like to see an investigation on how much we can reuse from the upcoming SOME/IP infrastructure.


- DDS stack abstraction:

- Pluggable DDS implementations via defined interfaces

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Ideally this stack abstraction is mw::com (as highlighted in a previous comment)

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

Projects

Status: Backlog
Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants