-
Notifications
You must be signed in to change notification settings - Fork 16.9k
[stable/nginx-ingress] Add support for extraContainers, extraVolumeMounts and extraVolumes #3983
Conversation
I really like this pattern. Can you bump the version for a minor release as its adding features? should be |
/assign @paulczar |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I really like this pattern. Can you bump the version for a minor release as its adding features?
should be 0.12.0 based on current version.
/ok-to-test |
06a436f
to
571fa8c
Compare
I have rebased and bumped the version to 0.12.0 as requested. |
/retest |
/assign |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
/lgtm |
/assign @unguiculus |
@sathieu you'll need to rebase from master to fix the conflicts. |
Please rename: |
…unts and extraVolumes
571fa8c
to
ee594fb
Compare
@paulczar @unguiculus Done. |
/lgtm |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: paulczar, sathieu, timstoop, unguiculus The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
…unts and extraVolumes (helm#3983)
…unts and extraVolumes (helm#3983)
…unts and extraVolumes (helm#3983) Signed-off-by: voron <av@arilot.com>
Replaces #3810.