Skip to content

ci: add provider integration test matrix#415

Open
walterchris wants to merge 2 commits into
mainfrom
openssl/ci-matrix
Open

ci: add provider integration test matrix#415
walterchris wants to merge 2 commits into
mainfrom
openssl/ci-matrix

Conversation

@walterchris

@walterchris walterchris commented May 31, 2026

Copy link
Copy Markdown
Collaborator

Runs the OpenSSL provider integration suites (cli + capi + nginx) across five OpenSSL 3.0.x distributions in parallel. The nginx suite runs only where OpenSSL carries the complete STORE-dispatch fix (openssl#18262) — the 3.0.11/3.0.13 cells — and is skipped on 3.0.2 and the RHEL-family 3.0.7 cells.

Copilot AI review requested due to automatic review settings May 31, 2026 18:39

Copilot AI 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.

Pull request overview

Adds a new GitHub Actions workflow that runs the three OpenSSL provider integration suites (cli, capi, nginx) across six OpenSSL 3.0.x distro containers in parallel, with nginx skipped on distros affected by the upstream STORE-dispatch bug. Each suite becomes its own PR check, expanding cross-distro coverage for the provider beyond the existing CodeQL/Rust workflows that pin a single OpenSSL build.

Changes:

  • New provider-matrix.yml workflow with a six-cell container matrix (Ubuntu 22.04/24.04, Debian 12, AlmaLinux/Rocky/RHEL UBI 9.2) and fail-fast: false.
  • Per-family (debian/rhel) bootstrap steps that install build deps, pin RHEL-family to the 9.2 vault for OpenSSL 3.0.7, and install the Rust toolchain pinned to 1.93.
  • Builds azihsm_api_native and azihsm_ossl_provider with --features mock, installs the native lib into the system loader path, conditionally installs nginx 1.29.0, and invokes each suite via cargo nextest under the ci-provider-integration profile.

Comment thread .github/workflows/provider-matrix.yml Outdated
Comment thread .github/workflows/provider-matrix.yml
Comment thread .github/workflows/provider-matrix.yml
@walterchris walterchris force-pushed the openssl/ci-matrix branch 2 times, most recently from 6808ac5 to d4b7efe Compare June 2, 2026 07:11
Copilot AI review requested due to automatic review settings June 2, 2026 07:11

Copilot AI 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.

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 6 comments.

Comment thread .github/workflows/provider-matrix.yml
Comment thread .github/workflows/provider-matrix.yml Outdated
Comment thread xtask/src/integration_tests.rs Outdated
Comment thread xtask/src/integration_tests.rs
Comment thread xtask/src/custom_openssl.rs Outdated
Comment thread xtask/src/custom_openssl.rs Outdated
Copilot AI review requested due to automatic review settings June 2, 2026 12:19

Copilot AI 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.

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 6 comments.

Comment thread xtask/src/integration_tests.rs
Comment thread plugins/ossl_prov/integration-tests/nginx/src/nginx_tests.rs Outdated
Comment thread .github/workflows/provider-matrix.yml
Comment thread .github/workflows/provider-matrix.yml
Comment thread xtask/src/integration_tests.rs Outdated
Comment thread .github/workflows/provider-matrix.yml Outdated
Comment thread .github/workflows/provider-matrix.yml Outdated
Copilot AI review requested due to automatic review settings June 4, 2026 09:58

Copilot AI 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.

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Comment thread xtask/src/integration_tests.rs
Comment thread .github/workflows/provider-matrix.yml Outdated
Copilot AI review requested due to automatic review settings June 6, 2026 05:22

Copilot AI 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.

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Comment thread xtask/src/integration_tests.rs

Copilot AI 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.

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Comment thread .github/workflows/provider-matrix.yml
Comment thread .github/workflows/provider-matrix.yml Outdated
walterchris and others added 2 commits June 8, 2026 21:48
Run the OpenSSL provider integration suites (cli + capi + nginx) across
six OpenSSL 3.0.x distributions in parallel via GitHub Actions container
jobs. Each cell is its own PR check (fail-fast: false). The nginx suite
is skipped on distros whose OpenSSL predates the upstream STORE-dispatch
fix (openssl#18262).

Signed-off-by: Christian Walter <christian.walter@9elements.com>
Copilot AI review requested due to automatic review settings June 8, 2026 19:48

Copilot AI 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.

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Comment on lines +177 to +184
- name: Install nginx 1.29.0
if: matrix.nginx
run: |
codename=$(. /etc/os-release && echo "$VERSION_CODENAME")
host=$(. /etc/os-release && echo "$ID")
curl -fsSL -o /tmp/nginx.deb \
"https://nginx.org/packages/mainline/${host}/pool/nginx/n/nginx/nginx_1.29.0-1~${codename}_amd64.deb"
apt-get install -y --no-install-recommends /tmp/nginx.deb
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