Skip to content

Conversation

crenshaw-dev
Copy link
Member

@crenshaw-dev crenshaw-dev commented Jun 5, 2025

Things are hard to debug unless error messages are wrapped. Long live verbosity.

Argo CD test PR: argoproj/argo-cd#23282

Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
Copy link

codecov bot commented Jun 5, 2025

Codecov Report

Attention: Patch coverage is 9.28571% with 127 lines in your changes missing coverage. Please review.

Project coverage is 53.62%. Comparing base (8849c3f) to head (876f0c8).
Report is 47 commits behind head on master.

Files with missing lines Patch % Lines
pkg/utils/kube/resource_ops.go 0.00% 26 Missing ⚠️
pkg/utils/kube/ctl.go 0.00% 23 Missing ⚠️
pkg/diff/diff.go 8.69% 21 Missing ⚠️
pkg/sync/sync_context.go 31.03% 18 Missing and 2 partials ⚠️
pkg/cache/cluster.go 5.00% 18 Missing and 1 partial ⚠️
pkg/engine/engine.go 0.00% 5 Missing ⚠️
agent/main.go 0.00% 3 Missing ⚠️
pkg/utils/kube/kube.go 0.00% 3 Missing ⚠️
pkg/cache/references.go 0.00% 2 Missing ⚠️
pkg/utils/kube/convert.go 33.33% 2 Missing ⚠️
... and 3 more
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #732      +/-   ##
==========================================
- Coverage   54.26%   53.62%   -0.65%     
==========================================
  Files          64       64              
  Lines        6164     6514     +350     
==========================================
+ Hits         3345     3493     +148     
- Misses       2549     2745     +196     
- Partials      270      276       +6     

☔ 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.

Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
@crenshaw-dev crenshaw-dev marked this pull request as ready for review June 6, 2025 12:39
@crenshaw-dev crenshaw-dev requested a review from a team as a code owner June 6, 2025 12:39
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
Copy link

sonarqubecloud bot commented Jun 6, 2025

@crenshaw-dev crenshaw-dev merged commit cebed7e into argoproj:master Jun 6, 2025
5 checks passed
RoelofKuijpers pushed a commit to RoelofKuijpers/gitops-engine that referenced this pull request Jul 29, 2025
* chore: wrap errors

Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>

* report list result along with error

Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>

* fixes

Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>

---------

Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
Signed-off-by: Roelof Kuijpers <roelof.kuijpers@energyessentials.nl>
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.

2 participants