-
Notifications
You must be signed in to change notification settings - Fork 457
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
feat(fluent-bit): Support tpl in extraContainers #495
feat(fluent-bit): Support tpl in extraContainers #495
Conversation
6ebc719
to
b0e97a6
Compare
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.
@JaredTan95 could you rebase this PR and update the chart annotations for this change?
6d38b53
to
4a92681
Compare
Signed-off-by: Jared Tan <jian.tan@daocloud.io>
4a92681
to
eb423c0
Compare
Co-authored-by: Steve Hipwell <steve.hipwell@gmail.com> Signed-off-by: Jared Tan <jian.tan@daocloud.io>
Signed-off-by: Jared Tan <jian.tan@daocloud.io>
@JaredTan95 you need to bump the chart version or the checks will fail. |
Signed-off-by: Jared Tan <jian.tan@daocloud.io>
Done, My chart version is overwritten due to the merging of other PRs before. |
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
The same as #163, support tpl in extraContainers.