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

Track managed resources when update is skipped#421

Merged
scothis merged 1 commit intovmware-labs:mainfrom
scothis:track-without-update
Aug 31, 2023
Merged

Track managed resources when update is skipped#421
scothis merged 1 commit intovmware-labs:mainfrom
scothis:track-without-update

Conversation

@scothis
Copy link
Contributor

@scothis scothis commented Aug 31, 2023

Tracked resources need to be refreshed on each reconcile request. Previously, the ResourceManager would only track resource when they are created or updated. Now it will always track existing resources and will not be skipped when the update is skipped because the resource is in sync.

The tracked resources need to be refreshed on each reconcile request.
Previously, the ResourceManager would only track resource when they are
created or updated. Now it will always track existing resources and will
not be skipped when the update is skipped because the resource is in
sync.

Signed-off-by: Scott Andrews <andrewssc@vmware.com>
@scothis scothis changed the title Track manged resources when update is skipped Track managed resources when update is skipped Aug 31, 2023
@codecov
Copy link

codecov bot commented Aug 31, 2023

Codecov Report

Patch coverage: 33.33% and no project coverage change.

Comparison is base (919abf6) 60.78% compared to head (e123106) 60.78%.

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

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #421   +/-   ##
=======================================
  Coverage   60.78%   60.78%           
=======================================
  Files          25       25           
  Lines        2494     2494           
=======================================
  Hits         1516     1516           
  Misses        893      893           
  Partials       85       85           
Files Changed Coverage Δ
reconcilers/resourcemanager.go 78.26% <33.33%> (ø)

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

@scothis scothis merged commit a688686 into vmware-labs:main Aug 31, 2023
@scothis scothis deleted the track-without-update branch August 31, 2023 14: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