Skip to content

chore: add RELEASE.md to document release process - #2828

Open
cdesiniotis wants to merge 1 commit into
NVIDIA:mainfrom
cdesiniotis:add-releasemd
Open

chore: add RELEASE.md to document release process#2828
cdesiniotis wants to merge 1 commit into
NVIDIA:mainfrom
cdesiniotis:add-releasemd

Conversation

@cdesiniotis

Copy link
Copy Markdown
Contributor

Description

This addresses SIG-REL-01 and SIG-REL-08 from the OSS Scorecard. This takes inspiration from https://github.com/NVIDIA/nvidia-container-toolkit/blob/main/RELEASE.md.

Note, there are some steps, like creation of a version bump PR, creation of a release issue, etc. which the toolkit repo has some scripts for. We could develop similar scripts / GHA workflows in the future for this project as well and evolve the RELEASE.md accordingly.

Checklist

  • No secrets, sensitive information, or unrelated changes
  • Lint checks passing (make lint)
  • Generated assets in-sync (make validate-generated-assets)
  • Go mod artifacts in-sync (make validate-modules)
  • Test cases are added for new code paths

Testing

OSS scorecard was run with and without this change.

Signed-off-by: Christopher Desiniotis <cdesiniotis@nvidia.com>
@cdesiniotis

Copy link
Copy Markdown
Contributor Author

Canceling the GHA's to same some electricity ⚡

@cdesiniotis cdesiniotis self-assigned this Aug 28, 2026
@coderabbitai

coderabbitai Bot commented Aug 28, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

Added RELEASE.md with GPU Operator artifact destinations, calendar versioning details, reference documentation, and a release checklist. The checklist covers version updates, pull requests, tags, image and chart publication, OLM catalog publication, and GitHub release publication.

Merge Risk: 🟡 Moderate · up to 96826

This PR adds release instructions, but the documented Helm chart destinations do not match the publication workflow, which could lead to an incorrect release. Update the destinations before merging; the branch-placeholder issue is a smaller follow-up.


Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🧹 Nitpick comments (1)
RELEASE.md (1)

16-16: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick win

Use an explicit branch placeholder

No in-repository workflow or script renders RELEASE.md. If no external process renders it, readers can copy bump-release-{{ .VERSION }} literally. Replace both occurrences with bump-release-<VERSION> or document the rendering step.


ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: QUIET

Plan: Enterprise

Run ID: 22853f2e-834a-42d4-8707-c50618213bca

📥 Commits

Reviewing files that changed from the base of the PR and between fa49ea6 and 968262c.

📒 Files selected for processing (1)
  • RELEASE.md

Included review availability: Your plan provides up to 12 included reviews per hour; 10 remain after this review.

Comment thread RELEASE.md
Comment on lines +8 to +9
The container and helm chart are published to nvcr.io.
The OLM bundle is published to the [Red Hat certified operators production catalog](https://github.com/redhat-openshift-ecosystem/certified-operators).

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win

Correct the Helm chart publication destinations.

The supplied workflow publishes the chart to oci://ghcr.io/${repository_owner}/gpu-operator/charts, but this document lists nvcr.io and gh-pages. Update the artifact description and checklist to match the workflow. Keep nvcr.io or gh-pages only if separate publication steps exist and are documented.

Also applies to: 22-23

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