Skip to content

Made deploy show policy errors reported by cf-promises - #207

Merged
olehermanse merged 1 commit into
cfengine:masterfrom
victormlg:deploy-show-stderr-errors
Nov 6, 2026
Merged

olehermanse merged 1 commit into
cfengine:masterfrom
victormlg:deploy-show-stderr-errors

Conversation

@victormlg

Copy link
Copy Markdown
Contributor

No description provided.

@codecov

codecov Bot commented Nov 5, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 49.64%. Comparing base (93a2806) to head (c4e3545).

Files with missing lines Patch % Lines
cf_remote/remote.py 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #207      +/-   ##
==========================================
+ Coverage   49.54%   49.64%   +0.09%     
==========================================
  Files          18       18              
  Lines        3217     3217              
==========================================
+ Hits         1594     1597       +3     
+ Misses       1623     1620       -3     
Files with missing lines Coverage Δ
cf_remote/remote.py 32.45% <0.00%> (ø)

... and 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@larsewi larsewi 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.

Showed cf-promises errors from remote hosts

Commit message does not read well

cf-agent runs cf-promises to validate the policy, and cf-promises
writes errors like a duplicate bundle to stderr. Since only stdout was
captured, these errors were silently dropped. Redirected stderr to
stdout and extended the error pattern to also match the
"file.cf:12:0: error:" format used by cf-promises.

Signed-off-by: Victor Moene <victor.moene@northern.tech>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@victormlg
victormlg force-pushed the deploy-show-stderr-errors branch from afc5eed to c4e3545 Compare November 5, 2026 15:39
@victormlg
victormlg requested a review from larsewi November 5, 2026 15:42
@larsewi larsewi changed the title Showed cf-promises errors from remote hosts Made deploy show policy errors reported by cf-promises Nov 5, 2026
@olehermanse
olehermanse merged commit f0abec2 into cfengine:master Nov 6, 2026
12 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants