+
Skip to content

Feature: Persistency KVS - Feature Requirements #1067

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 7 commits into from
May 21, 2025
Merged

Feature: Persistency KVS - Feature Requirements #1067

merged 7 commits into from
May 21, 2025

Conversation

sbachmann-qorix
Copy link
Contributor

Closes #95

@sbachmann-qorix sbachmann-qorix marked this pull request as draft May 9, 2025 13:54
@sbachmann-qorix sbachmann-qorix mentioned this pull request May 9, 2025
Closed
@github-actions GitHub Actions
Copy link

github-actions bot commented May 9, 2025 •
edited
Loading

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run //:license-check

Status: ⚠️ Needs Review

Click to expand output 
[License Check Output]
Extracting Bazel installation...
Starting local Bazel server and connecting to it...
INFO: Invocation ID: 6ab51776-2bd9-415c-b3dc-815ef65d4c00
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 0 packages loaded
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/score/score/BUILD
ERROR: no such target '//:license-check': target 'license-check' not declared in package '' defined by /home/runner/work/score/score/BUILD
INFO: Elapsed time: 5.149s
INFO: 0 processes.
ERROR: Build did NOT complete successfully
ERROR: Build failed. Not running target

@github-actions GitHub Actions
Copy link

github-actions bot commented May 9, 2025

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

@sbachmann-qorix sbachmann-qorix changed the title Feature: Persistency KVS Feature: Persistency KVS - Feature Requirements May 9, 2025
@sbachmann-qorix sbachmann-qorix force-pushed the sbachmann_kvs_feature_req branch from d59695d to d3d0ba1 Compare May 12, 2025 09:59
@vinodreddy-g vinodreddy-g requested a review from guysagnes May 13, 2025 07:35
@sbachmann-qorix sbachmann-qorix marked this pull request as ready for review May 13, 2025 07:42
@sbachmann-qorix sbachmann-qorix force-pushed the sbachmann_kvs_feature_req branch from ae2f808 to 306c995 Compare May 14, 2025 06:58
PandaeDo
PandaeDo reviewed May 14, 2025
View reviewed changes
vbogdanb
vbogdanb reviewed May 15, 2025
View reviewed changes
vbogdanb
vbogdanb reviewed May 15, 2025
View reviewed changes
guysagnes
guysagnes reviewed May 15, 2025
View reviewed changes
@guysagnes guysagnes mentioned this pull request May 19, 2025
Closed
LittleHuba and others added 6 commits May 19, 2025 16:07
@LittleHuba @sbachmann-qorix
persistency: Feature request for KVS
73cd46e 
Initial draft
@johannes-esr @sbachmann-qorix
persistency: Feature request for KVS
b90bc96 
Update of draft document
@sbachmann-qorix
persistency_kvs: refined feature requirements
1ed1024 
Signed-off-by: Sven Bachmann <sven.bachmann.ext@qorix.ai>
@sbachmann-qorix
persistency_kvs: refined req wording to match IREB standards
8766f6a 
International Requirements Engineering Board (IREB)

Signed-off-by: Sven Bachmann <sven.bachmann.ext@qorix.ai>
@sbachmann-qorix
persistency_kvs: removed reqs after review from Vinod
afb4d41 
These requirements are not needed at a feature level.

Signed-off-by: Sven Bachmann <sven.bachmann.ext@qorix.ai>
@sbachmann-qorix
persistency_kvs: run spell checker
528db0b 
Signed-off-by: Sven Bachmann <sven.bachmann.ext@qorix.ai>
@sbachmann-qorix sbachmann-qorix requested a review from Copilot May 19, 2025 14:28
Copilot
Copilot AI reviewed May 19, 2025
View reviewed changes
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

Files not reviewed (4)
  • docs/features/persistency/index.rst: Language not supported
  • docs/features/persistency/key-value-storage/index.rst: Language not supported
  • docs/features/persistency/kvs/index.rst: Language not supported
  • docs/features/persistency/kvs/requirements/index.rst: Language not supported

@sbachmann-qorix
persistency_kvs: integrated FT review from 2025-05-15
0ee8b13 
Signed-off-by: Sven Bachmann <sven.bachmann.ext@qorix.ai>
@sbachmann-qorix sbachmann-qorix force-pushed the sbachmann_kvs_feature_req branch from 306c995 to 0ee8b13 Compare May 19, 2025 14:30
guysagnes
guysagnes approved these changes May 20, 2025
View reviewed changes
Copy link

@guysagnes guysagnes left a comment

Choose a reason for hiding this comment

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

all review finding solved

@vinodreddy-g vinodreddy-g merged commit 4dc266d into eclipse-score:main May 21, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@vinodreddy-g vinodreddy-g vinodreddy-g approved these changes

@qor-lb qor-lb qor-lb approved these changes

@PandaeDo PandaeDo PandaeDo approved these changes

+2 more reviewers

@vbogdanb vbogdanb vbogdanb approved these changes

@guysagnes guysagnes guysagnes approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Feature Request for Persistency

8 participants

@sbachmann-qorix @vbogdanb @guysagnes @vinodreddy-g @qor-lb @PandaeDo @LittleHuba @johannes-esr
点击 这是indexloc提供的php浏览器服务，不要输入任何密码和下载