8000 process: adjust test levels and methods by pahmann · Pull Request #792 · eclipse-score/score · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

process: adjust test levels and methods #792

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 4 commits into from
Apr 3, 2025
Merged

process: adjust test levels and methods #792

merged 4 commits into from
Apr 3, 2025

Conversation

pahmann
Copy link
Contributor
@pahmann pahmann commented Mar 25, 2025

Implements a finding from the 3rd interim audit.

closes #791

@pahmann
process: adjust test levels and methods
8000
8dfce3b 
Implements a finding from the 3rd interim audit.

closes #791

Signed-off-by: Philipp Ahmann <Philipp.Ahmann@de.bosch.com>
@github-actions GitHub Actions
Copy link
github-actions bot commented Mar 25, 2025 •
edited
Loading

License Check Results

🚀 The license check preparation job ran successfully.

Status: ⚠️ Needs Review

Click to expand output 
2025/04/01 15:28:35 Downloading https://releases.bazel.build/7.4.0/release/bazel-7.4.0-linux-x86_64...
Extracting Bazel installation...
Starting local Bazel server and connecting to it...
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 0 packages loaded
Loading: 0 packages loaded
    currently loading: docs
Loading: 0 packages loaded
    currently loading: docs
Analyzing: target //docs:license.check.python (1 packages loaded, 0 targets configured)
Analyzing: target //docs:license.check.python (1 packages loaded, 0 targets configured)

Analyzing: target //docs:license.check.python (91 packages loaded, 10 targets configured)

Analyzing: target //docs:license.check.python (96 packages loaded, 10 targets configured)

Analyzing: target //docs:license.check.python (98 packages loaded, 10 targets configured)

Analyzing: target //docs:license.check.python (98 packages loaded, 10 targets configured)

Analyzing: target //docs:license.check.python (129 packages loaded, 809 targets configured)

Analyzing: target //docs:license.check.python (139 packages loaded, 2430 targets configured)

Analyzing: target //docs:license.check.python (139 packages loaded, 2430 targets configured)

Analyzing: target //docs:license.check.python (142 packages loaded, 4444 targets configured)

Analyzing: target //docs:license.check.python (147 packages loaded, 4487 targets configured)

Analyzing: target //docs:license.check.python (148 packages loaded, 4607 targets configured)

INFO: Analyzed target //docs:license.check.python (149 packages loaded, 4736 targets configured).
[9 / 13] JavaToolchainCompileClasses external/rules_java~/toolchains/platformclasspath_classes; 0s processwrapper-sandbox
[11 / 13] JavaToolchainCompileBootClasspath external/rules_java~/toolchains/platformclasspath.jar; 0s processwrapper-sandbox
[12 / 13] Building docs/license.check.python.jar (); 0s multiplex-worker
INFO: Found 1 target...
Target //docs:license.check.python up-to-date:
  bazel-bin/docs/license.check.python
  bazel-bin/docs/license.check.python.jar
INFO: Elapsed time: 25.220s, Critical Path: 2.50s
INFO: 13 processes: 9 internal, 3 processwrapper-sandbox, 1 worker.
INFO: Build completed successfully, 13 total actions
INFO: Running command line: bazel-bin/docs/license.check.python docs/formatted.txt -review -project automotive.score -repo https://github.com/eclipse-score/score -token otyhZ4eaRYK1tKLNNF-Y
[main] INFO Querying Eclipse Foundation for license data for 69 items.
[main] INFO Found 45 items.
[main] INFO Querying ClearlyDefined for license data for 25 items.
[main] INFO Found 25 items.
[main] INFO License information could not be automatically verified for the following content:
[main] INFO 
[main] INFO pypi/pypi/-/docutils/0.21.2
[main] INFO 
[main] INFO This content is either not correctly mapped by the system, or requires review.
[main] INFO A review is required for pypi/pypi/-/docutils/0.21.2.
[main] INFO A review request already exists https://gitlab.eclipse.org/eclipsefdn/emo-team/iplab/-/issues/19880 .

@github-actions GitHub Actions
Copy link

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

masc2023
masc2023 requested changes Mar 26, 2025
View reviewed changes
@pahmann
process: fix findings from test level review
289e2a2 
closes #791

Signed-off-by: Philipp Ahmann <Philipp.Ahmann@de.bosch.com>
@pahmann pahmann moved this to In Progress in Process Community Mar 27, 2025
@pahmann pahmann self-assigned this Mar 27, 2025
aschemmel-tech
aschemmel-tech requested changes Mar 28, 2025
View reviewed changes
Copy link
Contributor
@aschemmel-tech aschemmel-tech left a comment

Choose a reason for hiding this comment

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

see inline comments

@pahmann
process: fix findings for test levels
1daae02 
closes #791

Signed-off-by: Philipp Ahmann <Philipp.Ahmann@de.bosch.com>
@pahmann
Copy link
Contributor Author
pahmann commented Apr 1, 2025

@aschemmel-tech and @masc2023 I hope I covered all your findings. Kindly have a re-check.

masc2023
masc2023 previously approved these changes Apr 1, 2025
View reviewed changes
Copy link
Contributor
@masc2023 masc2023 left a comment

Choose a reason for hiding this comment

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

My topics are resolved

@pahmann
Copy link
Contributor Author
pahmann commented Apr 2, 2025

@aschemmel-tech if your findings are also resolved, I would merge this PR.

aschemmel-tech
aschemmel-tech requested changes Apr 2, 2025
View reviewed changes
Copy link
Contributor
@aschemmel-tech aschemmel-tech left a comment

Choose a reason for hiding this comment

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

Old findings resolved, but one small new one found.

docs/platform_management_plan/software_verification.rst Outdated
* - Inspection
- inspection
- 1, -, -
- 1, 2, -
Copy link
Contributor

Choose a reason for hiding this comment

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

Ah, sorry, I missed this. We also want to inspect the stakeholder requirements (level 3) - see also https://eclipse-score.github.io/score/main/process/general_concepts/score_review_concept.html

Copy link
Contributor Author

Choose a reason for hiding this comment

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

resolved.

@pahmann
process: adjust inspection level in verification plan
f6148ca 
closes: #791
Signed-off-by: Philipp Ahmann <Philipp.Ahmann@de.bosch.com>
aschemmel-tech
aschemmel-tech approved these changes Apr 3, 2025
View reviewed changes
Copy link
Contributor
@aschemmel-tech aschemmel-tech left a comment

Choose a reason for hiding this comment

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

fine now

@aschemmel-tech aschemmel-tech merged commit 1203604 into main Apr 3, 2025
9 checks passed
@aschemmel-tech aschemmel-tech deleted the pahmann_verification_test_level_cleanup branch April 3, 2025 06:17
@github-project-automation github-project-automation bot moved this from In Progress to Done in Process Community Apr 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aschemmel-tech aschemmel-tech aschemmel-tech approved these changes

@masc2023 masc2023 masc2023 left review comments

Assignees

@pahmann pahmann

Labels
community:process community:testing
Projects
Process Community
Status: Done
Milestone
2025-05-30 Process & Tools Enable Tes...
Development

Successfully merging this pull request may close these issues.

Bugfix: correct test levels, methods and derivation techniques
3 participants
@pahmann @masc2023 @aschemmel-tech
0