Skip to content

Rework of component requirements related to snapshots#337

Open
tizava wants to merge 1 commit into
eclipse-score:mainfrom
etas-contrib:snapshot-comp-req-rework
Open

Rework of component requirements related to snapshots#337
tizava wants to merge 1 commit into
eclipse-score:mainfrom
etas-contrib:snapshot-comp-req-rework

Conversation

@tizava

@tizava tizava commented Jul 10, 2026

Copy link
Copy Markdown
Contributor
  • Snapshots are now explicit (Driven by the users)
  • Snapshots are created from live data (Not from data flushed to disk)
  • New API to acquire status of the snapshot slot (Occupied/Free)
  • General rework and cleanup

@github-actions

Copy link
Copy Markdown

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.4.2) and connecting to it...
INFO: Invocation ID: 3f0a8cde-0cbf-4958-87a4-3bf31aa72f8a
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 3 packages loaded
Loading: 3 packages loaded
    currently loading: 
Loading: 3 packages loaded
    currently loading: 
Loading: 3 packages loaded
    currently loading: 
Analyzing: target //:license-check (4 packages loaded, 0 targets configured)
Analyzing: target //:license-check (4 packages loaded, 0 targets configured)

Analyzing: target //:license-check (26 packages loaded, 10 targets configured)

Analyzing: target //:license-check (86 packages loaded, 10 targets configured)

Analyzing: target //:license-check (137 packages loaded, 2680 targets configured)

Analyzing: target //:license-check (150 packages loaded, 5325 targets configured)

Analyzing: target //:license-check (155 packages loaded, 5374 targets configured)

Analyzing: target //:license-check (155 packages loaded, 5374 targets configured)

Analyzing: target //:license-check (155 packages loaded, 5374 targets configured)

Analyzing: target //:license-check (158 packages loaded, 7261 targets configured)

Analyzing: target //:license-check (160 packages loaded, 7603 targets configured)

Analyzing: target //:license-check (161 packages loaded, 10015 targets configured)

Analyzing: target //:license-check (162 packages loaded, 10023 targets configured)

Analyzing: target //:license-check (162 packages loaded, 10023 targets configured)

Analyzing: target //:license-check (162 packages loaded, 10023 targets configured)

Analyzing: target //:license-check (162 packages loaded, 10023 targets configured)

INFO: Analyzed target //:license-check (163 packages loaded, 10149 targets configured).
[6 / 16] checking cached actions
INFO: From Generating Dash formatted dependency file ...:
INFO: Successfully converted 66 packages from Cargo.lock to bazel-out/k8-fastbuild/bin/formatted.txt
[15 / 17] [Prepa] JavaToolchainCompileBootClasspath external/rules_java+/toolchains/platformclasspath.jar
INFO: Found 1 target...
Target //:license.check.license_check up-to-date:
  bazel-bin/license.check.license_check
  bazel-bin/license.check.license_check.jar
INFO: Elapsed time: 30.461s, Critical Path: 2.48s
INFO: 17 processes: 12 internal, 4 processwrapper-sandbox, 1 worker.
INFO: Build completed successfully, 17 total actions
INFO: Running command line: bazel-bin/license.check.license_check ./formatted.txt <args omitted>
usage: org.eclipse.dash.licenses.cli.Main [-batch <int>] [-cd <url>]
       [-confidence <int>] [-ef <url>] [-excludeSources <sources>] [-help] [-lic
       <url>] [-project <shortname>] [-repo <url>] [-review] [-summary <file>]
       [-timeout <seconds>] [-token <token>]

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

Please have a look again on the requirements. It seems to me that they are not fully atomic. You can use https://eclipse-score.github.io/process_description//main/process_areas/requirements_engineering/guidance/requirements_templates.html#gd_temp__req_formulation and also the question if the requirement is testable. Understandable they are.

:belongs_to: comp__persistency_snapshot

The component shall maintain a configurable maximum number of snapshots.
The component shall provide APIs to create, restore, and delete snapshots.

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.

Please specify amount of APIs. Requirement is not testable without this information.

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.

Sorry but maybe I don't understand your question. The APIs are three, the ones specified in the description of the requirement: create, restore and delete.

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 is was I meant. How should I know that these are three? I think I would need the information that there are 3 APIs. One for create....

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