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

Implement in-place pod request scaling #12

Merged
merged 6 commits into from
Jun 2, 2024
Merged

Implement in-place pod request scaling #12

merged 6 commits into from
Jun 2, 2024

Conversation

ctrox
Copy link
Owner

@ctrox ctrox commented Jun 1, 2024

If enabled, we make use of the (still alpha) feature gate InPlacePodVerticalScaling to change the pod requests to a minimum on scaling down and increasing them to the original value on restore. This has been marked as experimental and also needs to be enabled by setting a flag on the manager and of course the feature flag needs to be enabled.

TODO:

  • The additional rbac rules should only be applied when enabling the feature
  • A diff check for the ttrpc gen should be added to the ci

ctrox added 3 commits June 1, 2024 18:27
Instead of serving the metrics over HTTP, we now use ttrpc and implement
a more generic "shim service" that can be used for different purposes
than just metrics.
the shim ttrpc service now exposes a SubscribeStatus function which the
manager subscribes to for receiving zeropod status updates, such as when
checkpointing and restoring.
If enabled, we make use of the (still alpha) feature gate
InPlacePodVerticalScaling to change the pod requests to a minimum on
scaling down and increasing them to the original value on restore. This
has been marked as experimental and also needs to be enabled by setting
a flag on the manager and of course the feature flag needs to be
enabled.
@ctrox ctrox force-pushed the resource-requests branch 2 times, most recently from 9552aae to 3eb988c Compare June 2, 2024 08:08
@ctrox ctrox force-pushed the resource-requests branch from 3eb988c to 2b896f9 Compare June 2, 2024 09:16
ctrox added 2 commits June 2, 2024 11:36
This adds a kustomize component to enable to in-place-scaling feature on
demand. It is enabled by default for the kind overlay for local and e2e
testing.
@ctrox ctrox merged commit cf75cc7 into main Jun 2, 2024
4 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.

1 participant