Skip to content

deal with TimeoutError from ssl.py#45683

Merged
haampie merged 1 commit intodevelopfrom
fix/timeout-error-handling
Aug 12, 2024
Merged

deal with TimeoutError from ssl.py#45683
haampie merged 1 commit intodevelopfrom
fix/timeout-error-handling

Conversation

@haampie
Copy link
Copy Markdown
Member

@haampie haampie commented Aug 12, 2024

urllib (or rather ssl.py) can throw TimeoutError, which is not caught in various places, causing the error to be fatal.

See for example https://gitlab.spack.io/spack/spack/-/jobs/12182279.

@spackbot-app spackbot-app bot added binary-packages core PR affects Spack core functionality fetching utilities labels Aug 12, 2024
@haampie haampie added this to the v0.22.2 milestone Aug 12, 2024
@alalazo alalazo self-assigned this Aug 12, 2024
@haampie haampie added the hotfix label Aug 12, 2024
@haampie haampie merged commit 3b59817 into develop Aug 12, 2024
@haampie haampie deleted the fix/timeout-error-handling branch August 12, 2024 11:06
diehlpk pushed a commit to diehlpk/spack that referenced this pull request Aug 14, 2024
Chrismarsh pushed a commit to Chrismarsh/spack that referenced this pull request Aug 16, 2024
tldahlgren pushed a commit to AcriusWinter/spack that referenced this pull request Aug 20, 2024
FrederickDeny pushed a commit to FrederickDeny/spack that referenced this pull request Aug 26, 2024
haampie added a commit that referenced this pull request Sep 21, 2024
@haampie haampie mentioned this pull request Sep 21, 2024
26 tasks
haampie added a commit that referenced this pull request Sep 21, 2024
haampie added a commit that referenced this pull request Sep 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants