Skip to content

fix(kvs): make unit tests pass on QNX targets - #586

Merged
umaucher merged 1 commit into
mainfrom
arkjedrz_tooling-08-fixes
Oct 26, 2026
Merged

umaucher merged 1 commit into
mainfrom
arkjedrz_tooling-08-fixes

Conversation

@arkjedrz

Copy link
Copy Markdown
Contributor
  • Rust: add test-only tempdir helper. On QNX it creates the directory in the cwd. QNX /tmp can't hold directories.
  • C++: skip flush_failure_rotate_snapshots when run as root. Root ignores file permissions, so flush succeeds.
  • C++: use ASSERT_FALSE so the test can't abort the binary.

@github-actions

github-actions Bot commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor

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]
Starting local Bazel server (8.7.0) and connecting to it...
INFO: Invocation ID: 16c3609f-71bb-42df-b5ed-3e0fa493143a
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 2 packages loaded
Loading: 2 packages loaded
    currently loading: 
Loading: 2 packages loaded
    currently loading: 
Loading: 2 packages loaded
    currently loading: 
Loading: 2 packages loaded
    currently loading: 
Loading: 2 packages loaded
    currently loading: 
Loading: 2 packages loaded
    currently loading: 
Analyzing: target //:license-check (3 packages loaded, 0 targets configured)
Analyzing: target //:license-check (3 packages loaded, 0 targets configured)

Analyzing: target //:license-check (45 packages loaded, 14 targets configured)

Analyzing: target //:license-check (99 packages loaded, 14 targets configured)

Analyzing: target //:license-check (100 packages loaded, 14 targets configured)

Analyzing: target //:license-check (100 packages loaded, 14 targets configured)

Analyzing: target //:license-check (149 packages loaded, 3357 targets configured)

Analyzing: target //:license-check (199 packages loaded, 6007 targets configured)

Analyzing: target //:license-check (200 packages loaded, 9018 targets configured)

Analyzing: target //:license-check (206 packages loaded, 11382 targets configured)

Analyzing: target //:license-check (206 packages loaded, 11382 targets configured)

Analyzing: target //:license-check (209 packages loaded, 15886 targets configured)

INFO: Analyzed target //:license-check (213 packages loaded, 17903 targets configured).
[11 / 16] [Prepa] JavaToolchainCompileClasses external/rules_java+/toolchains/platformclasspath_classes ... (2 actions, 0 running)
INFO: From Generating Dash formatted dependency file ...:
INFO: Successfully converted 50 packages from Cargo.lock to bazel-out/k8-fastbuild/bin/tools/formatted.txt
[14 / 16] JavaToolchainCompileBootClasspath external/rules_java+/toolchains/platformclasspath.jar; 0s disk-cache, processwrapper-sandbox
[15 / 16] Building tools/license.check.license_check.jar (); 0s disk-cache, multiplex-worker
INFO: Found 1 target...
Target //tools:license.check.license_check up-to-date:
  bazel-bin/tools/license.check.license_check
  bazel-bin/tools/license.check.license_check.jar
INFO: Elapsed time: 51.233s, Critical Path: 2.77s
INFO: 16 processes: 12 internal, 3 processwrapper-sandbox, 1 worker.
INFO: Build completed successfully, 16 total actions
INFO: Running command line: bazel-bin/tools/license.check.license_check tools/formatted.txt <args omitted>
[main] INFO Querying Eclipse Foundation for license data for 42 items.
[main] INFO Found 32 items.
[main] INFO Querying ClearlyDefined for license data for 10 items.
[main] INFO Found 10 items.
[main] INFO License information could not be automatically verified for the following content:
[main] INFO 
[main] INFO crate/cratesio/-/lazy_static/1.5.1
[main] INFO 
[main] INFO This content is either not correctly mapped by the system, or requires review.

@github-actions

Copy link
Copy Markdown
Contributor

Documentation preview for this pull request is available at:
pr-586: https://eclipse-score.github.io/persistency/pr-586/

- Rust: add test-only tempdir helper.
  On QNX it creates the directory in the cwd.
  QNX /tmp can't hold directories.
- C++: skip flush_failure_rotate_snapshots when run as root.
  Root ignores file permissions, so flush succeeds.
- C++: use ASSERT_FALSE so the test can't abort the binary.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@arkjedrz
arkjedrz force-pushed the arkjedrz_tooling-08-fixes branch from 4b50523 to 239effe Compare October 26, 2026 14:48
@umaucher
umaucher added this pull request to the merge queue Oct 26, 2026
Merged via the queue into main with commit 235ee24 Oct 26, 2026
19 checks passed
@umaucher
umaucher deleted the arkjedrz_tooling-08-fixes branch October 26, 2026 15:45
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.

2 participants