NVIDIA / NVIDIA/nvcf

Extend dynamic authentication discovery across nv-boot registry clients

Open
#558 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Go
Stars
218
Forks
72
Avg merge
1d 12h
Merged PRs (30d)
427

Description

Description

Most registry clients in nv-boot-starter-registries authenticate against fixed,
per-registry token endpoints with duplicated per-client auth logic. This issue tracks
future improvements to generalize dynamic authentication discovery across the module.
Implementation details are TBD and will be refined per client before each piece of work
starts.

Planned scope, in rough order:

  1. Move the NGC artifact registry client off its legacy path and onto the OCI distribution
    spec flow (TBD; needs more information from NGC on the supported artifact API).
  2. Refactor the oci package to provide dynamic authentication discovery as the shared
    implementation.
  3. Refactor the ngc, acr, artifactory, docker, and harbor clients to use the new oci
    package.
  4. ecr and volcengine clients: TBD

Definition of Done

  • Challenge discovery lives in the oci package and is reused by clients instead of
    being duplicated per client.
  • The NGC artifact registry client no longer depends on the legacy endpoint.
  • Existing registry suites keep passing; migrated behavior is covered by tests.

Resources

  • Relates to #556

By submitting this issue, you acknowledge that you are an assigned member of the NVCF development team and agree to follow our code of conduct and our contributing guidelines.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reviewing the oci, ngc, acr, artifactory, docker, harbor, ecr, and volcengine packages, along with related issue #556. Clarify the per-client migration scope and the supported NGC artifact API before implementation. Done means challenge discovery is shared through oci, the legacy NGC endpoint is removed, and registry suites plus migration tests pass.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
authentication, backend
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Quiet
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.