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

Update scaling-with-hpa.md #192

Closed
wants to merge 1 commit into from
Closed

Conversation

asteurer
Copy link
Contributor

Removed the duplicate hpa example yaml, and updated the yaml to reflect an upgrade done on PR spinkube/spin-operator#268

Removed the duplicate hpa example yaml, and updated the yaml to reflect an upgrade done on PR spinkube/spin-operator#268

Signed-off-by: asteurer <[email protected]>
@calebschoepp calebschoepp self-requested a review July 10, 2024 16:55
@bacongobbler
Copy link
Contributor

bacongobbler commented Jul 10, 2024

Thanks @asteurer.

The documentation is intended to reflect the latest version of Spinkube. Because spinkube/spin-operator#268 has not yet made it into a release, we should hold off on publishing this until a new version of Spinkube (with an updated version of spin-operator) has been released. Otherwise users will try to run these commands and will fail because the quickstart guide currently suggests that the user should install spin-operator v0.2.0.

@@ -167,7 +124,7 @@ metadata:
spec:
image: ghcr.io/spinkube/spin-operator/cpu-load-gen:20240311-163328-g1121986
executor: containerd-shim-spin
enableAutoscaling: true
replicas: 1
Copy link
Contributor

@bacongobbler bacongobbler Jul 10, 2024

Choose a reason for hiding this comment

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

With minReplicas and maxReplicas set, isn't this redundant?

EDIT: raised feedback in the upstream PR.

@bacongobbler
Copy link
Contributor

New location: spinkube/spin-operator#270

@asteurer asteurer closed this Aug 15, 2024
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