Skip to content

Conversation

@shjala
Copy link
Member

@shjala shjala commented Nov 6, 2025

Add GitHub Actions workflow to collect TPM measurements from active/updating partitions on amd64 architecture. The results are uploaded as artifacts to be fed later to PCR prediction tool.

@codecov
Copy link

codecov bot commented Nov 6, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 20.39%. Comparing base (2281599) to head (6f23dc1).
⚠️ Report is 113 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5356      +/-   ##
==========================================
+ Coverage   19.52%   20.39%   +0.86%     
==========================================
  Files          19       19              
  Lines        3021     2314     -707     
==========================================
- Hits          590      472     -118     
+ Misses       2310     1721     -589     
  Partials      121      121              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Add GitHub Actions workflow to collect TPM measurements from
active/updating partitions on amd64 architecture. The results
are uploaded as artifacts to be fed later to PCR prediction tool.

Signed-off-by: Shahriyar Jalayeri <[email protected]>
@shjala
Copy link
Member Author

shjala commented Nov 10, 2025

@eriknordmark / @OhmSpectator I don't know how to test this, Nikolay do you know a way to test the github action completly ? I tried act to no avail.

@OhmSpectator
Copy link
Member

@eriknordmark / @OhmSpectator I don't know how to test this, Nikolay do you know a way to test the github action completly ? I tried act to no avail.

We used @rene's EVE repo for that. Fast merge into master, run actions, check. It may require also some extra settings on the repo's side, but in the end it always worth it.

@shjala
Copy link
Member Author

shjala commented Nov 10, 2025

@eriknordmark / @OhmSpectator I don't know how to test this, Nikolay do you know a way to test the github action completly ? I tried act to no avail.

We used @rene's EVE repo for that. Fast merge into master, run actions, check. It may require also some extra settings on the repo's side, but in the end it always worth it.

@rene can you help me with this?

Copy link
Contributor

@eriknordmark eriknordmark left a comment

Choose a reason for hiding this comment

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

LGTM but would be good to test the workflow on Renê's setup.

@shjala
Copy link
Member Author

shjala commented Nov 13, 2025

Since PCR 5 is not part of our sealing policy, I think I can reduce this to only two measurements instead of 4.

@shjala shjala marked this pull request as draft November 13, 2025 08:51
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.

3 participants