[DRAFT]: Modeling splitting out API into its own module - #2882
Draft
aryangorwade wants to merge 8 commits into
Draft
aryangorwade wants to merge 8 commits into
aryangorwade wants to merge 8 commits into
Conversation
aryangorwade
force-pushed
the
aryan-api-module-split
branch
from
September 11, 2026 00:55
442274a to
c5b3c4d
Compare
Relocate shared image path logic and NVIDIADriver constants so api/ no longer imports root internal packages ahead of becoming a standalone Go module. Signed-off-by: Aryan Gorwade <agorwade@nvidia.com>
Give API consumers a Go-compatible versioned module while keeping root development builds pinned to the local API source. Signed-off-by: Aryan Gorwade <agorwade@nvidia.com>
Signed-off-by: Aryan Gorwade <agorwade@nvidia.com>
Fail CI when direct dependencies shared by the root and API modules use different versions. Signed-off-by: Aryan Gorwade <agorwade@nvidia.com>
Verify operator and API tags share a commit and that released binaries record the independently downloadable API module version. Signed-off-by: Aryan Gorwade <agorwade@nvidia.com>
Have Dependabot update shared root and API dependencies together while preserving grouped Kubernetes upgrades. Signed-off-by: Aryan Gorwade <agorwade@nvidia.com>
Run the unpublished-tag check for pull requests whose titles identify an operator version bump. Signed-off-by: Aryan Gorwade <agorwade@nvidia.com>
Skip internal pipelines for API-only tags and prevent them from reaching Nspect or NGC publishing jobs. Signed-off-by: Aryan Gorwade <agorwade@nvidia.com>
aryangorwade
force-pushed
the
aryan-api-module-split
branch
from
September 11, 2026 21:26
c5b3c4d to
470d8a1
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
1 participant
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.
Description
This is a PR splitting
/apiinto a standalone module, as described in this design doc, this would serve to fix GPU Operator versioning issues (Go only allowsv0.x.yandv1.x.y).This architectural split involves:
api/into/apito avoid circular importsgo.modandgo.sum; rootgo.modreferences this module nowCI checks introduced:
Changes to CI pipeline:
.nvidia-ci.yamland.common-ci.yamlto ensure that tagging a release commit with operator and api tags does not cause a separate release process to trigger for theapitag in addition to the operator tagDependabot changes:
dependabot.yamlusinggroup-by: dependency-nameand across-directorypattern to update each shared dependency between both modules in one PR (one PR per dependency; for both modules). Note that groups such ask8s.io/*can be created, resulting in one cross-directory PR for all deps underk8s.io/*.Checklist
make lint)make validate-generated-assets)make validate-modules)Testing
None done yet as this is a sample PR.