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

feat: add blutgang integration #87

Merged
merged 12 commits into from
Apr 26, 2024
Merged

feat: add blutgang integration #87

merged 12 commits into from
Apr 26, 2024

Conversation

jhkimqd
Copy link
Collaborator

@jhkimqd jhkimqd commented Apr 25, 2024

Description

This PR integrates blutgang to kurtosis-cdk as an L2 load balancer/cache.

Blutgang caches RPC calls and significantly reduces recurring queries to the RPC nodes.

Before cache
blutgang1

After cache
blutgang2

This PR sets up the blutgang service in front of the RPC endpoints as shown in the blutgang-config.toml file:

[sequencer]
url = "{{.l2_sequencer_url}}"
ws_url = "{{.l2_sequencer_ws_url}}"
# The maximum amount of time we can use this rpc in a row.
max_consecutive = 150
# Max amount of queries per second.
max_per_second = 200

[rpc]
url = "{{.l2_rpc_url}}"
ws_url = "{{.l2_ws_url}}"
# The maximum amount of time we can use this rpc in a row.
max_consecutive = 150
# Max amount of queries per second.
max_per_second = 200

[rpcpless]
url = "{{.l2_rpc_pless_url}}"
ws_url = "{{.l2_ws_pless_url}}"
# The maximum amount of time we can use this rpc in a row.
max_consecutive = 150
# Max amount of queries per second.
max_per_second = 200

The queries/results are cached inside the local db within the blutgang service.

@jhkimqd jhkimqd requested a review from leovct April 25, 2024 07:39
leovct
leovct previously requested changes Apr 25, 2024
Copy link
Member

@leovct leovct left a comment

Choose a reason for hiding this comment

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

lgtm, a few nits :)

params.yml Outdated Show resolved Hide resolved
params.yml Outdated Show resolved Hide resolved
params.yml Outdated Show resolved Hide resolved
params.yml Outdated Show resolved Hide resolved
cdk_blutgang.star Outdated Show resolved Hide resolved
@jhkimqd jhkimqd enabled auto-merge April 25, 2024 14:33
Copy link
Collaborator Author

@jhkimqd jhkimqd left a comment

Choose a reason for hiding this comment

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

.

@jhkimqd jhkimqd disabled auto-merge April 26, 2024 04:09
Copy link
Collaborator Author

@jhkimqd jhkimqd left a comment

Choose a reason for hiding this comment

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

.

@jhkimqd jhkimqd requested a review from leovct April 26, 2024 04:12
Copy link
Member

@leovct leovct left a comment

Choose a reason for hiding this comment

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

lgtm!!

main.star Outdated Show resolved Hide resolved
main.star Outdated Show resolved Hide resolved
@jhkimqd jhkimqd merged commit 7344bab into main Apr 26, 2024
4 of 5 checks passed
@jhkimqd jhkimqd deleted the jihwan/blutgang branch April 26, 2024 06:59
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

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