Skip to content

Blog: YAML vs Python orchestrators#4529

Open
elliotgunn wants to merge 8 commits intomainfrom
blog/yaml-for-workflow-orchestration
Open

Blog: YAML vs Python orchestrators#4529
elliotgunn wants to merge 8 commits intomainfrom
blog/yaml-for-workflow-orchestration

Conversation

@elliotgunn
Copy link
Copy Markdown
Contributor

@elliotgunn elliotgunn commented Apr 8, 2026

No description provided.

@elliotgunn elliotgunn marked this pull request as draft April 8, 2026 15:46
@elliotgunn elliotgunn requested a review from akeller April 8, 2026 15:46
akeller
akeller reviewed Apr 8, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@akeller akeller left a comment

Choose a reason for hiding this comment

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

No huge concerns here. Just a few nitpicks.

Comment thread src/contents/blogs/2026-04-10-yaml-for-workflow-orchestration/index.md Outdated
Comment thread src/contents/blogs/2026-04-10-yaml-for-workflow-orchestration/index.md Outdated
Comment thread src/contents/blogs/2026-04-10-yaml-for-workflow-orchestration/index.md Outdated
Comment thread src/contents/blogs/2026-04-10-yaml-for-workflow-orchestration/index.md
Comment thread src/contents/blogs/2026-04-10-yaml-for-workflow-orchestration/index.md
Comment thread src/contents/blogs/2026-04-10-yaml-for-workflow-orchestration/index.md
@elliotgunn @akeller
Apply suggestions from code review
0f91777 
Co-authored-by: Amara Graham <akeller@users.noreply.github.com>
@elliotgunn elliotgunn marked this pull request as ready for review April 10, 2026 14:37
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 10, 2026 •
edited
Loading

Visual Changes Detected

Make sure changes are expected and review the Playwright report for details.

Playwright report artifact: https://github.com/kestra-io/docs/actions/runs/24263248219/artifacts/6379649037

To accept the updated visuals, click Accept, select the PR branch, and run the workflow.

Accept

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 10, 2026 •
edited
Loading

☁️ Cloudflare Worker Preview Deployed!

🔗 https://ks-blog-yaml-for-workflow-orchest-docs.kestra-io.workers.dev
🔗 https://1e745ca2-docs.kestra-io.workers.dev

@elliotgunn elliotgunn requested a review from akeller April 10, 2026 14:51
@akeller
Copy link
Copy Markdown
Contributor

akeller commented Apr 10, 2026

Reviewing this now w/ the deployed preview link!

akeller
akeller requested changes Apr 10, 2026
View reviewed changes
Comment thread src/contents/blogs/2026-04-10-yaml-for-workflow-orchestration/index.md Outdated

Asset-centric models tried to fix this from the inside. Instead of "run task A, then task B," you define the data assets (tables, models, datasets) that should exist and let the framework figure out what needs to run to produce them. The dependency graph comes from the assets rather than being manually declared.

### some kind of visual would be cool here
Copy link
Copy Markdown
Contributor

@akeller akeller Apr 10, 2026

Choose a reason for hiding this comment

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

😆 gotta fix this before you merge.

Copy link
Copy Markdown
Contributor Author

@elliotgunn elliotgunn Apr 10, 2026

Choose a reason for hiding this comment

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

UGH i'm sorry!!!

Copy link
Copy Markdown
Contributor

@akeller akeller Apr 10, 2026

Choose a reason for hiding this comment

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

@elliotgunn will you add a visual here or do you just want to remove it to get this merged?

Copy link
Copy Markdown
Contributor Author

@elliotgunn elliotgunn Apr 10, 2026

Choose a reason for hiding this comment

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

I have an idea but it's going to require Claude to create it. waiting for my limit to reset

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 10, 2026

🔦 Lighthouse Benchmark

Last updated: 2026-04-10T20:52:22
Last commit: 3d4089e

Tested: https://ks-blog-yaml-for-workflow-orchest-docs.kestra-io.workers.dev on 2026-04-10 20:52 UTC
Compared against main baseline from 2026-04-10

Scores (0–100, higher is better)

Page Performance Accessibility Best Practices SEO
Home 62 ▲ +5 79 59 92
Pricing 97 88 59 100
Enterprise 93 ▲ +9 78 59 100
Cloud 88 ▼ -3 83 59 100
About Us 81 ▼ -14 87 59 100
Docs Landing 88 84 59 92
Contribute to Kestra (simple docs) 96 84 59 92
Flow (full featured docs) 89 86 59 92
Blog Index 73 ▼ -8 86 59 100
Blog Post (sample) 86 ▼ -11 83 59 100
VS Page (sample) 94 88 59 100
Plugins Landing 93 77 59 92
Plugin Page (sample) 97 87 59 100
Plugin Debug Page (sample) 97 87 59 100
Plugin Debug Return Page (sample) 96 87 59 100
Blueprints Landing 90 77 56 ▼ -3 92
Blueprint Audit Logs CSV Export 70 ▲ +9 82 59 100

Core Web Vitals (lower is better)

Page LCP FCP TBT CLS Speed Index
Home 1.72 s ▲ 1.08 s ▲ 486 ms ▼ 0.000 3.29 s ▲
Pricing 1.07 s ▲ 0.59 s ▲ 103 ms ▼ 0.000 0.93 s
Enterprise 1.64 s ▲ 0.79 s ▲ 52 ms ▼ 0.000 1.06 s ▲
Cloud 2.16 s ▼ 0.55 s ▲ 87 ms ▼ 0.000 1.04 s
About Us 3.45 s ▼ 0.54 s ▲ 78 ms 0.000 0.94 s
Docs Landing 1.68 s 0.64 s ▲ 169 ms ▼ 0.000 1.18 s
Contribute to Kestra (simple docs) 0.94 s ▲ 0.57 s ▲ 137 ms ▼ 0.000 0.80 s ▲
Flow (full featured docs) 1.21 s ▲ 0.57 s ▲ 229 ms ▼ 0.000 1.14 s ▲
Blog Index 8.78 s ▼ 0.55 s ▲ 104 ms ▼ 0.001 0.99 s ▲
Blog Post (sample) 2.49 s ▼ 0.54 s ▲ 90 ms ▼ 0.000 0.76 s ▲
VS Page (sample) 1.57 s 0.59 s ▲ 68 ms ▼ 0.000 0.99 s ▲
Plugins Landing 1.06 s ▲ 0.58 s ▲ 60 ms ▼ 0.000 2.35 s ▼
Plugin Page (sample) 0.98 s ▲ 0.51 s ▲ 32 ms ▼ 0.051 1.57 s
Plugin Debug Page (sample) 0.92 s ▲ 0.48 s ▲ 66 ms 0.001 1.53 s ▼
Plugin Debug Return Page (sample) 1.02 s ▲ 0.52 s ▲ 90 ms ▼ 0.025 1.54 s ▼
Blueprints Landing 1.55 s ▲ 0.83 s ▼ 26 ms ▲ 0.002 2.09 s ▼
Blueprint Audit Logs CSV Export 0.95 s ▲ 0.62 s ▲ 149 ms ▲ 0.485 2.17 s
Legend

▲ improved  ·  ▼ regressed  ·  (blank) no significant change
Score threshold: ±3 pts  ·  Metric threshold: ±10% of baseline

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@akeller akeller akeller requested changes

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@elliotgunn @akeller