feat: add keda autoscaler for container and application service (only…#2265
feat: add keda autoscaler for container and application service (only…#2265
Conversation
|
Qovery can create a Preview Environment for this PR.
This comment has been generated from Qovery AI 🤖.
|
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## staging #2265 +/- ##
===========================================
- Coverage 47.73% 47.42% -0.32%
===========================================
Files 1254 1265 +11
Lines 22717 23317 +600
Branches 6635 6869 +234
===========================================
+ Hits 10844 11057 +213
- Misses 9808 10131 +323
- Partials 2065 2129 +64
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
041a81e to
aed396c
Compare
bf0d6ba to
9d0dfd3
Compare
RemiBonnet
left a comment
There was a problem hiding this comment.
Thanks Pierre 🙏
I added a few comments, but I think it would be better if I rewrite this part
The logic feels too complex for what are just small settings, and it currently affects some sensitive parts of the codebase. I think this logic should be isolated, as it is now, it’s hard to maintain and risky. Let’s talk about it tomorrow!
| min_running_instances: number | ||
| max_running_instances: number | ||
|
|
||
| // Autoscaling mode selection |
There was a problem hiding this comment.
Can we have an specific interface for keda and extends this one? (you can probably extends from the api doc)
7ec2455 to
7a94741
Compare
c57c64a to
900321c
Compare
900321c to
5dbde4f
Compare
5dbde4f to
ddbdaa0
Compare
| @@ -0,0 +1,88 @@ | |||
| import type { ScaledObjectStatusDto } from 'qovery-ws-typescript-axios/dist/api' | |||
There was a problem hiding this comment.
@pggb25 This comment is still relevant, could you please apply @RemiBonnet's suggestion?
| min_running_instances: number | ||
| max_running_instances: number | ||
|
|
||
| // Autoscaling mode selection |
|
🎉 This PR is included in version 1.272.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
#2265) * feat(keda): add support for keda autoscaler * fix: handle comments: use pattern matching * fix: bug * fix: test * fix: fix test * feat: handle UX feedback * remove dead code * handle comments
… qovery admin)
Summary
This PR allow to configure KEDA autoscaling for container and application service
Issue:
Screenshots / Recordings
Testing
yarn testoryarn test -u(if you need to regenerate snapshots)yarn formatyarn lintPR Checklist
.cursor/rules)feat(service): add new Terraform service) - required for semantic-release