Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: shed: test the nv18 migration #10296

Merged
merged 1 commit into from
Feb 17, 2023
Merged

Conversation

arajasek
Copy link
Contributor

@arajasek arajasek commented Feb 16, 2023

Related Issues

Updates the shed tester to work on the nv18 migration.

Proposed Changes

  • Update shed
  • Move the pre-migration up to start 30 minutes before the upgrade

Additional Info

Checklist

Before you mark the PR ready for review, please make sure that:

  • Commits have a clear commit message.
  • PR title is in the form of of <PR type>: <area>: <change being made>
    • example: fix: mempool: Introduce a cache for valid signatures
    • PR type: fix, feat, build, chore, ci, docs, perf, refactor, revert, style, test
    • area, e.g. api, chain, state, market, mempool, multisig, networking, paych, proving, sealing, wallet, deps
  • New features have usage guidelines and / or documentation updates in
  • Tests exist for new functionality or change in behavior
  • CI is green

@arajasek arajasek requested a review from a team as a code owner February 16, 2023 21:20
}

preMigration2Time := time.Since(startTime)
preMigrationTime := time.Since(startTime)
Copy link
Contributor

Choose a reason for hiding this comment

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

Does this mean we only need a single pre-migration now?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yessir -- this migration is easy-peasy, so a single pre-migration gets the job done.

Copy link
Contributor

@travisperson travisperson left a comment

Choose a reason for hiding this comment

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

LGTM; seems like maybe we should take sometime to make this shed command work off the upgrade schedule instead of having to change the upgrade heights for each upgrade.

@arajasek arajasek merged commit 1ec02c5 into release/v1.20.0 Feb 17, 2023
@arajasek arajasek deleted the asr/migration-test branch February 17, 2023 21:20
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants