Skip to content

Conversation

@hulto
Copy link
Collaborator

@hulto hulto commented Jan 8, 2026

What type of PR is this?

/kind feature

What this PR does / why we need it:

  • Adds a JWT to claim tasks API
  • Agent passes jwt to report and fetch calls
  • Server verifies JWT is not valid
  • For now warns when invalid will error when imixv1 is removed.

Which issue(s) this PR fixes:

Fixes #

@hulto hulto requested a review from KCarretto January 8, 2026 02:21
@github-actions
Copy link
Contributor

github-actions bot commented Jan 8, 2026

Summary

Tests 📝 Passed ✅ Failed ❌ Skipped ⏭️ Other ❓ Flaky 🍂 Duration ⏱️
2960    ±0 2960    ±0 0    ±0 0    ±0 0    ±0 0    ±0 1ms    ±0

Previous Results

Build 🏗️ Result 🧪 Tests 📝 Passed ✅ Failed ❌ Skipped ⏭️ Other ❓ Flaky 🍂 Duration ⏱️
#392 2960 2960 0 0 0 0 30.9s

Insights

Average Tests per Run Total Flaky Tests Total Failed Slowest Test (p95)
2960 0 0 39.9s

Slowest Tests

Test 📝 Results 📊 Duration (avg) ⏱️ Duration (p95) ⏱️
eldritch: random::string_impl::tests::test_string_uniform 3 38.0s 39.9s
eldritch: random::string_impl::tests::test_string_uniform 3 38.0s 39.9s
eldritch: random::string_impl::tests::test_string_uniform 3 38.0s 39.9s
eldritch: process::info_impl::tests::test_info_default 3 3.4s 9.7s
eldritch: process::info_impl::tests::test_info_default 3 3.4s 9.7s
eldritch: process::info_impl::tests::test_info_default 3 3.4s 9.7s
eldritch: pivot::port_scan_impl::tests::test_portscan_return_type_starlark_dict_from_interpreter 3 3.0s 8.9s
eldritch: pivot::port_scan_impl::tests::test_portscan_return_type_starlark_dict_from_interpreter 3 3.0s 8.9s
eldritch: pivot::port_scan_impl::tests::test_portscan_return_type_starlark_dict_from_interpreter 3 3.0s 8.9s
eldritch: pivot::ssh_copy_impl::tests::test_pivot_ssh_copy 3 4.3s 8.8s

🎉 No failed tests in this run. | 🍂 No flaky tests in this run.

Github Test Reporter by CTRF 💚

🔄 This comment has been updated

Copy link
Collaborator

@KCarretto KCarretto left a comment

Choose a reason for hiding this comment

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

Please also remove the committed 83MB tavern binary tavern/tavern

@hulto hulto enabled auto-merge January 10, 2026 00:04
@hulto hulto requested a review from KCarretto January 10, 2026 19:54
Copy link
Collaborator

@KCarretto KCarretto left a comment

Choose a reason for hiding this comment

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

I feel pretty strongly about adding a TaskContext to make this cleaner, otherwise lgtm

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