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.

Sign up for GitHub

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

Jump to bottom

chore: update deployment start #1502

Merged
merged 2 commits into from
Dec 6, 2023
Merged

chore: update deployment start #1502

merged 2 commits into from
Dec 6, 2023

Conversation

ra0x3
Copy link
Contributor

@ra0x3 ra0x3 commented Dec 6, 2023 •
edited
Loading

Thanks for opening a PR with the Fuel Indexer project. Please review the Checklist below and ensure you've completed all of the necessary steps to make this PR review as painless as possible.

Checklist

  • Ensure your top-level commit message is in line with our contributor guidelines.
  • Please add proper labels.
  • If there is an issue associated with this PR, please link the issue (right-hand sidebar)
  • If there is not an issue associated with this PR, add this PR to the "Fuel Indexer" project (right-hand sidebar)
  • Please allow Codeowners at least 24 hours to do a first-pass review.
  • Please add thoroughly detailed testing steps below.
  • Please keep your Changelog message short and sweet.

Description

  • Updates our deployment start to delay a bit (so as to not trigger rate limiting)
  • Also bumps the block page size

Testing steps

  • QA Results attached below

Changelog

  • chore: update deployment start

@ra0x3 ra0x3 self-assigned this Dec 6, 2023
@ra0x3 ra0x3 force-pushed the rashad/update-deployment-start branch from 4c1048f to 68a8fee Compare December 6, 2023 13:37
@ra0x3
Copy link
Contributor Author

ra0x3 commented Dec 6, 2023

QA Results

Rashads-MacBook-Air.local-2023-12-06.indexer-qa.txt

@ra0x3 ra0x3 marked this pull request as ready for review December 6, 2023 15:07
lostman
lostman approved these changes Dec 6, 2023
View reviewed changes
Copy link
Contributor

@lostman lostman left a comment

Choose a reason for hiding this comment

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

🛳️

deekerno
deekerno approved these changes Dec 6, 2023
View reviewed changes
Copy link
Contributor

@deekerno deekerno left a comment

Choose a reason for hiding this comment

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

🚄

@ra0x3 ra0x3 merged commit 8dc3654 into develop Dec 6, 2023
19 checks passed
@ra0x3 ra0x3 deleted the rashad/update-deployment-start branch December 6, 2023 15:10
@ra0x3 ra0x3 mentioned this pull request Dec 6, 2023
Merged
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lostman lostman lostman approved these changes

@deekerno deekerno deekerno approved these changes

Assignees

@ra0x3 ra0x3

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ra0x3 @lostman @deekerno