Skip to content

merge execution#214

Merged
raphael-goetz merged 6 commits into
mainfrom
#207-merge-execution
Jun 6, 2026
Merged

merge execution#214
raphael-goetz merged 6 commits into
mainfrom
#207-merge-execution

Conversation

@raphael-goetz

Copy link
Copy Markdown
Member

Resolves: #207

Copilot AI review requested due to automatic review settings June 5, 2026 12:29
@github-actions

github-actions Bot commented Jun 5, 2026

Copy link
Copy Markdown

GitLab Pipeline Action

General information

Link to pipeline: https://gitlab.com/code0-tech/development/taurus/-/pipelines/2581708125

Status: Passed
Duration: 1 minutes

Job summaries

docs:preview

Documentation preview available at https://code0-tech.gitlab.io/-/development/telescopium/-/jobs/14727162884/artifacts/out/index.html

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR merges the previous “test execution” path into the normal execution pipeline and adds end-to-end reporting so Taurus can transmit a completed ExecutionResult (including per-node results) to Aquila after each flow run (dynamic mode).

Changes:

  • Remove the separate test_execution.* NATS request/reply path and standardize on execution.* consumption.
  • Add an Aquila execution gRPC client and send ExecutionResult updates after each execution.
  • Extend taurus-core to produce execution reports containing per-node results (including parameter results), and bump tucana to support the new result shapes.

Reviewed changes

Copilot reviewed 12 out of 13 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
README.md Updates the high-level Taurus description to include execution result transmission.
docs/errors.md Updates T-TAURUS-000001 description to reflect unified execution subject/payload.
docs/dev.md Documents execution-result transmission behavior in dynamic mode.
crates/taurus/src/client/runtime_execution.rs Adds Aquila execution gRPC client for pushing ExecutionResult.
crates/taurus/src/client/mod.rs Exposes the new runtime execution client module.
crates/taurus/src/app/worker.rs Removes test execution subscription and sends execution results to Aquila after runs.
crates/taurus/src/app/mod.rs Wires the new execution client into dynamic-mode startup and the worker.
crates/taurus-manual/src/main.rs Adds CLI option to enqueue an execution on a running Taurus via NATS.
crates/taurus-core/src/runtime/execution/value_store.rs Stores/returns per-node execution results with parameter result support.
crates/taurus-core/src/runtime/engine/executor.rs Captures parameter results and propagates them into stored node results.
crates/taurus-core/src/runtime/engine.rs Introduces EngineExecutionReport and report-returning execution APIs.
Cargo.toml Bumps tucana workspace dependency version.
Cargo.lock Updates lockfile for tucana version bump.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread crates/taurus/src/client/runtime_execution.rs Outdated
Comment thread crates/taurus/src/app/worker.rs
Comment thread crates/taurus-core/src/runtime/engine/executor.rs
Comment thread crates/taurus-core/src/runtime/engine.rs
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Raphael Götz <52959657+raphael-goetz@users.noreply.github.com>
@raphael-goetz raphael-goetz merged commit 204d5be into main Jun 6, 2026
2 checks passed
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.

Merge Test & Normal Execution to push every Execution Result

2 participants