Skip to content

Conversation

@hiddeco
Copy link
Contributor

@hiddeco hiddeco commented Dec 11, 2025

⚠️ Not extensively tested (yet).

Return errors from the orchestrator through the engine to the controller to trigger a progressive backoff. The engine now passes through the orchestrator results unchanged. The controller then preserves the Running status when an error is returned and returns it to trigger the backoff.

Return errors from the orchestrator through the engine to the
controller to trigger a progressive backoff. The engine now passes
through orchestrator results unchanged. The controller then preserves
the `Running` status when an error is returned and returns it to
trigger the backoff.

Signed-off-by: Hidde Beydals <[email protected]>
@hiddeco hiddeco requested a review from a team as a code owner December 11, 2025 23:03
@hiddeco hiddeco added kind/bug Something isn't working as intended; If unsure that something IS a bug, start a discussion instead area/controller Affects the (main) controller labels Dec 11, 2025
@netlify
Copy link

netlify bot commented Dec 11, 2025

Deploy Preview for docs-kargo-io ready!

Name Link
🔨 Latest commit 2b30547
🔍 Latest deploy log https://app.netlify.com/projects/docs-kargo-io/deploys/693b4dd2eb0d0a00080b40ba
😎 Deploy Preview https://deploy-preview-5491.docs.kargo.io
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@hiddeco hiddeco self-assigned this Dec 11, 2025
@codecov
Copy link

codecov bot commented Dec 11, 2025

Codecov Report

❌ Patch coverage is 71.42857% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 55.35%. Comparing base (41252b5) to head (2b30547).

Files with missing lines Patch % Lines
pkg/controller/promotions/promotions.go 60.00% 2 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #5491   +/-   ##
=======================================
  Coverage   55.35%   55.35%           
=======================================
  Files         430      430           
  Lines       31851    31846    -5     
=======================================
- Hits        17631    17629    -2     
+ Misses      13214    13212    -2     
+ Partials     1006     1005    -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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

Labels

area/controller Affects the (main) controller kind/bug Something isn't working as intended; If unsure that something IS a bug, start a discussion instead

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant