Skip to content

fix(nvidia): use standalone ratsd-token module - #496

Merged
cowbon merged 1 commit into
veraison:mainfrom
cowbon:ratsd-token
Oct 22, 2026
Merged

cowbon merged 1 commit into
veraison:mainfrom
cowbon:ratsd-token

Conversation

@cowbon

@cowbon cowbon commented Oct 20, 2026 •

Copy link
Copy Markdown
Collaborator

Description

The NVIDIA scheme imports the legacy token library from the ratsd repository. Switch its production and test imports to github.com/veraison/ratsd-token, pinned to veraison/ratsd-token#1 from the standalone repository.

Update CMW to the library's required v0.3.0, normalize the Go directive to 1.26.0, and refresh module checksums. Keep github.com/veraison/ratsd/tokens for SEV-SNP's TSMReport, which is not provided by the standalone token repository.

Validation

  • SEV-SNP tests pass.
  • go test ./store-plugin/nvidia-coserv ./verification/api github.com/veraison/ratsd-token passes; the store plugin has no test files.
  • go mod tidy -diff and git diff --check pass.
  • NVIDIA tests cannot compile locally because the NVIDIA Attestation SDK header nvat.h is missing.

Checklist

  • Code compiles correctly (NVIDIA compilation requires the missing SDK header)
  • Created tests which fail without the change (if possible) (not applicable to this import and dependency migration)
  • All tests passing (targeted checks pass; NVIDIA tests are blocked as described above)
  • Extended the README / documentation, if necessary (not necessary)

@cowbon
cowbon requested a review from a team as a code owner October 20, 2026 17:23

@THS-on THS-on 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.

The

Comment thread go.sum
Import the legacy token library from github.com/veraison/ratsd-token and pin the standalone repository revision. Update CMW to the required v0.3.0 version and refresh module checksums.

Signed-off-by: Ian Chin Wang <ian.chin.wang@oracle.com>
@cowbon
cowbon merged commit 6b502cb into veraison:main Oct 22, 2026
7 checks passed
@cowbon
cowbon deleted the ratsd-token branch October 22, 2026 17:53
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.

2 participants