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

14 fix: slow start aggression (#9031) (#9087) #9132

Merged
merged 1 commit into from
Feb 6, 2024

Conversation

nfuden
Copy link
Contributor

@nfuden nfuden commented Feb 2, 2024

  • fix: slow start aggression

  • create changelog

  • update version for the changelog

  • update runtime key to be configurable per cluster

* fix: slow start aggression

* create changelog

* update version for the changelog

* update runtime key to be configurable per cluster

Co-authored-by: sopida-chotwanwirach <[email protected]>
@github-actions github-actions bot added the keep pr updated signals bulldozer to keep pr up to date with base branch label Feb 2, 2024
@solo-changelog-bot
Copy link

Issues linked to changelog:
solo-io#9031

@nfuden nfuden changed the title fix: slow start aggression (#9031) (#9087) 14 fix: slow start aggression (#9031) (#9087) Feb 5, 2024
@@ -167,16 +169,23 @@ func configureLeastRequestLb(out *envoy_config_cluster_v3.Cluster, cfg *v1.LoadB
}
}

func toSlowStartConfig(cfg *v1.LoadBalancerConfig_SlowStartConfig) *envoy_config_cluster_v3.Cluster_SlowStartConfig {
func toSlowStartConfig(clusterInfo *envoy_config_cluster_v3.Cluster, cfg *v1.LoadBalancerConfig_SlowStartConfig) *envoy_config_cluster_v3.Cluster_SlowStartConfig {

Choose a reason for hiding this comment

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

projects/gloo/pkg/plugins/loadbalancer/plugin.go

Copy link
Contributor Author

Choose a reason for hiding this comment

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

?

@nfuden nfuden added work in progress signals bulldozer to keep pr open (don't auto-merge) and removed work in progress signals bulldozer to keep pr open (don't auto-merge) labels Feb 6, 2024
@soloio-bulldozer soloio-bulldozer bot merged commit a38c127 into v1.14.x Feb 6, 2024
15 checks passed
@soloio-bulldozer soloio-bulldozer bot deleted the back/1.14-slow branch February 6, 2024 16:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
keep pr updated signals bulldozer to keep pr up to date with base branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants