add compile-time configuration for kvs maximum storage size - #165
add compile-time configuration for kvs maximum storage size#165atarekra wants to merge 17 commits into
Conversation
License Check Results🚀 The license check job ran with the Bazel command: bazel run --lockfile_mode=error //:license-checkStatus: Click to expand output |
|
The created documentation from the pull request is available at: docu-html |
|
I see the following points : |
PandaeDo
left a comment
There was a problem hiding this comment.
Could you please solve all the build errors?
|
|
I see this PR is up to date, but the change should be discussed. |
|
Discussed during CFT 12.03.2026 - configuration of this parameter is okay, but the parameter should be init-time configurable. |
ec58b4a to
72f9fc0
Compare
There was a problem hiding this comment.
don't change the file modes of cpp/hpp/BUILD files they should stay 100644
|
Thanks for the pull request. This thread has been quiet for 30 days, so we are marking it as stale for now. Please take a quick look and let us know whether it is still up to date, still relevant, needs review, or is ready to merge. Any new activity will remove the stale label automatically. If nothing changes in the next 10 days, we will close it to keep the backlog current. |
|
@arkjedrz Pls review |
PandaeDo
left a comment
There was a problem hiding this comment.
Unblock from my side because I'll be on vacation. Please continue to solve conflicts
|
@arkjedrz Your points have been resolved , Pls review. |
Signed-off-by: atarekra <ahmed.tarek-ramadan@valeo.com>
This pull request introduces support for specifying the maximum size of the Key-Value Storage (KVS) at compile time, fulfilling the related requirement:
"The Key-Value-Storage shall support specification of its maximum size at compile time."
and resolves #161