Closed mgnisia closed 3 months ago
Thank you @mgnisia for your contribution. We will review this shortly.
@caroltyk Hi Carol, could you maybe take another look? that would be awesome.
Hi @mgnisia The PR looks good to me but I'll pass this to the engineers and QA for testing first. Let me update here when it is planned for release.
Thanks for the update 🙂 will revert the version change beginning of next week 👍🏻
Good morning @buraksekili revert the changes. It would be awesome if you cloud take another look. 👀
LGTM! @buraksekili we should proceed with merging this PR Thanks,
@buraksekili can you foresee when this PR will be merged?
@mgnisia thanks for your contribution 🏅 your contribution is going to be released within the next release.
Description
Add
annotations
variable to the values.yaml file of all relevant helm chartsRelated Issue
Motivation and Context
To support an automated reloading of the gateway or other components of the tyk stack a common approach is to utilize tools like reloader. Currently the helm charts don't support any kind of annotations on the deployment level.
Test Coverage For This Change
I have been templating the helm charts on my machine (
helm template
) with:The templating worked without any issues.
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
.