Skip to content
This repository was archived by the owner on Nov 12, 2025. It is now read-only.

Document RestateDeployment#629

Merged
jackkleeman merged 1 commit intomainfrom
restatedeployment
Jul 4, 2025
Merged

Document RestateDeployment#629
jackkleeman merged 1 commit intomainfrom
restatedeployment

Conversation

@jackkleeman
Copy link
Contributor

No description provided.

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Jun 11, 2025

Deploying documentation with  Cloudflare Pages  Cloudflare Pages

Latest commit: a9422ac
Status: ✅  Deploy successful!
Preview URL: https://39c8e050.documentation-beg.pages.dev
Branch Preview URL: https://restatedeployment.documentation-beg.pages.dev

View logs

Copy link
Contributor

@slinkydeveloper slinkydeveloper left a comment

Choose a reason for hiding this comment

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

looks awesome!

Comment on lines +54 to +59
env:
- name: PORT
value: "9080"
ports:
- containerPort: 9080
name: restate
Copy link
Contributor

Choose a reason for hiding this comment

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

If the user doesn't specify this, perhaps we can inject this ourselves?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

the operator defaults to 9080 anyway, is it necessary? i want to treat their pod spec as opaque as possible ideally

@jackkleeman jackkleeman merged commit 8a0fbb6 into main Jul 4, 2025
9 checks passed
@jackkleeman jackkleeman deleted the restatedeployment branch July 4, 2025 11:03
@github-actions github-actions bot locked and limited conversation to collaborators Jul 4, 2025
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.

2 participants