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

OKR 2018 Q2 #1281

Merged
merged 9 commits into from Apr 14, 2018

Conversation

@daviddias
Member

daviddias commented Mar 26, 2018

Currently just a placeholder. Please submit your proposals for Objects and Key Results through this PR. I'll keep revisiting, add my own and trying to coalesce things until we feel there is something ready to discuss as a group or to present to the rest of the IPFS Community.

@vmx

This comment has been minimized.

Member

vmx commented Mar 26, 2018

From my side I'd have:

  • Awesome Endaevour: Flow #1260
  • Making js-ipfs reliable
@dryajov

This comment has been minimized.

Member

dryajov commented Mar 26, 2018

Just a preliminary list of things I'd like to see being worked on:

  • Deploy ipfs to the galaxy - #804
  • DHT - #856
  • mplex pull streams - libp2p/js-libp2p-mplex#73
  • move stack to webworkers - #1195
  • make minification work (remove constructor.name usage from everywhere)
@victorb

This comment has been minimized.

Member

victorb commented Mar 26, 2018

  • Something regarding fixing the tests, some suggestions from the top of my head:
    • keep the same code coverage but lower the time the entire suite takes to run
    • fully isolated tests (be able to run npm test in two terminal tabs at the same time without any failures)
  • Fixing stability and error handling
  • Improve developer experience for bubbling up and managing changes across multiple repositories/modules
  • Deployed js-ipfs bootstrappers and gateway
@daviddias

This comment has been minimized.

Member

daviddias commented Mar 28, 2018

Hi everyone! Just completed my first take on OKRs for next quarter. These mostly come from my perspective as the lead of the project, from listening to your feedback and also by trying to project what is the right direction of the project. These are not finalized and until they are frozen they are open to discussion. I expect that some of these will be dropped once the time allocation exercise is performed.

I've used Github reviews feature to tag the people that are fully allocated or that have committed full time to work on the js-ipfs & js-libp2p projects. These are: @wraithgar, @jacobheun, @achingbrain, @dryajov, @vasco-santos, @fsdiogo and @hugomrdias.

I also want to ask the feedback and participation on this discussion to the people that have been contributing part time (or even quarter time) to the project, you too should feel welcome to tackle KRs for the js-ipfs & js-libp2p projects and of course, propose changes, new ones and so on. These people are: @vmx, @pgte, @VictorBjelkholm, @hacdias, @mkg20001, @richardschneider, @JonKrone, @nunofmn, @alanshaw, @olizilla, @AdamStone and @chriscool. (Hope I didn't miss anyone 🤞🏽)

You will see in the list that I haven't assigned Priority nor Owner. If you are interested in tackling one of these KR, please do a line comment. I'll do line comments for the ones I believe I can do and that I should do + tag the people that I've already talked about on what to focus on next quarter.

Thank you all in advance! Looking forward to get all of your feedback!

OKR.md Outdated
- `PX` - OWNER - The uncaught errors and memory usage are under control and a daemon can run for a week non stop
- `PX` - OWNER - One or more js-ipfs daemons are part of the IPFS infrastructure
- `PX` - OWNER - js.ipfs.io exposes an IPFS Gateway provided by a js-ipfs daemon

This comment has been minimized.

@daviddias

daviddias Mar 28, 2018

Member

@VictorBjelkholm can I assign you on this one given that you already started this work with #804?

This comment has been minimized.

@victorb

victorb Mar 28, 2018

Member

👍 Should we have one bootstrapper node be a js-ipfs node as well? My thinking is that the js-ipfs gateway probably won't be used a lot, while a default bootstrap node would get a lot of usage

This comment has been minimized.

@daviddias

daviddias Mar 28, 2018

Member

See KR on L12 One or more js-ipfs daemons are part of the IPFS infrastructure. That should be it :)

This comment has been minimized.

@victorb

victorb Mar 28, 2018

Member

Ah, I would make it more explicit, include "bootstrap" somewhere in the text 👍

OKR.md Outdated
### The core can handle vast amounts of data
- `PX` - OWNER - The MFS is implemented in js-ipfs

This comment has been minimized.

@daviddias

daviddias Mar 28, 2018

Member

@achingbrain is assigned with this one

OKR.md Outdated
### The core can handle vast amounts of data
- `PX` - OWNER - The MFS is implemented in js-ipfs
- `PX` - OWNER - npm on IPFS is used as a test vector of js-ipfs capabilities

This comment has been minimized.

@daviddias

daviddias Mar 28, 2018

Member

@achingbrain is also assigned with this one given that the MFS is a dependency.

OKR.md Outdated
- `PX` - OWNER - npm on IPFS is used as a test vector of js-ipfs capabilities
- `PX` - OWNER - js-ipfs in the browser can load a dataset of at least 100Gb (create archives.js.ipfs.io)
- `PX` - OWNER - js-ipfs in Node.js can load a dataset of at least 1TB (use npm to prove it)
- `PX` - OWNER - A S3 IPFS Repo backend exists and it is easily pluggable on `new IPFS`

This comment has been minimized.

@daviddias

daviddias Mar 28, 2018

Member

@jacobheun is tasked with this one

OKR.md Outdated
- `PX` - OWNER - js-ipfs in the browser can load a dataset of at least 100Gb (create archives.js.ipfs.io)
- `PX` - OWNER - js-ipfs in Node.js can load a dataset of at least 1TB (use npm to prove it)
- `PX` - OWNER - A S3 IPFS Repo backend exists and it is easily pluggable on `new IPFS`
- `PX` - OWNER - The PIN API is finished and merged.

This comment has been minimized.

@daviddias

daviddias Mar 28, 2018

Member

@JonKrone has been working on this one following the work of @AdamStone. @JonKrone let us know if you will have the availability to take this one to the finish line.

OKR.md Outdated
- `PX` - achingbrain - The MFS is implemented in js-ipfs
- `PX` - achingbrain - npm on IPFS is used as a test vector of js-ipfs capabilities
- `PX` - OWNER - js-ipfs in the browser can fetch a dataset of at least 100Gb (and create archives.js.ipfs.io)

This comment has been minimized.

@daviddias

daviddias Mar 31, 2018

Member

Needs an Owner.

OKR.md Outdated
- `PX` - achingbrain - The MFS is implemented in js-ipfs
- `PX` - achingbrain - npm on IPFS is used as a test vector of js-ipfs capabilities
- `PX` - OWNER - js-ipfs in the browser can fetch a dataset of at least 100Gb (and create archives.js.ipfs.io)
- `PX` - OWNER - js-ipfs in Node.js can add and transfer a dataset of at least 1TB (use npm to prove it)

This comment has been minimized.

@daviddias

daviddias Mar 31, 2018

Member

Needs an Owner.

OKR.md Outdated
### The daemon is stable and so core is reliable
- `PX` - OWNER - The uncaught errors and memory usage are under control and a daemon can run for a week non stop.

This comment has been minimized.

@daviddias

daviddias Mar 31, 2018

Member

Needs an Owner.

This comment has been minimized.

@daviddias

daviddias Apr 12, 2018

Member

Any takers? @fsdiogo how do you feel about it?

OKR.md Outdated
- `PX` - achingbrain - The MFS is implemented in js-ipfs.
- `PX` - achingbrain - npm on IPFS is used as a test vector of js-ipfs capabilities.
- `PX` - OWNER - js-ipfs in the browser can fetch a dataset of at least 100Gb (and create archives.js.ipfs.io).

This comment has been minimized.

@daviddias

daviddias Mar 31, 2018

Member

Needs an Owner.

This comment has been minimized.

@hugomrdias

hugomrdias Apr 6, 2018

Collaborator

I can own this one.

OKR.md Outdated
- `PX` - achingbrain - The MFS is implemented in js-ipfs.
- `PX` - achingbrain - npm on IPFS is used as a test vector of js-ipfs capabilities.
- `PX` - OWNER - js-ipfs in the browser can fetch a dataset of at least 100Gb (and create archives.js.ipfs.io).
- `PX` - OWNER - js-ipfs in Node.js can add and transfer a dataset of at least 1TB.

This comment has been minimized.

@daviddias

daviddias Mar 31, 2018

Member

Needs an Owner.

OKR.md Outdated
- `PX` - OWNER - js-ipfs in the browser can fetch a dataset of at least 100Gb (and create archives.js.ipfs.io).
- `PX` - OWNER - js-ipfs in Node.js can add and transfer a dataset of at least 1TB.
- `PX` - jacobheun - A S3 IPFS Repo backend exists and it is easily pluggable on `new IPFS`.
- `PX` - OWNER - The PIN API is finished and merged.

This comment has been minimized.

@daviddias

daviddias Mar 31, 2018

Member

Needs an Owner.

This comment has been minimized.

@daviddias

daviddias Apr 3, 2018

Member

@JonKrone would you like to be the owner for this one given that you already did most of the work?

This comment has been minimized.

@JonKrone

JonKrone Apr 6, 2018

Contributor

@diasdavid Yep yep, happy to. I misunderstood this PR, sorry for the late response.

OKR.md Outdated
### libp2p levels up, learns new superpowers and a dedicated team is formed
- `PX` - OWNER - libp2p has primitives to keep connections under control.

This comment has been minimized.

@daviddias

daviddias Mar 31, 2018

Member

Needs an Owner.

This comment has been minimized.

@gpestana

gpestana Apr 3, 2018

@diasdavid, count me in to help with the libp2p improvements. Need some time to ramp up with the code base, but should be quick so I can help with the PR reviews and whatnot for now.

And when talking about libp2p here, you mean strictly js-libp2p?

This comment has been minimized.

@hugomrdias

hugomrdias Apr 6, 2018

Collaborator

I can own this one.

OKR.md Outdated
### The Community is pumped by having a selection of example applications, blog posts and video tutorials on how to build DApps with IPFS
- `PX` - OWNER - Blog post on PeerPad Architecture with focus on its Read/Write Capability System.

This comment has been minimized.

@daviddias

daviddias Mar 31, 2018

Member

Needs an Owner.

@daviddias

This comment has been minimized.

Member

daviddias commented Apr 2, 2018

I would also like to include as part of the OKRs:

Not sure if it fits though

@daviddias

This comment has been minimized.

Member

daviddias commented Apr 3, 2018

I feel that given that we increase on firepower we had last quarter, we are being pretty humble with OKRs for Q2. Let's make more ambitious (still realistic plans). I'm considering adding (raw list, will polish when committing to the doc)

  • Achieve CLI and HTTP-API feature Parity with go-ipfs https://github.com/ipfs/ipfs/blob/master/IMPLEMENTATION_STATUS.md
  • Solve outstanding issues for full electron support #843
  • Other Runtime support (Cordova, React Native, Ionic and more)
  • OpenTracing for js-ipfs
  • ipfs-service-worker browser gateway (for the great fun!)
  • IPNS
  • ~~IPNS over PubSub~
  • IPNS over the DHT
  • Parity to IPFS module
  • NAT Traversal supported
  • Adding Torrent support to js-ipfs
  • Create the IPFS Web Component (ipfs/notes#29)
  • Bring code coverage up to 100% everywhere!
@wraithgar

This comment has been minimized.

Contributor

wraithgar commented Apr 4, 2018

You can put me down for:

@vasco-santos

This comment has been minimized.

Member

vasco-santos commented Apr 6, 2018

@diasdavid I will have the following OKRs:

  • ipfs-service-worker browser gateway
  • IPNS
  • IPNS over PubSub
  • IPNS over the DHT
@dryajov

This comment has been minimized.

Member

dryajov commented Apr 6, 2018

@diasdavid I can take care of

  • NAT Traversal supported

daviddias added some commits Apr 12, 2018

OKR.md Outdated
- `PX` - achingbrain - The MFS is implemented in js-ipfs.
- `PX` - achingbrain - npm on IPFS is used as a test vector of js-ipfs capabilities.
- `PX` - hugomrdias - js-ipfs in the browser can fetch a dataset of at least 100Gb (and create archives.js.ipfs.io).
- `PX` - OWNER - js-ipfs in Node.js can add and transfer a dataset of at least 1TB.

This comment has been minimized.

@daviddias

daviddias Apr 12, 2018

Member

@hugomrdias wanna take this one (related to the one above)?

@daviddias daviddias merged commit 5b06945 into master Apr 14, 2018

6 of 7 checks passed

ci/circleci Your tests failed on CircleCI
Details
Node Security No known vulnerabilities found
Details
continuous-integration/appveyor/branch AppVeyor build succeeded
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/jenkins/pr-merge This commit looks good
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@daviddias daviddias deleted the okr branch Apr 14, 2018

@wafflebot wafflebot bot removed the in progress label Apr 14, 2018

@daviddias daviddias changed the title from OKR.md to OKR 2018 Q2 Apr 14, 2018

@camelmasa camelmasa referenced this pull request Apr 19, 2018

Closed

Implement IPNS #209

This was referenced Jun 21, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment