Skip to content

Conversation

@mlitre
Copy link
Contributor

@mlitre mlitre commented Oct 2, 2025

Describe your changes

This PR adds a first version of an EEBus module that supports the LPC use case. This module interacts with a go library and uses grpc to communicate with this library.
A test framework was also created to validate this module with the grpc calls.

Changes based on @andistorm 's work in #1127.

Requires:

Issue ticket number and link

Checklist before requesting a review

  • I have performed a self-review of my code
  • I have made corresponding changes to the documentation
  • I read the contribution documentation and made sure that my changes meet its requirements

Related

Issues to create after merging

  • If generation is enabled, check if newly generated files differ from the exisiting ones (check for ci)
  • Stabilize Code generator by checking gRPC, protoc and go version in cmake
  • Refactor code generator extensions by making it more independent from grpc source code
  • Allow configuring of eebus service instead of hardcoded example
    • multiple calls in modules/EEBUS/grpc_calls

@mlitre mlitre force-pushed the feature/eebus-rebased branch 5 times, most recently from 3cc37c2 to 45aeeb0 Compare October 10, 2025 16:46
@mlitre mlitre force-pushed the feature/eebus-rebased branch 4 times, most recently from 65c82a2 to eed1e47 Compare October 23, 2025 19:27
@mlitre mlitre marked this pull request as ready for review October 23, 2025 19:28
@mlitre mlitre force-pushed the feature/eebus-rebased branch 2 times, most recently from 0ba03b9 to ec92977 Compare November 6, 2025 15:39
@mlitre mlitre force-pushed the feature/eebus-rebased branch from 40424e7 to 76c9bd6 Compare November 19, 2025 10:50
@mlitre mlitre requested a review from djchhp as a code owner November 19, 2025 10:50
@felix-ulonska
Copy link

⚠️ PR Affected by Security Incident

This PR was affected by a security incident where a compromised account force-pushed to this branch and closed the PR.

✅ Branch History Restored

We've run an automation that restored the complete history of this branch to the time before the attack. All your commits are preserved. Nothing has been updated in the respective branch in case someone force-pushed after the attack but before our automation.

🔄 Action Required

GitHub won't allow reopening this PR after the close and force-push. Please:

  • Open a new PR from your branch to the target branch. You can use the following link to open a new PR: Open PR
  • Link back to this PR in the description to include previous discussions

We apologize for the inconvenience.

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.

5 participants