Skip to content

Remove defer to handle new containerregistry limiter#166

Open
nicolasbender wants to merge 1 commit into
mainfrom
fix-download-limit
Open

Remove defer to handle new containerregistry limiter#166
nicolasbender wants to merge 1 commit into
mainfrom
fix-download-limit

Conversation

@nicolasbender

@nicolasbender nicolasbender commented Jun 1, 2026

Copy link
Copy Markdown
Member

go-containerregistry introduces a limiter in the latest patch version which results in a deadlock when it needs to spawn more than 4 jobs. This enables bumping #165

@modulo11

modulo11 commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

Is that behavior not more like a bug in the library and should be fixed there?

If we now explicitly call Close() we should check for its error.

@nicolasbender

Copy link
Copy Markdown
Member Author

Is that behavior not more like a bug in the library and should be fixed there?

If we now explicitly call Close() we should check for its error.

We ignored the error already when we used defer, so I doubt this would change much. Regarding the library: This seems to be introduced as a safeguard and as an alternative, we could also increase the default job size. But this has the risk of failing again when the new limit is still too low

@nicolasbender

Copy link
Copy Markdown
Member Author

Is that behavior not more like a bug in the library and should be fixed there?

You are probably right, we are also not the only one with this issue but the respective PR to fix it seems to be deleted? google/go-containerregistry#2277

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

2 participants