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

Jetstream Autoscaling Guide #703

Merged
merged 43 commits into from
Jun 17, 2024
Merged

Jetstream Autoscaling Guide #703

merged 43 commits into from
Jun 17, 2024

Conversation

Bslabe123
Copy link
Collaborator

No description provided.

@liurupeng
Copy link
Collaborator

/hold please don't merge before I take a look, thanks

@liurupeng
Copy link
Collaborator

I feel we have added a bunch of files in order to use terraform to deploy the resources for HPA instead of deploying the pod monitoring and cmsa with yamls, what's the benefits going with this approach? @Bslabe123

@liurupeng
Copy link
Collaborator

/gcbrun

@Bslabe123
Copy link
Collaborator Author

/gcbrun

@Bslabe123 Bslabe123 merged commit c62d2ba into main Jun 17, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rsgowman rsgowman rsgowman left review comments

@achandrasekar achandrasekar achandrasekar left review comments

@annapendleton annapendleton annapendleton approved these changes

@liurupeng liurupeng liurupeng approved these changes

@ahg-g ahg-g Awaiting requested review from ahg-g ahg-g is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@Bslabe123 @liurupeng @achandrasekar @rsgowman @annapendleton