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(consensus): use iota-http for running the gRPC http server #5916

Conversation

VorobyevIlya
Copy link
Contributor

@VorobyevIlya VorobyevIlya commented Mar 13, 2025

Description of change

Porting from upstream MystenLabs/sui@c97668f as part of this PR

Links to any relevant issues

Be sure to reference any related issues by adding fixes #(issue).

Type of change

Choose a type of change, and delete any options that are not relevant.

  • Enhancement (a non-breaking change which adds functionality)

How the change has been tested

Describe the tests that you ran to verify your changes.

Make sure to provide instructions for the maintainer as well as any relevant configurations.

Change checklist

Tick the boxes that are relevant to your changes, and delete any items that are not.

  • I have followed the contribution guidelines for this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked that new and existing unit tests pass locally with my changes

@VorobyevIlya VorobyevIlya self-assigned this Mar 13, 2025
Copy link

vercel bot commented Mar 13, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

4 Skipped Deployments
Name Status Preview Comments Updated (UTC)
apps-backend ⬜️ Ignored (Inspect) Visit Preview Mar 18, 2025 4:51pm
apps-ui-kit ⬜️ Ignored (Inspect) Visit Preview 💬 Add feedback Mar 18, 2025 4:51pm
rebased-explorer ⬜️ Ignored (Inspect) Visit Preview Mar 18, 2025 4:51pm
wallet-dashboard ⬜️ Ignored (Inspect) Visit Preview 💬 Add feedback Mar 18, 2025 4:51pm

@iota-ci iota-ci added consensus Issues related to the Core Consensus team core-protocol labels Mar 13, 2025
@VorobyevIlya VorobyevIlya changed the title use sui-http for running the gRPC http server use iota-http for running the gRPC http server Mar 13, 2025
@VorobyevIlya VorobyevIlya force-pushed the consensus/feat/use-sui-http-for-running-the-gRPC-http-server branch from d05a221 to ccbd495 Compare March 13, 2025 16:56
@VorobyevIlya VorobyevIlya changed the title use iota-http for running the gRPC http server feat(consensus): use iota-http for running the gRPC http server Mar 13, 2025
@VorobyevIlya VorobyevIlya marked this pull request as ready for review March 13, 2025 19:20
@VorobyevIlya VorobyevIlya requested review from a team as code owners March 13, 2025 19:20
}
}

// TODO: improve iota-http to allow for providing a MakeService so that this can
Copy link
Contributor

Choose a reason for hiding this comment

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

Would it make sense to create an issue for this and reference it here ?

@VorobyevIlya VorobyevIlya force-pushed the consensus/feat/impl-ToSocketAddrs-for-Multiaddr branch from 368515c to f2ed986 Compare March 18, 2025 09:26
@VorobyevIlya VorobyevIlya requested a review from a team as a code owner March 18, 2025 09:26
@VorobyevIlya VorobyevIlya force-pushed the consensus/feat/use-sui-http-for-running-the-gRPC-http-server branch from 35289c8 to 7b7fc99 Compare March 18, 2025 09:29
@piotrm50 piotrm50 added this to the v0.11.x - protocol v5 milestone Mar 18, 2025
@VorobyevIlya VorobyevIlya removed the request for review from a team March 18, 2025 09:48
@VorobyevIlya VorobyevIlya force-pushed the consensus/feat/impl-ToSocketAddrs-for-Multiaddr branch from f2ed986 to 5abf2c5 Compare March 18, 2025 13:10
Base automatically changed from consensus/feat/impl-ToSocketAddrs-for-Multiaddr to develop March 18, 2025 16:46
@VorobyevIlya VorobyevIlya force-pushed the consensus/feat/use-sui-http-for-running-the-gRPC-http-server branch from 7b7fc99 to fce5547 Compare March 18, 2025 16:50
@VorobyevIlya VorobyevIlya merged commit d297251 into develop Mar 18, 2025
36 checks passed
@VorobyevIlya VorobyevIlya deleted the consensus/feat/use-sui-http-for-running-the-gRPC-http-server branch March 18, 2025 19:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
consensus Issues related to the Core Consensus team core-protocol
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants