-
Notifications
You must be signed in to change notification settings - Fork 16.8k
Conversation
Hi @flands. Thanks for your PR. I'm waiting for a helm member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Addresses #12106 Signed-off-by: Steve Flanders <steve@omnition.io>
/assign thanks! |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: flands, maorfr 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 |
Good grief, not again! Stop merging these things! |
@kfox1111 how come? |
This was forced through once before. breaking the api. then we undid it. then its happening again. Another breaking change. :( The result last time was we came up with rawConfig for those that wanted to add sections. extraEntries is for adding entries to the existing sections. :( That this keeps happening seems like we really need some kind of howto use thing. |
i have added you as a collaborator, so from now on you and @edsiper can own this with less other maintainers getting in the way! :) |
Thank you. :) |
Addresses helm#12106 Signed-off-by: Steve Flanders <steve@omnition.io>
Addresses helm#12106 Signed-off-by: Steve Flanders <steve@omnition.io>
This reverts the incorrect patch merged as part of pr helm#13000 as well as adds warnings so that hopefully this never happens again.` Signed-off-by: Kevin Fox <Kevin.Fox@pnnl.gov>
Addresses helm#12106 Signed-off-by: Steve Flanders <steve@omnition.io>
This reverts the incorrect patch merged as part of pr helm#13000 as well as adds warnings so that hopefully this never happens again.` Signed-off-by: Kevin Fox <Kevin.Fox@pnnl.gov>
What this PR does / why we need it:
Some fluent-bit configurations do not work today
Which issue this PR fixes
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]
Signed-off-by: Steve Flanders steve@sflanders.net