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.

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

Prod Release 02/11/23 #356

Merged
merged 7 commits into from
Nov 2, 2023
Merged

Prod Release 02/11/23 #356

merged 7 commits into from
Nov 2, 2023

Conversation

morgsmccauley
Copy link
Collaborator

This PR removes the V1/V2 toggle from the frontend, defaulting it back
to the EMS gateway. The gateway will eventually be updated to point to
V2 therefore 'migrating' the frontend to that version.
I've created a new env var because updating the existing would cause the
Lambdas to start writing to V2. Will add the following env variables to
GitHub so they are picked up in the action:
- `LAMBDA_DEV:HASURA_ENDPOINT_V2` -
https://queryapi-hasura-graphql-mainnet-vcqilefdcq-ew.a.run.app/
- `LAMBDA_PROD:HASURA_ENDPOINT_V2` -
https://queryapi-hasura-graphql-mainnet-24ktefolwq-ew.a.run.app/

Resolves #343
Previously, indexer executions were hardcoded to run with `isHistorical
= false`. This caused the `Current Block Height` to be incorrectly set
to the `Current Historical Block height`.

This PR updates Runner so that `isHistorical` is correctly set.
Previously, we would need to manually send the metric data to the main
thread, then write the metric there. This PR updates worker metrics so
it is done seamlessly in the background, metrics can be written as
normal regardless of whether in the main thread or not. This is achieved
by sending all metrics collected within a worker to the main thread, and
then aggregating them there.
@morgsmccauley morgsmccauley requested a review from a team as a code owner November 2, 2023 02:56
@morgsmccauley morgsmccauley merged commit 7f7ba0c into stable Nov 2, 2023
9 checks passed
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