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

Add support to set KEDA_HTTP_DEFAULT_TIMEOUT in metrics server #181

Merged
merged 1 commit into from
Aug 30, 2021
Merged

Add support to set KEDA_HTTP_DEFAULT_TIMEOUT in metrics server #181

merged 1 commit into from
Aug 30, 2021

Conversation

JorTurFer
Copy link
Member

@JorTurFer JorTurFer commented Aug 29, 2021

This PR adds support to set KEDA_HTTP_DEFAULT_TIMEOUT in metrics-server by the same way that we are using for the operator.
I have created the PR following these steps, https://github.com/kedacore/charts/blob/master/CONTRIBUTING.md. Let me know if it's correct or I have to do/undo anything else :)

Checklist

Fixes #180

Signed-off-by: jorturfer <jorge_turrado@hotmail.es>
@JorTurFer JorTurFer changed the title Add support to set KEDA_HTTP_DEFAULT_TIMEOUT in metrics server Add support to set KEDA_HTTP_DEFAULT_TIMEOUT in metrics server Aug 29, 2021
Copy link
Member

@tomkerkhove tomkerkhove left a comment

Choose a reason for hiding this comment

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

Can you revert the new version please?

@JorTurFer
Copy link
Member Author

Can you revert the new version please?

do you mean all the changes but keda/templates/22-metrics-deployment.yaml, right?

@tomkerkhove
Copy link
Member

Yes, thank you!

@JorTurFer
Copy link
Member Author

sure! :)
I'll do it during today

@JorTurFer
Copy link
Member Author

Done!

@tomkerkhove tomkerkhove added this to the KEDA Core v2.5.0 milestone Aug 30, 2021
@tomkerkhove
Copy link
Member

Thank you!

@tomkerkhove tomkerkhove merged commit 93e2fdc into kedacore:master Aug 30, 2021
@JorTurFer JorTurFer deleted the timeout branch August 30, 2021 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support to set KEDA_HTTP_DEFAULT_TIMEOUT in metrics server
2 participants