Skip to content

ci: add Fedora conformance workflow - #3086

Merged
SDAChess merged 13 commits into
mainfrom
ci/conformance-fedora
Sep 2, 2026
Merged

ci: add Fedora conformance workflow#3086
SDAChess merged 13 commits into
mainfrom
ci/conformance-fedora

Conversation

@SDAChess

@SDAChess SDAChess commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

Summary

Add an initial Fedora conformance workflow that builds the OpenShell RPMs from the PR commit, installs them in a disposable Fedora guest, and verifies the local gateway with openshell status.

Related Issue

No issue required: this is an initial CI proof of concept for the RFC 014 conformance pipeline.

Changes

  • Add a Conformance workflow for PR mirror branches and manual dispatches
  • Reuse the existing CLI, gateway, and RPM build workflows
  • Boot a disposable Fedora guest with rootless Podman and enforcing SELinux
  • Install the generated openshell and openshell-gateway RPMs
  • Start and register the package-managed gateway
  • Run openshell status
  • Print gateway, Podman, and SELinux diagnostics when the guest check fails

Testing

  • nix run nixpkgs#actionlint -- .github/workflows/conformance.yml
  • git diff --check
  • mise run pre-commit — not run
  • GitHub Actions conformance run — pending PR creation
  • Unit tests are not applicable to this workflow-only change

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)
  • Architecture docs updated (not applicable)

@SDAChess SDAChess self-assigned this Sep 1, 2026
@SDAChess
SDAChess force-pushed the ci/conformance-fedora branch from 713df36 to b2c1ef2 Compare September 1, 2026 13:01
Comment thread .github/workflows/build-rpm.yml Fixed
Comment thread .github/workflows/conformance.yml Outdated
Comment thread .github/workflows/build-rpm.yml Outdated
Comment thread .github/workflows/rpm-package.yml
@SDAChess

SDAChess commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator Author

Closes #2973

@copy-pr-bot

copy-pr-bot Bot commented Sep 2, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@SDAChess

SDAChess commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator Author

/ok-to-test 32a43a6

Signed-off-by: Simon Scatton <sscatton@nvidia.com>
Signed-off-by: Simon Scatton <sscatton@nvidia.com>
Signed-off-by: Simon Scatton <sscatton@nvidia.com>
Signed-off-by: Simon Scatton <sscatton@nvidia.com>
Signed-off-by: Simon Scatton <sscatton@nvidia.com>
Signed-off-by: Simon Scatton <sscatton@nvidia.com>
Signed-off-by: Simon Scatton <sscatton@nvidia.com>
Signed-off-by: Simon Scatton <sscatton@nvidia.com>
Signed-off-by: Simon Scatton <sscatton@nvidia.com>
Signed-off-by: Simon Scatton <sscatton@nvidia.com>
Signed-off-by: Simon Scatton <sscatton@nvidia.com>
@SDAChess
SDAChess force-pushed the ci/conformance-fedora branch from 32a43a6 to 8681479 Compare September 2, 2026 10:34
Signed-off-by: Simon Scatton <sscatton@nvidia.com>
@SDAChess

SDAChess commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator Author

/ok-to-test afc6202

Comment thread .github/workflows/conformance.yml
Comment thread .github/workflows/conformance.yml
Comment thread .github/workflows/conformance.yml
Comment thread .github/workflows/conformance.yml
Comment thread .github/workflows/conformance.yml
Comment thread .github/workflows/conformance.yml
Comment thread .github/workflows/conformance.yml
Comment thread .github/workflows/conformance.yml
Comment thread .github/workflows/conformance.yml
Comment thread .github/workflows/conformance.yml

@elezar elezar left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I have some non-blocking questions / comments that can be addressed as follow-ups.

The other one would be whether it makes sense to rework / clean up the commit history, but since we squash on merge anyway, that should be fine.

@SDAChess
SDAChess enabled auto-merge September 2, 2026 12:59
@SDAChess
SDAChess added this pull request to the merge queue Sep 2, 2026
Merged via the queue into main with commit b92e9bd Sep 2, 2026
63 checks passed
@SDAChess
SDAChess deleted the ci/conformance-fedora branch September 2, 2026 13:19
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