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

gitserver: grpc: create FirstEverCommit RPC method and server implementation #62169

Conversation

ggilmore
Copy link
Contributor

@ggilmore ggilmore commented Apr 24, 2024

Part of #61689

This PR builds on #62165, and adds the FirstEverCommit new gRPC method along with a corresponding server implementation. The client implementation will be in a future PR.

Test plan

  • Unit tests

@cla-bot cla-bot bot added the cla-signed label Apr 24, 2024
@github-actions github-actions bot added team/product-platform team/source Tickets under the purview of Source - the one Source to graph it all labels Apr 24, 2024
Copy link
Contributor Author

ggilmore commented Apr 24, 2024

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @ggilmore and the rest of your teammates on Graphite Graphite

@ggilmore ggilmore force-pushed the 04-24-gitserver_grpc_create_firstevercommit_rpc_method_and_server_implementation branch from 98db01b to 7f3b667 Compare April 24, 2024 21:25
@ggilmore ggilmore marked this pull request as ready for review April 24, 2024 21:28
@ggilmore ggilmore force-pushed the 04-24-gitserver_port_firstevercommit_functionality_from_client_to_git_cli_backend branch from cf13ed9 to 26d91b0 Compare April 24, 2024 21:31
@ggilmore ggilmore force-pushed the 04-24-gitserver_grpc_create_firstevercommit_rpc_method_and_server_implementation branch 7 times, most recently from 2220186 to 1340dfb Compare April 24, 2024 21:46
cmd/gitserver/internal/server_grpc.go Show resolved Hide resolved
cmd/gitserver/internal/server_grpc_test.go Outdated Show resolved Hide resolved
internal/gitserver/v1/gitserver.proto Outdated Show resolved Hide resolved
@ggilmore ggilmore force-pushed the 04-24-gitserver_port_firstevercommit_functionality_from_client_to_git_cli_backend branch from 26d91b0 to fd32127 Compare April 25, 2024 19:08
@ggilmore ggilmore force-pushed the 04-24-gitserver_grpc_create_firstevercommit_rpc_method_and_server_implementation branch from 1340dfb to f6b6171 Compare April 25, 2024 19:09
@ggilmore ggilmore force-pushed the 04-24-gitserver_port_firstevercommit_functionality_from_client_to_git_cli_backend branch from fd32127 to 8e4b04d Compare April 25, 2024 19:12
@ggilmore ggilmore force-pushed the 04-24-gitserver_grpc_create_firstevercommit_rpc_method_and_server_implementation branch 3 times, most recently from 02ec89e to 5c223e4 Compare April 25, 2024 19:35
@ggilmore ggilmore force-pushed the 04-24-gitserver_port_firstevercommit_functionality_from_client_to_git_cli_backend branch from 8e4b04d to d794dc1 Compare April 25, 2024 23:14
@ggilmore ggilmore force-pushed the 04-24-gitserver_grpc_create_firstevercommit_rpc_method_and_server_implementation branch 2 times, most recently from 1f012eb to 9f0a175 Compare April 25, 2024 23:16
@ggilmore ggilmore force-pushed the 04-24-gitserver_port_firstevercommit_functionality_from_client_to_git_cli_backend branch from d794dc1 to 78388ea Compare April 26, 2024 17:20
@ggilmore ggilmore force-pushed the 04-24-gitserver_grpc_create_firstevercommit_rpc_method_and_server_implementation branch from 9f0a175 to 582385f Compare April 26, 2024 17:20
@ggilmore ggilmore force-pushed the 04-24-gitserver_port_firstevercommit_functionality_from_client_to_git_cli_backend branch from 78388ea to 1bbfc10 Compare April 26, 2024 18:59
@ggilmore ggilmore force-pushed the 04-24-gitserver_port_firstevercommit_functionality_from_client_to_git_cli_backend branch from 05a1f32 to 0dbd2b6 Compare April 30, 2024 05:19
@ggilmore ggilmore force-pushed the 04-24-gitserver_grpc_create_firstevercommit_rpc_method_and_server_implementation branch from 467450c to 6543606 Compare April 30, 2024 05:19
@ggilmore ggilmore force-pushed the 04-24-gitserver_port_firstevercommit_functionality_from_client_to_git_cli_backend branch from 0dbd2b6 to dd47983 Compare April 30, 2024 17:02
@ggilmore ggilmore force-pushed the 04-24-gitserver_grpc_create_firstevercommit_rpc_method_and_server_implementation branch from 6543606 to f258a65 Compare April 30, 2024 17:03
@ggilmore ggilmore force-pushed the 04-24-gitserver_port_firstevercommit_functionality_from_client_to_git_cli_backend branch from dd47983 to 9504034 Compare April 30, 2024 19:21
@ggilmore ggilmore force-pushed the 04-24-gitserver_grpc_create_firstevercommit_rpc_method_and_server_implementation branch from f258a65 to f5c873d Compare April 30, 2024 19:21
@ggilmore ggilmore force-pushed the 04-24-gitserver_port_firstevercommit_functionality_from_client_to_git_cli_backend branch from 9504034 to cd7c972 Compare April 30, 2024 19:21
@ggilmore ggilmore force-pushed the 04-24-gitserver_grpc_create_firstevercommit_rpc_method_and_server_implementation branch from f5c873d to 07f51e6 Compare April 30, 2024 19:22
@ggilmore ggilmore force-pushed the 04-24-gitserver_port_firstevercommit_functionality_from_client_to_git_cli_backend branch from cd7c972 to 0a3ffb5 Compare April 30, 2024 20:06
@ggilmore ggilmore force-pushed the 04-24-gitserver_grpc_create_firstevercommit_rpc_method_and_server_implementation branch from 07f51e6 to 9c9c844 Compare April 30, 2024 20:06
@ggilmore ggilmore force-pushed the 04-24-gitserver_port_firstevercommit_functionality_from_client_to_git_cli_backend branch from 0a3ffb5 to a8d5080 Compare April 30, 2024 20:34
@ggilmore ggilmore force-pushed the 04-24-gitserver_grpc_create_firstevercommit_rpc_method_and_server_implementation branch from 9c9c844 to 302550c Compare April 30, 2024 20:34
@ggilmore ggilmore force-pushed the 04-24-gitserver_port_firstevercommit_functionality_from_client_to_git_cli_backend branch from a8d5080 to df87c59 Compare April 30, 2024 22:29
@ggilmore ggilmore force-pushed the 04-24-gitserver_grpc_create_firstevercommit_rpc_method_and_server_implementation branch from 302550c to 241022e Compare April 30, 2024 22:29
Base automatically changed from 04-24-gitserver_port_firstevercommit_functionality_from_client_to_git_cli_backend to main May 1, 2024 04:14
@ggilmore ggilmore force-pushed the 04-24-gitserver_grpc_create_firstevercommit_rpc_method_and_server_implementation branch from 241022e to c4f47bf Compare May 1, 2024 04:16
Copy link
Contributor Author

ggilmore commented May 1, 2024

Merge activity

  • May 1, 12:16 AM EDT: Graphite rebased this pull request as part of a merge.
  • May 1, 12:32 AM EDT: @ggilmore merged this pull request with Graphite.

@ggilmore ggilmore merged commit 5af4aa1 into main May 1, 2024
13 checks passed
@ggilmore ggilmore deleted the 04-24-gitserver_grpc_create_firstevercommit_rpc_method_and_server_implementation branch May 1, 2024 04:32
ggilmore added a commit that referenced this pull request May 1, 2024
…ethod for FirstEverCommit (#62173)

Closes #61689

This PR switches the gitserver client implementation to use the new gRPC FirstCommitEver() method introduced in #62169 instead of shoving it through the `exec()` endpoint.

## Test plan

Unit tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed team/product-platform team/source Tickets under the purview of Source - the one Source to graph it all
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants