Skip to content
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

Timeout in OCI push #91

Closed
matthiasfehr opened this issue Jun 14, 2024 · 2 comments · Fixed by #92, actions/attest-sbom#75 or actions/attest-build-provenance#123
Closed

Timeout in OCI push #91

matthiasfehr opened this issue Jun 14, 2024 · 2 comments · Fixed by #92, actions/attest-sbom#75 or actions/attest-build-provenance#123

Comments

@matthiasfehr
Copy link

When using the action actions/attest-build-provenance with the option push-to-registry enabled, I regularly got a timeout error.

Digging into the action I discovered that the timeout corresponding to the OCI push is set here to 2 seconds, which is obviously pretty low and not every registry can fulfil the request in this time, especially not ones which are backed by an object storage.

May I ask here to increase the timeout to something more usual like e.g 30s?

For the time being, is there any possible workaround until this gets fixed?

@bdehamer
Copy link
Collaborator

@matthiasfehr I've got a fix queued up and will try to get this deployed today.

Out of curiosity, which OCI registry are you using?

@matthiasfehr
Copy link
Author

I'm running a self implemented OCI registry which can handle different storage backends, e.g. with the filesystem storage backend the operations are fast enough and the action succeeds without issues, but with the S3 backend it often takes a bit more than 2 seconds and therefore the action times out. It took me quite some time to figure out that the problem was the unusually low timeout of this action, so fixing it may prevent also others of breaking their heads over it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants
@bdehamer @matthiasfehr and others