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

fix(ci): cluster pinning with timeout and retry #2321

Merged
merged 1 commit into from
Jan 30, 2025

Conversation

lidel
Copy link
Member

@lidel lidel commented Jan 28, 2025

This PR is backport of https://github.com/protocol/badbits.dwebops.pub/pull/48053

  • Upload of <50MiB CAR with webui should take only a few seconds, everything more means something is really, really wrong
  • It runs pinning to cluster in a loop, assuming pin should not take longer than 1 minute.
  • If the first attempt fails, follow-up ones will run with debug enabled to see which cluster node is picked and fails
  • We retry up to 5 minutes
  • We disable collab-cluster-am6-1.cluster.dwebops.net for now

cc @SgtPooki @2color for visibility

@lidel lidel requested a review from a team as a code owner January 28, 2025 19:57
Copy link
Member

@SgtPooki SgtPooki left a comment

Choose a reason for hiding this comment

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

lgtm

@lidel lidel merged commit 9007e98 into main Jan 30, 2025
1 check passed
@lidel lidel deleted the fix-cluster-pinning-retry-timeout branch January 30, 2025 00:39
ipfs-gui-bot pushed a commit that referenced this pull request Feb 25, 2025
## [4.5.0](v4.4.2...v4.5.0) (2025-02-25)

 CID `bafybeiata4qg7xjtwgor6r5dw63jjxyouenyromrrb4lrewxrlvav7gzgi`

 ---

### Features

* add QR Code to ShareModal ([#2320](#2320)) ([5dbccfd](5dbccfd))

### Bug Fixes

* add hint about basic http auth for rpc ([#2338](#2338)) ([7d00889](7d00889))
* **ci:** cluster pinning with timeout and retry ([#2321](#2321)) ([9007e98](9007e98))
* **ci:** typo in ci.yml ([7fbb51b](7fbb51b))
* **peers:** chrome regression and update ipfs-geoip to v9.2 ([#2333](#2333)) ([699f735](699f735))

### Trivial Changes

* add console warning ([#2332](#2332)) ([76a0c52](76a0c52))
* **deps:** bump actions/setup-go from 4 to 5 ([#2221](#2221)) ([412b89f](412b89f))
* pull transifex translations ([#2331](#2331)) ([e62b487](e62b487))
@ipfs-gui-bot
Copy link
Collaborator

🎉 This PR is included in version 4.5.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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.

3 participants