-
Notifications
You must be signed in to change notification settings - Fork 31
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
🐛 Fix FS cache for image refs with tags #394
Conversation
Edit: Ready for review Leaving in a draft for now because ideally we want to fail the test if we do not hit cache. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #394 +/- ##
==========================================
- Coverage 33.95% 33.66% -0.29%
==========================================
Files 16 16
Lines 698 695 -3
==========================================
- Hits 237 234 -3
- Misses 435 436 +1
+ Partials 26 25 -1 ☔ View full report in Codecov by Sentry. |
Signed-off-by: Mikalai Radchuk <mradchuk@redhat.com>
c36e717
to
df2b2f0
Compare
@@ -344,6 +347,16 @@ func TestImageRegistry(t *testing.T) { | |||
BaseCachePath: testCache, | |||
} | |||
|
|||
// Create a logger with a simple function-based LogSink that writes to the buffer | |||
var buf bytes.Buffer |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice!
@@ -263,6 +263,8 @@ func (r *ClusterCatalogReconciler) needsUnpacking(catalog *v1alpha1.ClusterCatal | |||
return false | |||
} | |||
// if the spec.Source.Image.Ref was changed, unpack the new ref | |||
// NOTE: we must compare image reference WITHOUT sha hash here |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
dfe121e
Fixes #392