Skip to content

tools/sev-snp-measure-go: init#2326

Open
charludo wants to merge 2 commits intomainfrom
ch/sev-snp-measure-go
Open

tools/sev-snp-measure-go: init#2326
charludo wants to merge 2 commits intomainfrom
ch/sev-snp-measure-go

Conversation

@charludo
Copy link
Copy Markdown
Collaborator

Run nix run .#base.scripts.sev-snp-measure-tester to see identical output between the old python package and the new go package 🥳

@charludo charludo requested review from burgerdev and sespiros April 17, 2026 09:49
@charludo charludo added the no changelog PRs not listed in the release notes label Apr 17, 2026
@katexochen
Copy link
Copy Markdown
Member

I think keeping the comparison test agaist sev-snp-measure is a good idea.

@charludo
Copy link
Copy Markdown
Collaborator Author

I think keeping the comparison test agaist sev-snp-measure is a good idea.

Hm... Then maybe, have the generator pkg passed into calculateSnpLaunchDigest as a nix arg, default to the new one, and rewrite the tester accordingly?

Would you just like to keep this for manual testing, or would you want it in CI?

@katexochen
Copy link
Copy Markdown
Member

Would you just like to keep this for manual testing, or would you want it in CI?

CI I'd say, I wouldn't keep it for manual testing.

@charludo charludo force-pushed the ch/sev-snp-measure-go branch from aa31176 to 4ea2e5d Compare April 20, 2026 13:19
@charludo charludo force-pushed the ch/sev-snp-measure-go branch from 4ea2e5d to 996d69a Compare April 20, 2026 13:49
@charludo charludo requested a review from katexochen April 20, 2026 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no changelog PRs not listed in the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants