Tags: uber/kraken
Tags
Add unit tests for cluster_client.go Locations function (#539) * Add unit tests for cluster_client.go Locations function - Add table-driven tests for blobclient.Locations() - Test cases: empty cluster, single node, multiple nodes, failover, all fail - Use t.Cleanup() for automatic test server cleanup - Add mock hostlist.List implementation for testing * Simplify slice comparison: replace reflect.DeepEqual with require.Equal --------- Co-authored-by: gauravk <[email protected]>
Fixing lint errors (#527) ## Summary - Fix golangci-lint errcheck violations by properly handling errors. ## Changes - Use `closers.Close()` for deferred Close() calls in non-test code - Use `require.NoError()` for error checks in test files - Update `localdb.Fixture()` to accept `*testing.T` for proper test error handling - Fix type assertion checks in `file_map.go` Signed-off-by: egorikas <[email protected]>
feat: Metrics for writeback queue (#520) * feat: Metrics for writeback queue * Update lib/persistedretry/manager.go Co-authored-by: Copilot <[email protected]> * Report metric interval * Remove 0 checks * Update --------- Co-authored-by: Copilot <[email protected]>
Put images in log lines in build-index (#454) * Put images in log lines in build-index * Update build-index/tagserver/server.go Co-authored-by: Copilot <[email protected]> * Remove unneccessary fields * Remove unneccessary fields * Update build-index/tagstore/store.go Co-authored-by: Copilot <[email protected]> * Update build-index/tagserver/server.go Co-authored-by: Copilot <[email protected]> * Update build-index/tagserver/server.go Co-authored-by: Copilot <[email protected]> * Update build-index/tagserver/server.go Co-authored-by: Copilot <[email protected]> * Update build-index/tagstore/store.go Co-authored-by: Copilot <[email protected]> * Update build-index/tagserver/server.go Co-authored-by: Copilot <[email protected]> * Update build-index/tagserver/server.go Co-authored-by: Copilot <[email protected]> * Update build-index/tagstore/store.go Co-authored-by: Copilot <[email protected]> * Update build-index/tagserver/server.go Co-authored-by: Copilot <[email protected]> * Update build-index/tagstore/store.go Co-authored-by: Copilot <[email protected]> * Update build-index/tagstore/store.go Co-authored-by: Copilot <[email protected]> * Update build-index/tagserver/server.go Co-authored-by: Copilot <[email protected]> * Update build-index/tagserver/server.go Co-authored-by: Copilot <[email protected]> * Update build-index/tagserver/server.go Co-authored-by: Copilot <[email protected]> --------- Co-authored-by: Copilot <[email protected]>
feat(lib/backend): add functionality to close clients (#451) * feat(lib/backend): add functionality to close clients What? This PR aims to add functionality to close the clients and it's resources in case the resources have closers. Why? There are use cases in for gcsbacknd using `storage.transfermanager` to close the single downloader it creates. If we don't close it, there's a possibility of leaking resources. How? Add `Close` to client interface and implement the method for each client.\ Then `backend.Manager` will use them to expose a `Close()` method that can be used by build-index and origin to close them. * add close methods to tests for proper cleanups * resolved comments
PreviousNext