-
Notifications
You must be signed in to change notification settings - Fork 831
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
[Proof of concept] Horizontal Fleet Autoscaler prototype #336
Conversation
Build Failed 😱 Build Id: 29bf9a92-a6ae-4cc9-84b6-3065dc0e86ff Build Logs
|
I dont insist to do it this way, but I had an idea and it fit nicely with the existing code imo. |
Build Failed 😱 Build Id: 4ffddee8-e25b-4917-aaa3-5b59abedcca5 Build Logs
|
Build Failed 😱 Build Id: 569793c2-78e9-4916-8465-6dbbf6d690f1 Build Logs
|
Build Failed 😱 Build Id: 2423b7fb-3037-41bf-9485-0a7176cf7bb7 Build Logs
|
@markmandel @Kuqd I don't understand what I did wrong, could you give me a hint why the build is failing now, please? BTW, I hope you don't mind that I keep pushing to this PR, but I want to see it green :) |
Looks like |
Build Failed 😱 Build Id: 86e8fcc0-55be-4719-8469-1665606a9abf Build Logs
|
Updated branch, maybe that was the problem... 🙏 |
Build Failed 😱 Build Id: f323a0bb-9542-491e-bcc7-55393633eca5 Build Logs
|
Build Succeeded 👏 Build Id: 82bbdf9e-a281-40fd-8cb9-b0a6e99b6277 The following development artifacts have been built, and will exist for the next 30 days:
(experimental) To install this version:
|
Are we able to close this PR now? Just doing some cleanup on the repo 😄 |
@markmandel can we wait until the other solution is validated? |
Sure 😄 just like keeping WIP stuff out of the PR queue - makes life easier for reviewers to know what they need to keep an eye on. Or at least, we should tag this with a [WIP] so it's easy to differentiate, and people know it's not for review yet. |
I feel this is not needed anymore. |
Quick working prototype.
As proposed in #334
It's functional and can be used for testing (or development) until the better, separated controller solution, is implemented)
Not ready for review.