Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

gitserver: Simplify clone and update #61860

Merged
merged 1 commit into from
Apr 18, 2024

Conversation

eseliger
Copy link
Member

@eseliger eseliger commented Apr 13, 2024

Those two used to have separate mechanisms for locking and for making sure the operation keeps going in the background.

This PR unifies those two where possible.

Test plan:

The clone test covers this pretty well, and I made a few adjustments there to cover the new behavior.
Also, E2E tests will cover clone failures.

@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 13, 2024
@eseliger eseliger force-pushed the es/04-13-gitservermovesearchobservability branch from b3406cc to e58bcc0 Compare April 13, 2024 23:54
@eseliger eseliger force-pushed the es/04-13-gitserversimplifycloneandupdate branch from bfe36b0 to 10821e8 Compare April 13, 2024 23:54
@eseliger eseliger force-pushed the es/04-13-gitservermovesearchobservability branch from e58bcc0 to 0ec6a10 Compare April 14, 2024 00:07
@eseliger eseliger force-pushed the es/04-13-gitserversimplifycloneandupdate branch 2 times, most recently from 234d9cb to 0f26c76 Compare April 14, 2024 00:30
@eseliger eseliger force-pushed the es/04-13-gitservermovesearchobservability branch from 0ec6a10 to d63cb3c Compare April 15, 2024 13:43
@eseliger eseliger force-pushed the es/04-13-gitserversimplifycloneandupdate branch from 0f26c76 to 266ff89 Compare April 15, 2024 13:43
@eseliger eseliger marked this pull request as ready for review April 15, 2024 13:46
@eseliger eseliger requested a review from a team April 15, 2024 13:46
@eseliger eseliger force-pushed the es/04-13-gitservermovesearchobservability branch from d63cb3c to c70f9bb Compare April 15, 2024 17:20
@eseliger eseliger force-pushed the es/04-13-gitserversimplifycloneandupdate branch from 266ff89 to f1dd99d Compare April 15, 2024 17:21
@eseliger eseliger force-pushed the es/04-13-gitservermovesearchobservability branch from c70f9bb to 26fe870 Compare April 15, 2024 19:18
@eseliger eseliger force-pushed the es/04-13-gitserversimplifycloneandupdate branch from f1dd99d to 539175c Compare April 15, 2024 19:18
@eseliger eseliger force-pushed the es/04-13-gitservermovesearchobservability branch from 26fe870 to 1f6abf3 Compare April 17, 2024 14:20
@eseliger eseliger force-pushed the es/04-13-gitserversimplifycloneandupdate branch from 539175c to 9e00e68 Compare April 17, 2024 14:20
@eseliger eseliger force-pushed the es/04-13-gitservermovesearchobservability branch from 1f6abf3 to 3b622df Compare April 17, 2024 20:36
@eseliger eseliger force-pushed the es/04-13-gitserversimplifycloneandupdate branch from 9e00e68 to 4bac598 Compare April 17, 2024 20:37
@eseliger eseliger force-pushed the es/04-13-gitservermovesearchobservability branch from 3b622df to 12a8301 Compare April 18, 2024 14:53
@eseliger eseliger force-pushed the es/04-13-gitserversimplifycloneandupdate branch from 4bac598 to 49fa32a Compare April 18, 2024 14:53
@eseliger eseliger force-pushed the es/04-13-gitservermovesearchobservability branch from 12a8301 to fc03429 Compare April 18, 2024 20:40
Base automatically changed from es/04-13-gitservermovesearchobservability to main April 18, 2024 21:21
Those two used to have separate mechanisms for locking and for making sure the operation keeps going in the background.

This PR unifies those two where possible.

Test plan:

The clone test covers this pretty well, and I made a few adjustments there to cover the new behavior.
Also, E2E tests will cover clone failures.
@eseliger eseliger force-pushed the es/04-13-gitserversimplifycloneandupdate branch from 49fa32a to a3cde3f Compare April 18, 2024 21:22
Copy link
Member Author

eseliger commented Apr 18, 2024

Merge activity

  • Apr 18, 5:23 PM EDT: Graphite rebased this pull request after merging its parent, because this pull request is set to merge when ready.
  • Apr 18, 5:34 PM EDT: @eseliger merged this pull request with Graphite.

@eseliger eseliger merged commit 73e84a4 into main Apr 18, 2024
@eseliger eseliger deleted the es/04-13-gitserversimplifycloneandupdate branch April 18, 2024 21:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
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.

2 participants