Skip to content

Update default config in EC2 construct for Restate 1.3.0#73

Merged
pcholakov merged 2 commits intomainfrom
update-default-ec2-server-config
Apr 12, 2025
Merged

Update default config in EC2 construct for Restate 1.3.0#73
pcholakov merged 2 commits intomainfrom
update-default-ec2-server-config

Conversation

@pcholakov
Copy link
Collaborator

This change updates the default baseline restate-server config deployed by the EC2 construct to the following:

roles = [
    "worker",
    "admin",
    "metadata-server",
    "log-server",
]
node-name = "restate-0"
cluster-name = "Restate"
default-num-partitions = 4
rocksdb-total-memory-size = "512 MB"

[admin]
bind-address = "0.0.0.0:9070"

[admin.query-engine]
memory-size = "256.0 MiB"

[ingress]
bind-address = "0.0.0.0:8080"

This fixes #72

@pcholakov pcholakov merged commit c7623e3 into main Apr 12, 2025
2 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Apr 12, 2025
Copy link
Collaborator

@tillrohrmann tillrohrmann left a comment

Choose a reason for hiding this comment

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

Thanks for updating the cdk construct @pcholakov. Is it possible to not set the roles explicitly? Otherwise, we will have a bit of trouble when splitting the http ingress from the worker role.

Comment on lines 356 to 361
`roles = [`,
` "worker",`,
` "admin",`,
` "metadata-store",`,
` "metadata-server",`,
` "log-server",`,
`]`,
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can we not set the roles explicitly? Otherwise it's gonna be problematic once we enable the http-ingress role by default because then this deployment won't run the ingress once we release 1.4.0.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

That's a reasonable thing to do! I'll update for the release.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Default config is not compatible with Restate 1.3.0

2 participants