Skip to content

Script to easily launch the randomness service locally#324

Merged
aodhgan merged 13 commits intomasterfrom
gabriel/randomness-local-script
Feb 10, 2025
Merged

Script to easily launch the randomness service locally#324
aodhgan merged 13 commits intomasterfrom
gabriel/randomness-local-script

Conversation

@GabrielMartinezRodriguez
Copy link
Contributor

@GabrielMartinezRodriguez GabrielMartinezRodriguez commented Dec 26, 2024

Linked Issues

Description

This PR creates a script to easily launch the randomness service without needing to configure anything

Toggle Checklist

Checklist

Basics

  • B1. I have applied the proper label & proper branch name (e.g. norswap/build-system-caching).
  • B2. This PR is not so big that it should be split & addresses only one concern.
  • B3. The PR targets the lowest branch it can (ideally master).

Correctness

  • C1. Builds and passes tests.
  • C2. The code is properly parameterized & compatible with different environments.
  • C3. I have manually tested my changes & connected features.
  • C4. I have performed a thorough self-review of my code.

Architecture & Documentation

  • D1. I made it easy to reason locally about the code.
  • D2. All public-facing APIs & meaningful internal APIs are properly documented.
  • D3. If appropriate, the general architecture is documented.

@GabrielMartinezRodriguez GabrielMartinezRodriguez mentioned this pull request Dec 26, 2024
10 tasks
@GabrielMartinezRodriguez GabrielMartinezRodriguez changed the title Scr Script Dec 26, 2024
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Dec 26, 2024

Deploying happychain with  Cloudflare Pages  Cloudflare Pages

Latest commit: 04114c4
Status: ✅  Deploy successful!
Preview URL: https://b7ccd68b.happychain.pages.dev
Branch Preview URL: https://gabriel-randomness-local-scr.happychain.pages.dev

View logs

@GabrielMartinezRodriguez GabrielMartinezRodriguez changed the title Script Script to easily launch the randomness service locally Dec 26, 2024
@linear
Copy link

linear bot commented Dec 26, 2024

HAPPY-275 Script to easily launch the randomness service locally.

This script requires launching Anvil, deploying the random contract, setting the happy chain timestamp environment variable on the randomness service, compiling the transaction manager, executing its migrations, then compiling the randomness service and executing its migrations

@GabrielMartinezRodriguez GabrielMartinezRodriguez marked this pull request as ready for review December 26, 2024 18:30
@GabrielMartinezRodriguez GabrielMartinezRodriguez added the reviewing-1 Ready for, or undergoing first-line review label Dec 26, 2024
@norswap norswap added the merge-blocked-after-changes Ready to merge but waiting for downstack, after requested changes are made label Feb 9, 2025
@GabrielMartinezRodriguez
Copy link
Contributor Author

I was able to run make setup-local after the path changes in cb9808f which produced the log here https://gist.github.com/aodhgan/319866713a9582ea4a451f3789657720 There are NetworkErrors on basically every round when the service reaches steady state - just want to confirm this is expected behaviour? - since the drand beacon likely does not have the round the service is requesting

@aodhgan I sometimes receive this error as well. It happens due to how we handle requests to the drand network and how we initialize drand. Basically, the problem is that they return the error 'Too many requests.' However, this doesn’t cause a total system failure. I created this issue to address the problem:

https://linear.app/happychain/issue/HAPPY-359/improve-how-we-initialize-the-drand-start-round-and-how-we-fill-the

@GabrielMartinezRodriguez GabrielMartinezRodriguez force-pushed the gabriel/randomness-local-script branch from cb9808f to 8d30fe5 Compare February 10, 2025 11:59
@GabrielMartinezRodriguez GabrielMartinezRodriguez added merge-blocked Ready to merge, waiting for downstack and removed merge-blocked-after-changes Ready to merge but waiting for downstack, after requested changes are made labels Feb 10, 2025
@GabrielMartinezRodriguez GabrielMartinezRodriguez force-pushed the gabriel/randomness-local-script branch from af78c90 to aefc9c6 Compare February 10, 2025 15:31
@norswap norswap changed the base branch from gabriel/drand-service to graphite-base/324 February 10, 2025 17:16
@GabrielMartinezRodriguez GabrielMartinezRodriguez force-pushed the gabriel/randomness-local-script branch from aefc9c6 to abbe323 Compare February 10, 2025 18:40
@GabrielMartinezRodriguez GabrielMartinezRodriguez changed the base branch from graphite-base/324 to master February 10, 2025 18:41
@aodhgan aodhgan merged commit cd5bd34 into master Feb 10, 2025
3 checks passed
@aodhgan aodhgan deleted the gabriel/randomness-local-script branch February 10, 2025 19:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merge-blocked Ready to merge, waiting for downstack

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants