Skip to content

Improve retry logic#14

Merged
doanac merged 1 commit intomainfrom
retry-improvement
Jan 23, 2026
Merged

Improve retry logic#14
doanac merged 1 commit intomainfrom
retry-improvement

Conversation

@doanac
Copy link
Contributor

@doanac doanac commented Jan 23, 2026

We are starting to see errors from GCS where the socket fails:

|-> ERROR: HTTPSConnectionPool(host='storage.googleapis.com', port=443): Max retries exceeded with url: /quic-ci-artifacts/qualcomm-linux/qualcomm-linux/meta-qcom/21286077511-2/qcom-distro/qcom-armv7a/qcom-multimedia-image-qcom-armv7a.rootfs-20260123181428.ext4?Expires=1769193637&GoogleAccessId=gh-runner-fileserver%40fio-jobserv-ci.iam.gserviceaccount.com&Signature=2q9R2tZlw8tvuXu0sbxDPl6Qzh3svbXNwSzzEwCu0AqUHhRh0s78aW4eJ96xMAuin2WZ9ey8Fm2dYTkS9E0LhAOOwohW3I9anKDMmxMev7svKEfAaAqNtcvyA%2F0Stow4BR8HRzJ9DhLOp8lhGj0%2F5qA6Y%2FTl5Fk5WXSgjFQ94%2Fju5ezr%2FtdqHPk8BoTGlbZz6acg%2B3m%2BREwilSTcETm%2BXpnMm5w%2F9PN3X6PrzbgezKMkLyQWvpd5zrH1d12cguurVmgU%2B0qKZ1TxQLU%2BQ91FrB0Kvx1NJwiPkFvNc6uADpjFmx%2F1xRMgwy9y7Gxwa24Yed9J82SbHMurWO83U9wVWg%3D%3D (Caused by SSLError(SSLEOFError(8, 'EOF occurred in violation of protocol (_ssl.c:2427)')))

This change adds logic to catch connection errors and retry.

We are starting to see errors from GCS where the socket fails:
```
|-> ERROR: HTTPSConnectionPool(host='storage.googleapis.com', port=443): Max retries exceeded with url: /quic-ci-artifacts/qualcomm-linux/qualcomm-linux/meta-qcom/21286077511-2/qcom-distro/qcom-armv7a/qcom-multimedia-image-qcom-armv7a.rootfs-20260123181428.ext4?Expires=1769193637&GoogleAccessId=gh-runner-fileserver%40fio-jobserv-ci.iam.gserviceaccount.com&Signature=2q9R2tZlw8tvuXu0sbxDPl6Qzh3svbXNwSzzEwCu0AqUHhRh0s78aW4eJ96xMAuin2WZ9ey8Fm2dYTkS9E0LhAOOwohW3I9anKDMmxMev7svKEfAaAqNtcvyA%2F0Stow4BR8HRzJ9DhLOp8lhGj0%2F5qA6Y%2FTl5Fk5WXSgjFQ94%2Fju5ezr%2FtdqHPk8BoTGlbZz6acg%2B3m%2BREwilSTcETm%2BXpnMm5w%2F9PN3X6PrzbgezKMkLyQWvpd5zrH1d12cguurVmgU%2B0qKZ1TxQLU%2BQ91FrB0Kvx1NJwiPkFvNc6uADpjFmx%2F1xRMgwy9y7Gxwa24Yed9J82SbHMurWO83U9wVWg%3D%3D (Caused by SSLError(SSLEOFError(8, 'EOF occurred in violation of protocol (_ssl.c:2427)')))
```

This change adds logic to catch connection errors and retry.

Signed-off-by: Andy Doan <andy@foundries.io>
Copy link

@ricardosalveti ricardosalveti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks, above & beyond!

@doanac doanac merged commit 896231d into main Jan 23, 2026
9 checks passed
@doanac doanac deleted the retry-improvement branch January 23, 2026 19:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants