Skip to content

Added POCs and Design Decision for IPC selection - #169

Merged
ChansAlive merged 11 commits into
eclipse-score:mainfrom
etas-contrib:feature/lola_poc
Oct 15, 2026
Merged

ChansAlive merged 11 commits into
eclipse-score:mainfrom
etas-contrib:feature/lola_poc

Conversation

@se-lo

@se-lo se-lo commented Aug 17, 2026 •

Copy link
Copy Markdown
Contributor

Added POCs and Design Decision for IPC selection:

  • GRPC
  • mw::com full SOA abstraction Async
  • mw::com message_passing abstraction (Async, With and Without Reply, Engine per Connection, per Client Thread)

@se-lo
se-lo had a problem deploying to workflow-approval August 17, 2026 11:23 — with GitHub Actions Failure
@se-lo
se-lo had a problem deploying to workflow-approval August 17, 2026 11:23 — with GitHub Actions Failure
@se-lo
se-lo requested a review from OliverHeilwagen August 17, 2026 11:23
@github-actions

github-actions Bot commented Aug 17, 2026 •

Copy link
Copy Markdown
Contributor

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run --lockfile_mode=error //:license-check

Status: ⚠️ Needs Review

Click to expand output
[License Check Output]
Extracting Bazel installation...
Starting local Bazel server (8.6.0) and connecting to it...
INFO: Invocation ID: e421bde0-a0e7-4595-9138-5795045bbe07
Computing main repo mapping: 
Computing main repo mapping: 
DEBUG: Rule 'abseil-cpp+' indicated that a canonical reproducible form can be obtained by modifying arguments integrity = "sha256-m3oGQwXp/ZTRJP+mzDWFkutCtdpYj7TgfQklSqQAhts="
DEBUG: Repository abseil-cpp+ instantiated at:
  <builtin>: in <toplevel>
Repository rule http_archive defined at:
  /home/runner/.bazel/external/bazel_tools/tools/build_defs/repo/http.bzl:431:31: in <toplevel>
Computing main repo mapping: 
Computing main repo mapping: 
DEBUG: Rule 'protobuf+' indicated that a canonical reproducible form can be obtained by modifying arguments integrity = "sha256-w6Cp7OiTLjHDtzbi2xixxC5wcM2biBOIsm0BqnHiTKI="
DEBUG: Repository protobuf+ instantiated at:
  <builtin>: in <toplevel>
Repository rule http_archive defined at:
  /home/runner/.bazel/external/bazel_tools/tools/build_defs/repo/http.bzl:431:31: in <toplevel>
DEBUG: Rule 'grpc+' indicated that a canonical reproducible form can be obtained by modifying arguments integrity = "sha256-CvN7gAlTEwtHwHW1ZoPuYL3D7aPDf8YAQZP1tWl1ggQ="
DEBUG: Repository grpc+ instantiated at:
  <builtin>: in <toplevel>
Repository rule http_archive defined at:
  /home/runner/.bazel/external/bazel_tools/tools/build_defs/repo/http.bzl:431:31: in <toplevel>
WARNING: For repository 'score_bazel_platforms', the root module requires module version score_bazel_platforms@0.1.2, but got score_bazel_platforms@1.1.0 in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
WARNING: For repository 'flatbuffers', the root module requires module version flatbuffers@25.12.19, but got flatbuffers@25.12.19.bcr.1 in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
WARNING: For repository 'score_tooling', the root module requires module version score_tooling@2.2.1, but got score_tooling@2.2.2 in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
WARNING: For repository 'score_process_description', the root module requires module version score_process_description@2.1.1, but got score_process_description@2.1.2 in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 1 packages loaded
Loading: 1 packages loaded
    currently loading: 
Loading: 1 packages loaded
    currently loading: 
Loading: 1 packages loaded
    currently loading: 
WARNING: Target pattern parsing failed.
ERROR: Skipping '//:license-check': no such target '//:license-check': target 'license-check' not declared in package '' defined by /home/runner/work/inc_security_crypto/inc_security_crypto/BUILD
ERROR: no such target '//:license-check': target 'license-check' not declared in package '' defined by /home/runner/work/inc_security_crypto/inc_security_crypto/BUILD
INFO: Elapsed time: 12.615s
INFO: 0 processes.
ERROR: Build did NOT complete successfully
ERROR: Build failed. Not running target

@se-lo
se-lo had a problem deploying to workflow-approval August 17, 2026 12:17 — with GitHub Actions Failure
@se-lo
se-lo had a problem deploying to workflow-approval August 17, 2026 12:17 — with GitHub Actions Failure
@github-actions

Copy link
Copy Markdown
Contributor

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

@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Documentation preview for this pull request is available at:
pr-169: https://eclipse-score.github.io/inc_security_crypto/pr-169/

@se-lo

se-lo commented Sep 9, 2026

Copy link
Copy Markdown
Contributor Author

Needs to be rebased once #204 is merged. There seems to be a issue with score_baselibs in the current version surfaced once we also introduce score_communication as dependency.

Comment thread score/crypto/src/ipc/docs/index.rst Outdated
Comment thread score/crypto/src/ipc/docs/index.rst Outdated
Comment thread score/crypto/src/ipc/docs/ipc_comparison.md Outdated
Comment thread score/crypto/src/ipc/docs/ipc_comparison.md Outdated
Comment thread score/crypto/src/ipc/docs/ipc_comparison.md Outdated
Comment thread score/crypto/src/ipc/docs/ipc_comparison.md Outdated
Comment thread score/tests/ipc_poc/BUILD Outdated
Comment thread score/tests/ipc_poc/message_passing_overview.md Outdated
Comment thread score/tests/ipc_poc/poc_low_level_no_reply.cpp Outdated
@se-lo
se-lo requested a deployment to workflow-approval October 13, 2026 15:44 — with GitHub Actions Waiting
@se-lo
se-lo requested a deployment to workflow-approval October 13, 2026 15:44 — with GitHub Actions Waiting
Comment thread score/crypto/src/ipc/message_passing_adapter/docs/requirements/requirements.rst Outdated
@se-lo
se-lo requested a deployment to workflow-approval October 14, 2026 12:10 — with GitHub Actions Waiting
@se-lo
se-lo requested a deployment to workflow-approval October 14, 2026 12:10 — with GitHub Actions Waiting
@se-lo
se-lo requested a deployment to workflow-approval October 14, 2026 12:10 — with GitHub Actions Waiting
@se-lo
se-lo requested a deployment to workflow-approval October 14, 2026 12:19 — with GitHub Actions Waiting
@se-lo
se-lo requested a deployment to workflow-approval October 14, 2026 12:19 — with GitHub Actions Waiting
@se-lo
se-lo requested a deployment to workflow-approval October 14, 2026 12:19 — with GitHub Actions Waiting

@OliverHeilwagen OliverHeilwagen 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.

Only minor corrections required.

Comment thread score/crypto/src/ipc/message_passing_adapter/docs/requirements/index.rst Outdated
Comment thread score/crypto/src/ipc/message_passing_adapter/docs/requirements/requirements.rst Outdated
Comment thread score/crypto/src/ipc/message_passing_adapter/docs/requirements/requirements.rst Outdated
Comment thread score/crypto/src/ipc/message_passing_adapter/docs/requirements/requirements.rst Outdated
Comment thread score/tests/ipc_poc/message_passing_overview.md Outdated
@se-lo
se-lo deployed to workflow-approval October 15, 2026 13:13 — with GitHub Actions Active
@se-lo
se-lo deployed to workflow-approval October 15, 2026 13:13 — with GitHub Actions Active
@se-lo
se-lo deployed to workflow-approval October 15, 2026 13:14 — with GitHub Actions Active
@ChansAlive
ChansAlive merged commit f830c81 into eclipse-score:main Oct 15, 2026
16 checks passed

This branch was successfully deployed

1 active deployment
workflow-approval — 34999fb6 Deployed Oct 15, 2026 by se-lo via Build and Test x86_64-qnx / approval #586
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