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

feat: update kubo to v0.19 #4655

Merged
merged 4 commits into from
Apr 20, 2023
Merged

Conversation

gfanton
Copy link
Member

@gfanton gfanton commented Apr 14, 2023

@gfanton gfanton requested a review from a team as a code owner April 14, 2023 13:20
@gfanton gfanton self-assigned this Apr 14, 2023
libp2p_relayv1.WithResources(ressources),
// libp2p_relayv1.WithACL(acl)), // not use for now
_, err = libp2p_relayv2.New(host,
// disable limits for now to have an equivalent off a relay v1
Copy link
Collaborator

Choose a reason for hiding this comment

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

Should be "equivalent of"

@gfanton gfanton changed the title feat: update kubo to v0.19 WIP: update kubo to v0.19 Apr 17, 2023
@gfanton gfanton force-pushed the feat/update-kubo-19 branch 4 times, most recently from 20d9730 to 890f95c Compare April 17, 2023 14:24
@gfanton gfanton requested a review from a team as a code owner April 17, 2023 14:38
@gfanton gfanton force-pushed the feat/update-kubo-19 branch 4 times, most recently from 96e28fc to 29c6bcf Compare April 17, 2023 15:16
@codecov
Copy link

codecov bot commented Apr 17, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.01 ⚠️

Comparison is base (139062b) 31.55% compared to head (ddde847) 31.54%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4655      +/-   ##
==========================================
- Coverage   31.55%   31.54%   -0.01%     
==========================================
  Files         436      436              
  Lines       22859    22854       -5     
  Branches     1842     1842              
==========================================
- Hits         7213     7210       -3     
+ Misses      15106    15104       -2     
  Partials      540      540              
Flag Coverage Δ
go.unittests 25.17% <0.00%> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
go/cmd/berty/p2p.go 23.23% <0.00%> (ø)
go/cmd/rdvp/main.go 0.00% <0.00%> (ø)
go/cmd/welcomebot/main.go 1.46% <0.00%> (ø)
go/pkg/bertymessenger/api.go 32.81% <0.00%> (-0.21%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@jefft0 jefft0 requested review from jefft0 and removed request for a team April 19, 2023 08:34
@gfanton gfanton force-pushed the feat/update-kubo-19 branch 5 times, most recently from 6e70f9b to e260a62 Compare April 19, 2023 20:38
Copy link
Contributor

@D4ryl00 D4ryl00 left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -589,8 +589,8 @@ func (m *Manager) setupIPFSConfig(cfg *ipfs_cfg.Config) ([]libp2p.Option, error)
// enable mdns
cfg.Discovery.MDNS.Enabled = false

// disable managed auto relay
cfg.Swarm.RelayClient.Enabled = ipfs_cfg.False
// make sure auto relay is enable
Copy link
Contributor

Choose a reason for hiding this comment

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

Why do you change autorelay to true?

Copy link
Member Author

Choose a reason for hiding this comment

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

that's a typo error, thanks !

@@ -620,8 +617,8 @@ func (m *Manager) setupIPFSConfig(cfg *ipfs_cfg.Config) ([]libp2p.Option, error)

// @NOTE(gfanton): disable quic transport so we can init a custom transport
// with reusport disable
cfg.Swarm.Transports.Network.QUIC = ipfs_cfg.False
p2popts = append(p2popts, libp2p.Transport(quict.NewTransport, quict.DisableReuseport()))
cfg.Swarm.Transports.Network.QUIC = ipfs_cfg.True
Copy link
Contributor

Choose a reason for hiding this comment

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

Is it ok the enable QUIC? Is reusePort disabled?

Copy link
Member Author

Choose a reason for hiding this comment

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

that was a mistake, thanks !

Signed-off-by: gfanton <8671905+gfanton@users.noreply.github.com>
Signed-off-by: gfanton <8671905+gfanton@users.noreply.github.com>
Signed-off-by: gfanton <8671905+gfanton@users.noreply.github.com>
Signed-off-by: gfanton <8671905+gfanton@users.noreply.github.com>
@gfanton gfanton changed the title WIP: update kubo to v0.19 feat: update kubo to v0.19 Apr 20, 2023
@gfanton gfanton merged commit 961b730 into berty:master Apr 20, 2023
@gfanton gfanton deleted the feat/update-kubo-19 branch April 20, 2023 19:05
@github-actions
Copy link

🎉 This PR is included in version 2.466.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
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants