Skip to content

Conversation

@bennostein
Copy link
Contributor

Putting this up for review now; should work as is if you follow instructions I added to ./examples/hackernews/README.md.

There's still some manual configuration changes required to scale out the followers of the Skip service: creating a corresponding skip-ingress-X service fronting each pod of the statefulset, and adding the routing logic to the reverse proxy.

The skip-ingress-X services can easily be scripted, but using the current setup changing that routing logic requires a HAProxy restart. I'm working on a more dynamic config for haproxy using maps, but that's still a WIP.

@bennostein bennostein force-pushed the kubernetes-example branch 2 times, most recently from c551270 to df3af00 Compare June 2, 2025 13:49
@bennostein bennostein force-pushed the kubernetes-example branch 2 times, most recently from 60be99c to 8d57055 Compare June 2, 2025 16:58
@bennostein bennostein marked this pull request as ready for review June 2, 2025 16:58
@bennostein
Copy link
Contributor Author

Marked as ready for review: the app can now scale up/down to X skip instances with just a kubectl scale --replicas=$X (X >= 2, since the leader is included in the count).

Will write up a detailed blog post/tutorial but following the steps I added to the example/hackernews/README.md should work for now.

@bennostein bennostein force-pushed the kubernetes-example branch from 8d57055 to 161213e Compare June 2, 2025 17:14
Copy link
Contributor

@hubyrod hubyrod left a comment

Choose a reason for hiding this comment

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

This is precision work 👌
One little change in the name of the container, a couple of suggestion for our local tooling (adding explicitly bash to the code section in markdown). Up to you :)

@bennostein bennostein merged commit 87c48df into SkipLabs:main Jun 3, 2025
5 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