-
Notifications
You must be signed in to change notification settings - Fork 13
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
TT-10045 feat: Add parameter for key hashing algorithm #110
Conversation
Hi @jakub-bochenski thanks a lot for your contribution, it looks good! The only thing I'd like to modify is to set a default value so that the chart would be backward compatible with existing values.yaml, like this
|
umm.. it's set in the values file already |
@caroltyk bump? Default values should be specified in values.yaml
https://helm.sh/docs/chart_template_guide/functions_and_pipelines/#using-the-default-function |
I don't think the test failure is cause by my change
|
@jakub-bochenski yes, you are right. your changes will not break integration tests. let's re-run it after updating the pr. we recently merged improvements regarding bootstrapping jobs, which might be the reason for the failure. |
1e581ce
to
7018cc1
Compare
Thank you @jakub-bochenski . Thank you for working with us on this. The PR is approved. After some testings it can be merged for next release. |
7018cc1
to
becd6c1
Compare
Description
Parametrize the TYK_GW_HASHKEYFUNCTION value
Related Issue
Resolves #109
Motivation and Context
See #109
Test Coverage For This Change
Screenshots (if appropriate)
Types of changes
Checklist
master
!master
branch (left side). Also, it would be best if you started your change off our latestmaster
.