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.

Sign up for GitHub

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

Jump to bottom

Warmup initial scale doesn't work as intended #222

Open
leokondrashov opened this issue Jun 28, 2023 · 3 comments
Open

Warmup initial scale doesn't work as intended #222

leokondrashov opened this issue Jun 28, 2023 · 3 comments
Assignees
@Lightxyz
Labels
good first issue Good for newcomers

Comments

@leokondrashov
Copy link
Contributor

Version & Branch

main

Expected Behaviour

When starting invocations, scales of functions should be based on the warmup profiler estimations.

Actual Behaviour

Since deployment takes some time, most of the instances of functions created as initial scale are terminated.

Steps to Reproduce the Problem

  1. Run any trace with >100 functions

Additional Info

This leads to necessity to increase warmup stage length and run for longer.
@leokondrashov leokondrashov added the good first issue Good for newcomers label Jun 28, 2023
@Lightxyz
Copy link
Contributor

Lightxyz commented Sep 7, 2023

Is there anyone working on this issue?

@leokondrashov
Copy link
Contributor Author

I don't think so. Feel free to take it.

@Lightxyz
Copy link
Contributor

Lightxyz commented Sep 8, 2023

I don't think so. Feel free to take it.

I'll try to fix it, as I need to run experiments with 1000 functions and I doubt it would currently work.

@Lightxyz Lightxyz linked a pull request Sep 10, 2023 that will close this issue
Initial scale kept for first minute of experiment #261
Closed
@Lightxyz Lightxyz self-assigned this Sep 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Lightxyz Lightxyz

Labels
good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Initial scale kept for first minute of experiment vhive-serverless/invitro
2 participants
@leokondrashov @Lightxyz