Skip to content

Conversation

snir911
Copy link
Contributor

@snir911 snir911 commented Jul 6, 2025

No description provided.

@snir911 snir911 requested a review from Copilot July 6, 2025 06:36
@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jul 6, 2025
Copy link

openshift-ci bot commented Jul 6, 2025

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

Copy link
Contributor

Caution

There are some errors in your PipelineRun template.

PipelineRun Error
osc-registry no kind "ImageDigestMirrorSet" is registered for version "config.openshift.io/v1" in scheme "k8s.io/client-go/kubernetes/scheme/register.go:83"

Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR introduces NVIDIA support and a Fedora-based PodVM image variant, adding necessary service definitions, scripts, container build logic, and workflow updates.

  • Add a systemd service and script to generate NVIDIA CDI configuration
  • Extend the Fedora Containerfile with an NVIDIA build target and related setup
  • Update GitHub Actions workflow and documentation to include the NVIDIA variant

Reviewed Changes

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

Show a summary per file
File Description
config/peerpods/podvm/bootc/nvidia/nvidia-cdi.service New systemd unit to kick off NVIDIA CDI generation
config/peerpods/podvm/bootc/nvidia/generate-nvidia-cdi.sh Script to load drivers and produce NVIDIA CDI YAML
config/peerpods/podvm/bootc/Containerfile.fedora Added NVIDIA build stage, driver install, and CDI setup
.github/workflows/bootc-podvm-build.yaml Workflow jobs for NVIDIA container/disk image builds
.github/README.md Documentation for bootc PodVM build workflows
Comments suppressed due to low confidence (4)

.github/workflows/bootc-podvm-build.yaml:116

  • The comparison build_disk != 'false' treats a boolean input as string; use if: github.event.inputs.build_disk instead to correctly gate the disk build.
    if: github.event_name != 'pull_request' && (github.event.inputs.build_disk != 'false') && (always() && !cancelled() && !failure())

.github/workflows/bootc-podvm-build.yaml:80

  • This condition will run the NVIDIA build on every push/PR regardless of provider; restrict it to Azure only by removing the non-workflow_dispatch clause.
    if: github.event.inputs.cloud_provider == 'azure' || github.event_name != 'workflow_dispatch'

.github/README.md:20

  • The README mentions triggers on main, but the workflow file only triggers on devel. Please synchronize the branch list in the docs.
- **Push events** to `main` or `devel` branches when bootc files change

config/peerpods/podvm/bootc/nvidia/generate-nvidia-cdi.sh:10

  • Ensure the target directories (/var/run/cdi and /var/log) exist or create them in the script to avoid write errors.
nvidia-ctk cdi generate --output=/var/run/cdi/nvidia.yaml > /var/log/nvidia-cdi-gen.log 2>&1

snir911 added 3 commits July 20, 2025 14:27
Containerfile with nvidia drivers installtion option.

Signed-off-by: Snir Sheriber <[email protected]>
Signed-off-by: Snir Sheriber <[email protected]>
@snir911 snir911 changed the title WIP: Fedora image Fedora image Jul 21, 2025
@snir911 snir911 marked this pull request as ready for review July 21, 2025 06:30
@openshift-ci openshift-ci bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jul 21, 2025
@openshift-ci openshift-ci bot requested review from bpradipt and wainersm July 21, 2025 06:31
Copy link

openshift-ci bot commented Jul 21, 2025

@snir911: all tests passed!

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

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.

1 participant