ipfs / ipfs/distributions

Should we check how many peers there are when pinning?

Open
#762 1 comment 1 reaction 1 assignee Claimed by @galargh View on GitHub
effort/hours P2 team/ipdx
Dominant language
Less
Stars
50
Forks
30
Avg merge
50m
Merged PRs (30d)
1

Description

We're waiting for 2 pin confirmations in https://github.com/ipfs/distributions/blob/master/scripts/ci/pin-to-cluster.sh#L21.

In https://github.com/ipfs/distributions/actions/runs/3126268124/jobs/5072181286 it was the case that there was only 1 peer in the peer_map so it was impossible to satisfy this requirement. I suggest we fail the job immediately if that's the case.

Discovered during https://github.com/ipfs/kubo/issues/9237

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.