|
| 1 | +# 🚀 IPFS Core Implementations Weekly Sync 🛰 Dec 16, 2019 |
| 2 | + |
| 3 | +- **Lead:** @stebalien |
| 4 | +- **Notetaker:** @jacobheun |
| 5 | +- **Attendees:** |
| 6 | + - @jimpick |
| 7 | + - @hugomdia |
| 8 | + - @jacobheun |
| 9 | + - @aschmahmann |
| 10 | + - @dirkmc |
| 11 | + - @pedromiguelss |
| 12 | + - @vasco-santos |
| 13 | + - @momack2 |
| 14 | + - @ribasushi |
| 15 | + - @lidel |
| 16 | +- **Recording:** https://youtu.be/fUWkQIQ9kqQ |
| 17 | + |
| 18 | +## Agenda |
| 19 | + |
| 20 | +- Ensure a notetaker exists! |
| 21 | +- Ask everyone to put their name into the list of attendees |
| 22 | +- Initiatives |
| 23 | +- Blockers (from the async status update) |
| 24 | +- General questions. Could be things like: |
| 25 | + - I'm stuck with something, I don't know who to ask. Who knows who to ask? |
| 26 | + - Who can help me with xyz? |
| 27 | +- Plan for this week |
| 28 | + - Select issues to work on |
| 29 | +- Review remaining parking lot topics if there is time left |
| 30 | + |
| 31 | +## Notes |
| 32 | + |
| 33 | +#### Upcoming/Shipped Releases |
| 34 | +@stebalien / @achingbrain / @alanshaw |
| 35 | + |
| 36 | +- Pre release of js-ipfs async before Christmas (fingers crossed) |
| 37 | +- There are some fixed DHT bugs but don't believe there is an RC pending |
| 38 | +- go-ipfs release at some point soon |
| 39 | + |
| 40 | +#### Upgrade Testing Infra/Process |
| 41 | +@jimpick / @stebalien / @daviddias |
| 42 | + |
| 43 | +- much of the team is together for the last part of last week, first part of this week |
| 44 | + - testground hacking |
| 45 | + - DHT tests for go-ipfs |
| 46 | + - onboarding Anton |
| 47 | + - Anton is investigating alternate "runners" (eg. Kubernetes) |
| 48 | + - Jim is working on connectivity tests |
| 49 | +- Looking to launch a usable version by Wednesday |
| 50 | +- Switching away from Docker Swarm to Kubernetes |
| 51 | + - Docker Swarm can't handle the larger number of nodes |
| 52 | + |
| 53 | +#### Subdomain Gateway (Base32, Origin isolation) |
| 54 | +@lidel |
| 55 | + |
| 56 | +- Waiting for review of [config options proposal in subdomain GW PR /go-ipfs/pull/6096](https://github.com/ipfs/go-ipfs/pull/6096#issuecomment-564939993) |
| 57 | + - All opinions welcome! |
| 58 | + |
| 59 | +#### Distributed Signaling (js-libp2p) |
| 60 | +@jacobheun |
| 61 | + |
| 62 | +- On hold awaiting js-libp2p refactor, should kick back off shortly after the new year. |
| 63 | + |
| 64 | +#### IPNS (go-ipfs / js-ipfs) |
| 65 | +@aschmahmann / @hugomrdias |
| 66 | + |
| 67 | +IPNS over PubSub |
| 68 | + - IPFS integration https://github.com/ipfs/go-ipfs/pull/6758 🚢🚀 |
| 69 | + - Don't worry still plenty of work to be done, but for now other things are higher priority |
| 70 | + |
| 71 | +#### Add Performance (go-ipfs / js-ipfs) |
| 72 | +@aschmahmann / @achingbrain |
| 73 | + |
| 74 | +Asynchronous Datastores |
| 75 | + - Blocked awaiting review https://github.com/ipfs/go-ipfs/pull/6785 |
| 76 | + - @dirkmc to take a look |
| 77 | + |
| 78 | +#### Migration to multihash keys in blockstore (js-ipfs) |
| 79 | +@alanshaw / @auhau |
| 80 | + |
| 81 | +- Blocked on not having time. |
| 82 | + |
| 83 | +#### Bitswap Updates |
| 84 | +@dirkmc |
| 85 | +- Started work on porting the [bitswap PoC changes](https://github.com/ipfs/go-bitswap/pull/189) to js-bitswap https://github.com/ipfs/js-ipfs-bitswap/pull/203 |
| 86 | + |
| 87 | +#### js-ipfs/js-libp2p/js-ipld async await refactor |
| 88 | +@achingbrain / @jacobheun / @vasco-santos / @alanshaw |
| 89 | + |
| 90 | +- js-libp2p |
| 91 | + - Got a pre release out the door, [email protected] |
| 92 | + - Integrated the libp2p pre release into js bitswap [js-ipfs-bitswap/pull/206](https://github.com/ipfs/js-ipfs-bitswap/pull/206) |
| 93 | + - Found a few issues that needed to be fixed, but otherwise the migration was fairly straight forward. |
| 94 | + - Released [email protected] with these fixes |
| 95 | + - Several PRs for updating all the [libp2p examples](https://github.com/libp2p/js-libp2p/pulls?utf8=%E2%9C%93&q=is%3Apr+is%3Aopen+examples) |
| 96 | + - Improved the API and Configuration [docs](https://github.com/libp2p/js-libp2p/tree/refactor/async-await/doc) |
| 97 | + - Getting started guide is in progress [js-libp2p#514](https://github.com/libp2p/js-libp2p/pull/514) |
| 98 | + - Starting integration into js-ipfs [js-ipfs#2680](https://github.com/ipfs/js-ipfs/pull/2680) |
| 99 | + - This week we'll be working on js-ipfs integration, bug fixes and cleaning up docs |
| 100 | + - Looking to get a Release Candidate out this week |
| 101 | + |
| 102 | +- js-ipfs |
| 103 | + - Many PRs open https://github.com/ipfs/js-ipfs/pulls?utf8=%E2%9C%93&q=is%3Apr+async%2Fawait+ |
| 104 | + - Merging all into master PR https://github.com/ipfs/js-ipfs/pull/2547 |
| 105 | + - Aiming for pre-release before EOY |
| 106 | + - Migration guide |
| 107 | + - https://gist.github.com/alanshaw/04b2ddc35a6fff25c040c011ac6acf26 |
| 108 | + - Blog post in draft (not ready for review) |
| 109 | + - https://github.com/ipfs/blog/pull/340 |
| 110 | + |
| 111 | + |
| 112 | +### Design Review Proposals |
| 113 | +> You want to propose something for design review. |
| 114 | +
|
| 115 | +@aschmahmann: Provider records using multihashes instead of CIDs (https://github.com/libp2p/go-libp2p-kad-dht/pull/422). It seems like the only thing blocking this is a sense of consensus, let's pick an answer and merge some code! |
| 116 | + |
| 117 | +DHT Provider record is blocked from being merged awaiting consensus on what types of keys the DHT should allow (multihashes, multihashes + CIDs, anything, anything less than 80 bytes, etc.) |
| 118 | + |
| 119 | +@stebalien will put this to the libp2p team with a deadline of when IPFS needs a decision by. |
| 120 | + |
| 121 | +### Blockers/Asks |
| 122 | +> You are blocked and need to talk about it to help unblock. List it here! |
| 123 | +- Please grade Q4 OKRs! |
| 124 | + |
| 125 | +### Questions |
| 126 | +> You have questions and need them answered. List them here. |
| 127 | +
|
| 128 | +### Parking Lot |
| 129 | +> Anything you would like to discuss that doesn't fit into any of the other categories. |
| 130 | +
|
| 131 | +* go-ipfs has switched from /ipfs to /p2p multiaddrs |
| 132 | + * js-ipfs will switch to using /p2p by default in the upcoming async release |
| 133 | +* shipped: |
| 134 | + * [js-ipfsd-ctl v1.0.0](https://github.com/ipfs/js-ipfsd-ctl/releases/tag/v1.0.0) |
| 135 | + * [ipfs-companion v2.10.0](https://github.com/ipfs-shipyard/ipfs-companion/releases/tag/v2.10.0) (only Firefox for now, Chromium soon) |
| 136 | + |
| 137 | +### Week Update (for async review) |
| 138 | +> Only post updates that are relevant to the IPFS Core Implementations Work. We know you do a ton more things but for this call, the focus are the implementations ❤️ 🌟 |
| 139 | +
|
| 140 | +@jacobheun |
| 141 | +- Done: |
| 142 | + - libp2p async refactor |
| 143 | + - Released a pre release |
| 144 | + - Added coalescing dial support [js-libp2p#518](https://github.com/libp2p/js-libp2p/pull/518) |
| 145 | + - Integrated pre release into js-ipfs-bitswap [js-ipfs-bitswap#206](https://github.com/ipfs/js-ipfs-bitswap/pull/206) |
| 146 | + - Misc fixes and documentation updates as part of downstream migration |
| 147 | +- Next: |
| 148 | + - Integrate libp2p pre release into js-ipfs |
| 149 | + - Finish reviewing all the libp2p examples |
| 150 | + - Create a migration guide from [email protected] to [email protected] |
| 151 | + - Do a final pass through the API and Configuration docs |
| 152 | + - Get an RC of libp2p out. |
| 153 | + |
| 154 | + |
| 155 | +@dirkmc |
| 156 | +- Done: |
| 157 | + - Started work on porting the [bitswap PoC changes](https://github.com/ipfs/go-bitswap/pull/189) to js-bitswap https://github.com/ipfs/js-ipfs-bitswap/pull/203 |
| 158 | +- Next: |
| 159 | + - [Fix bootstrapping](https://github.com/ipfs/go-ipfs/issues/6797) |
| 160 | + |
| 161 | +@aschmahmann |
| 162 | +- Done: |
| 163 | + - [IPNS over PubSub as an independent transport](https://github.com/ipfs/go-ipfs/pull/6758) shipped |
| 164 | + - [DHT Provider records using multihashes instead of CIDs PR](https://github.com/libp2p/go-libp2p-kad-dht/pull/422) |
| 165 | + - Reviewed [DHT PR for persisting routing table](https://github.com/libp2p/go-libp2p-kad-dht/pull/383) |
| 166 | + - Dealt with a few pubsub things |
| 167 | + - [separating meshsub from gossipsub](https://github.com/libp2p/go-libp2p-pubsub/issues/249) |
| 168 | + - potentially useful for IPNS and other protocols with small messages |
| 169 | + - libp2p decided to add some functionality to unblock Eth2 that potentially causes problems for running pubsub as a service. [issue](https://github.com/libp2p/go-libp2p-pubsub/issues/247) |
| 170 | +- Blocked: |
| 171 | + - DHT Provider record is blocked from being merged awaiting consensus on what types of keys the DHT should allow (multihashes, multihashes + CIDs, anything, anything less than 80 bytes, etc.) |
| 172 | + - Async Datastores PR https://github.com/ipfs/go-ipfs/pull/6785 |
| 173 | +- Next: |
| 174 | + - Work on improving DHT queries (both [don't query all the peers in the DHT](https://github.com/libp2p/go-libp2p-kad-dht/pull/291), and [query many disjoint paths](https://github.com/libp2p/go-libp2p-kad-dht/pull/204) - these PRs are not new I'm synthesizing them into a new PR based off the stabilize branch) |
| 175 | + |
| 176 | +@vasco-santos |
| 177 | +- Done: |
| 178 | + - API and CONFIG docs merged |
| 179 | + - getting started first iteration |
| 180 | + - [libp2p/js-libp2p#514](https://github.com/libp2p/js-libp2p/pull/514) |
| 181 | + - async refactor for ping |
| 182 | + - [libp2p/js-libp2p#505](https://github.com/libp2p/js-libp2p/pull/505) |
| 183 | + - js-libp2p release fixes |
| 184 | + - [libp2p/js-libp2p#510](https://github.com/libp2p/js-libp2p/pull/510) |
| 185 | + - [libp2p/js-libp2p#513](https://github.com/libp2p/js-libp2p/pull/513) |
| 186 | + - [libp2p/js-libp2p#517](https://github.com/libp2p/js-libp2p/pull/517) |
| 187 | + - more examples refactored |
| 188 | + - [libp2p/js-libp2p#507](https://github.com/libp2p/js-libp2p/pull/507) |
| 189 | + - [libp2p/js-libp2p#508](https://github.com/libp2p/js-libp2p/pull/508) |
| 190 | +- Blocked: N/A |
| 191 | +- Next: |
| 192 | + - New iteration on some `js-libp2p` examples |
| 193 | + - Finish getting started guide and blogpost |
| 194 | + |
| 195 | +@pedromiguelss |
| 196 | +- Done: |
| 197 | + - Analyze and unskip skipped tests on `interface-ipfs-core` |
| 198 | + - https://github.com/ipfs/js-ipfs/pull/2673 |
| 199 | + - https://github.com/ipfs/js-ipfs/pull/2677 |
| 200 | +- Blocked: N/A |
| 201 | +- Next: |
| 202 | + - Keep verifying skipped `interface-ipfs-core` tests on `js-ipfs` and/or `js-ipfs-http-client` |
| 203 | + |
| 204 | +@lidel |
| 205 | +- Done: |
| 206 | + - ipfs-companion |
| 207 | + - [Chrome Web Store Removal notification for IPFS Companion Beta](https://github.com/ipfs-shipyard/ipfs-companion/issues/832) |
| 208 | + - [feat: copy shareable link during file import](https://github.com/ipfs-shipyard/ipfs-companion/issues/833) |
| 209 | + - [fix: ipfs:// in Firefox incognito window](https://github.com/ipfs-shipyard/ipfs-companion/pull/835) |
| 210 | + - [fix: remove normalizeLinksContentScript](https://github.com/ipfs-shipyard/ipfs-companion/pull/836) |
| 211 | + - beta release: [v2.9.0.881](https://github.com/ipfs-shipyard/ipfs-companion/releases/tag/v2.9.0.881) |
| 212 | + - debugging iframe issue with custom protocol handler, reported [upstream bug in Firefox](https://bugzilla.mozilla.org/show_bug.cgi?id=1603737) |
| 213 | + - stable release: [v2.10.0](https://github.com/ipfs-shipyard/ipfs-companion/releases/tag/v2.10.0) |
| 214 | + - go-libp2p-core: [feat: support encoding/decoding peer IDs as CIDs _in text_](https://github.com/libp2p/go-libp2p-core/pull/41) |
| 215 | + - cidv1b32: |
| 216 | + - go-ipfs: [discussing new config for gateway functionality](https://github.com/ipfs/go-ipfs/pull/6096#issuecomment-564939993) |
| 217 | + - in-web-browsers: [docs: add test for ipfs:// origin isolation](https://github.com/ipfs/in-web-browsers/pull/156) |
| 218 | + - ipfs/public-gateway-checker: [feat: add public subdomain gateways and Origin check](https://github.com/ipfs/public-gateway-checker/pull/78) |
| 219 | + - Other |
| 220 | + - [Design Review for ipfs/docs/405](https://github.com/ipfs/docs/issues/405#issuecomment-564310469) |
| 221 | + - [Unspooling current MetaMask use of our gateways](https://github.com/ipfs/infra/issues/493#issuecomment-563406087) |
| 222 | + - Feedback for https://www.transifex.com/ipfs/public/ |
| 223 | + - Review: [js-ipfs-http-client async iterables refactor](https://github.com/ipfs/js-ipfs-http-client/pull/1183#pullrequestreview-331883544) |
| 224 | +- Next: |
| 225 | + - handle the security thing |
| 226 | + - finish webui test matrix PR |
| 227 | + |
| 228 | +@hugomrdias |
| 229 | +- Done: |
| 230 | + - ipfsd-ctl 1.0.0 released |
| 231 | + - Reduce bundle size in bitswap ipfs/js-ipfs-bitswap#203 https://github.com/ipfs/js-ipfs-bitswap/pull/203 |
| 232 | + - Release ipfs-utils 0.5.0 and backport unixfs 1.5 changes to 0.4.1 |
| 233 | + - Released aegir 20.5.0 https://github.com/ipfs/aegir/releases/tag/v20.5.0 |
| 234 | + - dedupe tests |
| 235 | + - https://github.com/ipfs/interface-js-ipfs-core/pull/573 |
| 236 | + - https://github.com/ipfs/js-ipfs-http-client/pull/1200 |
| 237 | + - https://github.com/ipfs/js-ipfs/pull/2669 |
| 238 | +- Blocked: |
| 239 | + - dedupe PRs need to be merged |
| 240 | +- Next: |
| 241 | + - Improve js-ipfs cli tests |
| 242 | + - Review async/await PRs |
| 243 | + |
| 244 | +@Stebalien |
| 245 | +- Done: |
| 246 | + - Testground work |
| 247 | + - Reviews |
| 248 | + - Finished the switch from /ipfs to /p2p mutliadd |
| 249 | + - Fixed a bunch of bugs |
| 250 | +- Next: |
| 251 | + - Finish shipping testground work. |
| 252 | + - Review bitswap patches |
| 253 | + - Plan out path for improving content routing |
| 254 | + - Review gateway subdomain proposal |
| 255 | + |
| 256 | +@alanshaw |
| 257 | +- Done: |
| 258 | + - Many many js-ipfs and js-ipfs-http-client async/await PRs https://github.com/ipfs/js-ipfs/pulls?utf8=%E2%9C%93&q=is%3Apr+async%2Fawait+ |
| 259 | +- Next: |
| 260 | + - Get async/await merged |
| 261 | + - Get UnixFSv1.5 merged |
| 262 | + - Pre-release js-ipfs 0.41 |
| 263 | + |
| 264 | +@achingbrain |
| 265 | +- Done: |
| 266 | + - UnixFSv1.5 interface tests https://github.com/ipfs/interface-js-ipfs-core/pull/572 |
| 267 | + - Detecting unfilled optional fields in protons |
| 268 | +- Next: |
| 269 | + - Get UnixFSv1.5 merged |
| 270 | + - Review async/await PRs |
0 commit comments