Skip to content
This repository has been archived by the owner on Aug 28, 2024. It is now read-only.

Allow errors to be quiet #429

Merged
merged 1 commit into from
Sep 6, 2023
Merged

Allow errors to be quiet #429

merged 1 commit into from
Sep 6, 2023

Conversation

scothis
Copy link
Contributor

@scothis scothis commented Sep 6, 2023

ErrQuiet errors are not logged within reconciler-runtime and do not trigger events to be recorded. This behavior is conventional rather than technical. An implementor may choose to ignore these conventions when it fits their purpose. Quite is not silent.

Also redefines ErrHaltSubReconcilers to wrap ErrQuiet. HaltSubReconcilers is deprecated in favor of ErrHaltSubReconcilers.

ErrQuiet errors are not logged within reconciler-runtime and do not
trigger events to be recorded. This behavior is conventional rather than
technical. An implementor may choose to ignore these conventions when it
fits their purpose. Quite is not silent.

Also redefines ErrHaltSubReconcilers to wrap ErrQuiet.
HaltSubReconcilers is deprecated in favor of ErrHaltSubReconcilers.

Signed-off-by: Scott Andrews <andrewssc@vmware.com>
@codecov
Copy link

codecov bot commented Sep 6, 2023

Codecov Report

Patch coverage: 91.11% and project coverage change: +0.19% 🎉

Comparison is base (ce2ead6) 60.89% compared to head (a6194fe) 61.09%.

❗ Current head a6194fe differs from pull request most recent head 8063046. Consider uploading reports for the commit 8063046 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #429      +/-   ##
==========================================
+ Coverage   60.89%   61.09%   +0.19%     
==========================================
  Files          26       26              
  Lines        2514     2529      +15     
==========================================
+ Hits         1531     1545      +14     
- Misses        896      898       +2     
+ Partials       87       86       -1     
Files Changed Coverage Δ
reconcilers/reconcilers.go 73.07% <0.00%> (-0.71%) ⬇️
reconcilers/resourcemanager.go 78.99% <85.71%> (+0.73%) ⬆️
reconcilers/aggregate.go 61.15% <100.00%> (+0.32%) ⬆️
reconcilers/child.go 84.72% <100.00%> (+0.21%) ⬆️
reconcilers/finalizer.go 81.15% <100.00%> (+0.27%) ⬆️
reconcilers/resource.go 74.48% <100.00%> (+0.39%) ⬆️
reconcilers/sync.go 78.43% <100.00%> (+0.88%) ⬆️
reconcilers/webhook.go 90.47% <100.00%> (+0.09%) ⬆️

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

@scothis scothis merged commit 0a566ab into vmware-labs:main Sep 6, 2023
1 check passed
@scothis scothis deleted the quiet-errs branch September 6, 2023 12:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants