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

go-libp2p 2022 Q4/2023 Q1 Roadmap #1806

Closed
p-shahi opened this issue Oct 7, 2022 · 11 comments
Closed

go-libp2p 2022 Q4/2023 Q1 Roadmap #1806

p-shahi opened this issue Oct 7, 2022 · 11 comments

Comments

@p-shahi
Copy link
Member

p-shahi commented Oct 7, 2022

🛣🗺 Roadmap document: https://github.com/libp2p/go-libp2p/blob/master/ROADMAP.md

children:

eta: 2023Q2

Current status of Roadmap

2022-10-20: Accepted. Internal go-libp2p stakeholders have aligned on this roadmap.

Description

This issue is intended to capture discussion around the go-libp2p 2022 Q4/2023 Q1 Roadmap.

Because GitHub doesn't have any feature to comment on Markdown files, please use this issue to leave any feedback, proposals, etc.

For internal discussions on Notion, please see: https://www.notion.so/pl-strflt/go-libp2p-22-Q4-23-Q1-Product-Roadmap-48bc31fd5b394a29812d6cda624fffa0

Roadmap drafts:

1st draft: #1784
2nd draft: #1831

@p-shahi p-shahi added the Epic label Oct 7, 2022
@BigLep
Copy link
Contributor

BigLep commented Oct 7, 2022

Related PRs:
#1784
#1804

@p-shahi
Copy link
Member Author

p-shahi commented Oct 7, 2022

Open questions/feedback from @BigLep

2022-10-17 update: In #1831 the following comments have been addressed


Open

@BigLep
Copy link
Contributor

BigLep commented Oct 18, 2022

@p-shahi : in looking at https://github.com/libp2p/go-libp2p/blob/master/ROADMAP.md fresh, here are the feedback items I have:

  1. I don't think we should have both "A.1⚡Handshakes at the Speed of Light" and "E.1 🧪 Future-proof testing - RTT test for Early Muxer Negotiation". I think E.1 is really just part of A.1. We're not going to mark A.1 done until we have E.1 as part of it. Specifically, I'm suggesting we inline E.1 into the A.1 description. It reduces clutter and makes it clear that we do this kind of testing for all new functionality we develop. It's not an effort that's tracked separately.

  2. "A.1⚡Handshakes at the Speed of Light" and "D.1 📺 Universal Browser Connectivity" are trains that are in motion that we're not going to slow down or divert resources from. We're committed to those and we're going to make them happen. That said, I think all remaining initiative gas should be poured into interoperability testing. I know we have to work in tandem with Laurent/Testground, but I want to get all of the go-libp2p side of the interoperability matrix filled in as soon as possible. I think that is worthy of its own milestone in go-libp2p. We can link out to "libp2p/test-plans" discussion, but there is specific work that go-libp2p needs to do. I'm sorry that wasn't clear, but that was what I was trying to get at here. Concretely, I think we should have a milestone for go-libp2p around "go-libp2 does interoperability testing across all important dimensions".

  3. I like that we're jumping into performance benchmarking soon. That said, I would do it after we lick interoperability testing (but before other items like you have listed currently).

  4. In terms of "End of Q4 (December)" items, I would invert the order from a priority regard. D.1 > C > B.1. At the minimum, I would put metrics (C) ahead of smart dialing (B.1).

  5. Compared to other roadmaps where someone sees target month, milestone name, and description, I think we're missing a bit here requiring someone to scroll down to the proper section to learn more. Not critical, but making this a link would help.

  6. Concerning discoverability of this roadmap, I see we added it to the README and pinned this issue (great - thanks). Will we also link from https://github.com/libp2p/specs/blob/master/ROADMAP.md ?

  7. Related to docs/roadmap #6, where will discuss libp2p's core tenants of security, stability, and performance?

If the above gets incorporated, I think we are good to say we're "internally aligned". I understand though if we need another round of back and forth. I'm happy to have a quick sync as well if that helps.

Thank you!

@p-shahi
Copy link
Member Author

p-shahi commented Oct 19, 2022

@BigLep please see: #1831
I had this open but as draft (without reviewers assigned) so it was likely you didn't see it during your pass through yesterday.
#1831 addresses 1, 3, 4, 5, and 7.

Regarding 2, I opted to move the testing and benchmarking items to the test-plans roadmap here. The priority of interop testing is highlighted in the go-libp2p Roadmap's Benchmarking and Testing section and made clear to the reader that it sits high at the list. Since interoperability is a shared concern, it makes sense to house it in the shared test-plans roadmap.

Regarding 6, I can add the links to the Spec roadmap too. Will have to do the same for Rust and JS.

@BigLep
Copy link
Contributor

BigLep commented Oct 19, 2022

ACK - sounds good @p-shahi . I also reviewed/approved #1831.

@marten-seemann
Copy link
Contributor

Closing this issue, since the roadmap was agreed upon and is merged. Potential changes to the roadmap can be suggested by creating a PR against that file.

@BigLep
Copy link
Contributor

BigLep commented Nov 21, 2022

Maybe there have been other conversations with @p-shahi , but the intent of having this open issue is so:

  1. Single place people can subscribe to get roadmap updates and ask questions
  2. Have a pinned issue for discoverability
  3. Have a single place to link update PRs to

@p-shahi
Copy link
Member Author

p-shahi commented Nov 21, 2022

Yes, let's keep this open.

@p-shahi p-shahi reopened this Nov 21, 2022
@marten-seemann
Copy link
Contributor

Sorry!

@p-shahi
Copy link
Member Author

p-shahi commented Dec 20, 2022

Discussion about removing TCP optimizations from roadmap: #1959

@p-shahi p-shahi removed the starmaps label Jan 31, 2023
@p-shahi p-shahi unpinned this issue Feb 2, 2023
@marten-seemann
Copy link
Contributor

Closing this issue since Q1 is over.

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

No branches or pull requests

3 participants