Skip to content

snapshot: add 'snapshot summary', 'snapshot test' #888

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

Merged
merged 3 commits into from
Nov 25, 2022

Conversation

bobheadxi
Copy link
Member

@bobheadxi bobheadxi commented Nov 25, 2022

These commands are tooling intended for use in on-prem-to-Cloud data migrations:

  • src snapshot summary will generate a summary snapshot of an instance. APIs used here must be available in Sourcegraph 3.20 and later - see RFC 760
  • src snapshot test will use the summary and other health indicators to try and show that the instance is healthy and in an expected state post-migration. APIs used here can be in the latest releases of Sourcegraph, because we will run a multi-version upgrade in Cloud to the latest version.
    • The instance health checking components of this can in the future be re-used in a different command

Closes https://github.com/sourcegraph/customer/issues/1612 , closes https://github.com/sourcegraph/customer/issues/1619

Also part of this tooling eventually will be other user-facing utilities like:

  1. database dumping for common deployment setups
  2. database dump and summary upload utilities to Sourcegraph-controlled GCS buckets

See https://github.com/sourcegraph/customer/issues/1525 for more details

Test plan

image

plus a wee bit of unit tests.

@bobheadxi bobheadxi requested a review from michaellzc November 25, 2022 20:49
@bobheadxi bobheadxi force-pushed the src-snapshot-summary-test branch from 9f6bf3d to 7ec986f Compare November 25, 2022 20:50
@bobheadxi bobheadxi requested a review from a team November 25, 2022 21:15
@bobheadxi bobheadxi merged commit 4d2957c into main Nov 25, 2022
@bobheadxi bobheadxi deleted the src-snapshot-summary-test branch November 25, 2022 22:27
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.

2 participants