crypto: refactor integration tests for QNX ITF execution - #166
Open
OliverHeilwagen wants to merge 4 commits into
Open
crypto: refactor integration tests for QNX ITF execution#166OliverHeilwagen wants to merge 4 commits into
OliverHeilwagen wants to merge 4 commits into
Conversation
OliverHeilwagen
commented
Aug 14, 2026
Contributor
- Add the shared ITF integration-test macro and process-runner utility.
- Rework integration tests and C++ examples for Docker and QEMU execution.
- Add the x86_64 QNX QEMU environment, IFS configuration, and required tools.
- Update Bazel, module, devcontainer, CI, and README configuration for QNX ITF.
- Use an atomic in-use flag for concurrent PKCS#11 session-key handling.
OliverHeilwagen
requested a deployment
to
workflow-approval
August 14, 2026 13:39 — with
GitHub Actions
Waiting
OliverHeilwagen
requested a deployment
to
workflow-approval
August 14, 2026 13:39 — with
GitHub Actions
Waiting
Contributor
License Check Results🚀 The license check job ran with the Bazel command: bazel run --lockfile_mode=error //:license-checkStatus: Click to expand output |
Contributor
|
The created documentation from the pull request is available at: docu-html |
ChansAlive
reviewed
Aug 14, 2026
ChansAlive
left a comment
Contributor
There was a problem hiding this comment.
Initial quick review comments
OliverHeilwagen
commented
Aug 14, 2026
OliverHeilwagen
commented
Aug 14, 2026
OliverHeilwagen
commented
Aug 14, 2026
OliverHeilwagen
commented
Aug 14, 2026
OliverHeilwagen
requested a deployment
to
workflow-approval
August 19, 2026 16:22 — with
GitHub Actions
Waiting
OliverHeilwagen
requested a deployment
to
workflow-approval
August 19, 2026 16:22 — with
GitHub Actions
Waiting
OliverHeilwagen
force-pushed
the
refactor_integration_test
branch
from
August 20, 2026 07:06
7273bb8 to
90fb030
Compare
OliverHeilwagen
requested a deployment
to
workflow-approval
August 20, 2026 07:06 — with
GitHub Actions
Waiting
OliverHeilwagen
requested a deployment
to
workflow-approval
August 20, 2026 07:06 — with
GitHub Actions
Waiting
OliverHeilwagen
commented
Aug 20, 2026
| - id: generate-compile-commands | ||
| name: bazel-compile-commands (for clang-tidy) | ||
| entry: bazel-compile-commands | ||
| entry: bazel-compile-commands --compiler /usr/bin/clang -R "-fno-canonical-system-headers=" //... |
Contributor
Author
There was a problem hiding this comment.
With the default created bazel-compile-commands external headers are not found by clang-tidy.
Note:
clang-tidy warnings slightly differ to clangd vscode language server especially in [misc-include-cleaner,-warnings-as-errors]
OliverHeilwagen
requested a deployment
to
workflow-approval
August 24, 2026 12:16 — with
GitHub Actions
Waiting
OliverHeilwagen
requested a deployment
to
workflow-approval
August 24, 2026 12:16 — with
GitHub Actions
Waiting
OliverHeilwagen
force-pushed
the
refactor_integration_test
branch
from
August 26, 2026 05:49
cbf54dc to
81b6a0a
Compare
OliverHeilwagen
requested a deployment
to
workflow-approval
August 26, 2026 05:49 — with
GitHub Actions
Waiting
OliverHeilwagen
requested a deployment
to
workflow-approval
August 26, 2026 05:49 — with
GitHub Actions
Waiting
Contributor
|
Documentation preview for this pull request is available at: |
- Add the shared ITF integration-test macro and process-runner utility. - Rework integration tests and C++ examples for Docker and QEMU execution. - Add the x86_64 QNX QEMU environment, IFS configuration, and required tools. - Update Bazel, module, devcontainer, CI, and README configuration for QNX ITF. - Use an atomic in-use flag for concurrent PKCS#11 session-key handling.
- Update ResolvedKey handling with RAII-based PKCS#11 key ownership and contention management. - Fix clang-tidy warnings across backend, configuration, and integration-test code. - Add dedicated Linux test matrix for default, PKCS#11, and OpenSSL backends, including distinct test-result uploads. - Simplify integration-test process execution and support target-specific deployment paths.
- Add setup via �[1mCreated '/workspaces/inc_security_crypto/.venv_test', to activate run:�(B�[m source /workspaces/inc_security_crypto/.venv_test/bin/activate. - Fix Python typing and linter warnings.
OliverHeilwagen
force-pushed
the
refactor_integration_test
branch
from
August 26, 2026 13:29
81b6a0a to
a2005d4
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.