Skip to content

Commit 5d6268f

Browse files
committed
close dockerhub on error
1 parent c2973f5 commit 5d6268f

1 file changed

Lines changed: 10 additions & 1 deletion

File tree

  • packages/orchestrator/internal/template/server

packages/orchestrator/internal/template/server/main.go

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ func New(
6060
templatePersistence storage.StorageProvider,
6161
limiter *limit.Limiter,
6262
info *service.ServiceInfo,
63-
) (*ServerStore, error) {
63+
) (s *ServerStore, e error) {
6464
logger.Info("Initializing template manager")
6565

6666
artifactsregistry, err := artifactsregistry.GetArtifactsRegistryProvider(ctx)
@@ -72,6 +72,15 @@ func New(
7272
if err != nil {
7373
return nil, fmt.Errorf("error getting docker remote repository provider: %w", err)
7474
}
75+
defer func() {
76+
if e == nil {
77+
return
78+
}
79+
80+
if err := dockerhubRepository.Close(); err != nil {
81+
logger.Error("error closing docker remote repository provider", zap.Error(err))
82+
}
83+
}()
7584

7685
buildPersistance, err := storage.GetBuildCacheStorageProvider(ctx, limiter)
7786
if err != nil {

0 commit comments

Comments
 (0)