Skip to content

[CRE-5839] Add workflow execution meta - #480

Merged
chudilka1 merged 1 commit into
capabilities-developmentfrom
CRE-5839-execution-metadata
Nov 3, 2026
Merged

chudilka1 merged 1 commit into
capabilities-developmentfrom
CRE-5839-execution-metadata

Conversation

@chudilka1

@chudilka1 chudilka1 commented Nov 2, 2026 •

Copy link
Copy Markdown

Description:
Find details here: CRE-5839

There is an ask that a workflow must read an execution state before deciding whether to generate and publish its final report. Workflows need workflowID, executionID, workflowOwner, donID, and the DON's F setting at execution time (e.g., LLR's receiver/router keys route config and throttle state by workflowId). Passing these via workflow config is impossible: the config hash feeds into the workflow ID (circular dependency). Today's workaround generates an empty report just to parse the 109-byte report metadata header — wasting a consensus round. So this proto is necessary for CRE SDK to let a workflow fetch the necessary data provided by the engine at runtime.

Changes

  • Add WorkflowExecutionMetadata with the workflow ID, execution ID, workflow owner, DON ID, and DON F.
  • Add WorkflowExecutionMetadata execution_metadata to ExecuteRequest.

Tests: Proto unit tests, Buf lint, and protobuf compatibility checks pass.

@chudilka1
chudilka1 requested review from a team as code owners November 2, 2026 17:14
@changeset-bot

changeset-bot Bot commented Nov 2, 2026 •

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: f725fea

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@chudilka1 chudilka1 closed this Nov 2, 2026
@chudilka1 chudilka1 reopened this Nov 2, 2026
@bolekk

bolekk commented Nov 2, 2026

Copy link
Copy Markdown
Contributor

@chudilka1 what is this PR for? I don't understand the description. Is there a ticket or a doc?

@github-actions

github-actions Bot commented Nov 2, 2026 •

Copy link
Copy Markdown
Contributor

The latest Buf updates on your PR. Results from workflow Regenerate Protobuf Files / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed⏩ skippedNov 2, 2026, 6:23 PM

@chudilka1
chudilka1 changed the base branch from main to capabilities-development November 2, 2026 17:43
@chudilka1
chudilka1 requested review from a team as code owners November 2, 2026 17:43
@chudilka1
chudilka1 force-pushed the CRE-5839-execution-metadata branch 2 times, most recently from e0fefd8 to dad3bdf Compare November 2, 2026 18:09
@chudilka1
chudilka1 force-pushed the CRE-5839-execution-metadata branch from dad3bdf to f725fea Compare November 2, 2026 18:22
@chudilka1
chudilka1 merged commit aa8c64a into capabilities-development Nov 3, 2026
11 checks passed
@chudilka1
chudilka1 deleted the CRE-5839-execution-metadata branch November 3, 2026 15:17
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.

4 participants