Send more metadata with a run - #5062
Conversation
|
I have enough context. This is a small, focused change: it adds
The only file in scope for review is Security Review ✅
|
The entry described the wire format; what a user needs to know is the names they can read in job code, and that it needs a newer worker.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #5062 +/- ##
=======================================
- Coverage 90.6% 90.6% -0.1%
=======================================
Files 421 421
Lines 20012 20016 +4
=======================================
- Hits 18137 18128 -9
- Misses 1875 1888 +13 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
Hey @josephjclark closing this one - your change is already in. Frank flagged that anything going into 2.18.0 needs to land on So the metadata is on the release branch, together with the bundled worker bumped to 1.29.1. It goes out in 2.18.0-pre2 for Megan to QA. Thanks for turning this around so quickly - Aisha's team have been waiting on it. |
Description
This PR extends the Run object with more metadata about the run.
The metadata will appear in the run under the
metakey. See the supporting Worker PR: OpenFn/kit#1500Validation steps
log(meta)with common. It should be just fine against worker 1.29.0. but themetaobject does not includeworkOrderIdworkOrderId. And some other stuff.AI Usage
Please disclose whether you've used AI anywhere in this PR (it's cool, we just
want to know!):
You can read more details in our
Responsible AI Policy