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

feat(fluent-bit): Support tpl in extraContainers #495

Merged
merged 4 commits into from
May 1, 2024

Conversation

JaredTan95
Copy link
Contributor

The same as #163, support tpl in extraContainers.

  • Support templating when value is a string
  • No templating when value is an array due to inconsistent behaviour

Copy link
Collaborator

@stevehipwell stevehipwell left a 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?

@JaredTan95 JaredTan95 force-pushed the extraContainers_supprt_tpl branch 4 times, most recently from 6d38b53 to 4a92681 Compare April 29, 2024 15:24
Signed-off-by: Jared Tan <jian.tan@daocloud.io>
charts/fluent-bit/Chart.yaml Outdated Show resolved Hide resolved
JaredTan95 and others added 2 commits April 30, 2024 23:38
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>
@stevehipwell
Copy link
Collaborator

@JaredTan95 you need to bump the chart version or the checks will fail.

Signed-off-by: Jared Tan <jian.tan@daocloud.io>
@JaredTan95
Copy link
Contributor Author

@JaredTan95 you need to bump the chart version or the checks will fail.

Done, My chart version is overwritten due to the merging of other PRs before.

Copy link
Collaborator

@stevehipwell stevehipwell left a comment

Choose a reason for hiding this comment

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

LGTM

@stevehipwell stevehipwell merged commit c4dc805 into fluent:main May 1, 2024
2 checks passed
@JaredTan95 JaredTan95 deleted the extraContainers_supprt_tpl branch May 1, 2024 08:46
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.

2 participants