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.

Sign up for GitHub

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

Jump to bottom

Some input plugins don't support filter plugin #173

Closed
nokute78 opened this issue Feb 6, 2017 · 5 comments
Closed

Some input plugins don't support filter plugin #173

nokute78 opened this issue Feb 6, 2017 · 5 comments

Comments

@nokute78
Copy link
Collaborator

nokute78 commented Feb 6, 2017

We should add flb_input_buf_write_start() and flb_input_buf_write_end() at packing.
It needs to call the filter handler.
nokute78 added a commit to nokute78/fluent-bit that referenced this issue Feb 6, 2017
@nokute78
in_mem: filter plugin support (fluent#173)
9b5c5bb 
Signed-off-by: Takahiro YAMASHITA <[email protected]>
@nokute78 nokute78 mentioned this issue Feb 6, 2017
Merged
nokute78 added a commit to nokute78/fluent-bit that referenced this issue Feb 6, 2017
@nokute78
in_disk: filter plugin support (fluent#173)
fb00a82 
Signed-off-by: Takahiro YAMASHITA <[email protected]>
@nokute78 nokute78 mentioned this issue Feb 6, 2017
Merged
nokute78 added a commit to nokute78/fluent-bit that referenced this issue Feb 6, 2017
@nokute78
in_head: filter plugin support (fluent#173)
9a8cd05 
Signed-off-by: Takahiro YAMASHITA <[email protected]>
@nokute78 nokute78 mentioned this issue Feb 6, 2017
Merged
nokute78 added a commit to nokute78/fluent-bit that referenced this issue Feb 6, 2017
@nokute78
in_health: filter plugin support (fluent#173)
29aa5d1 
Signed-off-by: Takahiro YAMASHITA <[email protected]>
@nokute78 nokute78 mentioned this issue Feb 6, 2017
Merged
nokute78 added a commit to nokute78/fluent-bit that referenced this issue Feb 6, 2017
@nokute78
in_proc: filter plugin support (fluent#173)
bbcd702 
Signed-off-by: Takahiro YAMASHITA <[email protected]>
@nokute78 nokute78 mentioned this issue Feb 6, 2017
Merged
nokute78 added a commit to nokute78/fluent-bit that referenced this issue Feb 6, 2017
@nokute78
in_random: filter plugin support (fluent#173)
bbe82b5 
Signed-off-by: Takahiro YAMASHITA <[email protected]>
@nokute78 nokute78 mentioned this issue Feb 6, 2017
Merged
nokute78 added a commit to nokute78/fluent-bit that referenced this issue Feb 6, 2017
@nokute78
in_lib: filter plugin support (fluent#173)
35a7bbf 
Signed-off-by: Takahiro YAMASHITA <[email protected]>
@nokute78 nokute78 mentioned this issue Feb 6, 2017
Merged
@nokute78
Copy link
Collaborator Author

nokute78 commented Feb 6, 2017

Rest of input plugins are

  • in_forward
  • in_http
  • in_kmsg
  • in_serial
  • in_stdin
  • in_tcp
  • in_xbee

edsiper added a commit that referenced this issue Feb 6, 2017
@edsiper
Merge pull request #180 from nokute78/lib_filter
4753fad 
in_lib: filter plugin support (#173)
edsiper added a commit that referenced this issue Feb 6, 2017
@edsiper
Merge pull request #179 from nokute78/randome_filter
f0c29d9 
in_random: filter plugin support (#173)
edsiper added a commit that referenced this issue Feb 6, 2017
@edsiper
Merge pull request #178 from nokute78/proc_filter
6740bad 
in_proc: filter plugin support (#173)
edsiper added a commit that referenced this issue Feb 6, 2017
@edsiper
Merge pull request #177 from nokute78/health_filter
c88b467 
in_health: filter plugin support (#173)
edsiper added a commit that referenced this issue Feb 6, 2017
@edsiper
Merge pull request #176 from nokute78/head_filter
4d48f76 
in_head: filter plugin support (#173)
edsiper added a commit that referenced this issue Feb 6, 2017
@edsiper
Merge pull request #175 from nokute78/disk_filter
be68ad8 
in_disk: filter plugin support (#173)
edsiper added a commit that referenced this issue Feb 6, 2017
@edsiper
Merge pull request #174 from nokute78/mem_filter
02a890e 
in_mem: filter plugin support (#173)
@edsiper
Copy link
Member

edsiper commented Feb 6, 2017

thanks for the patches!

I will keep this open until we fix all of them.

note: in_forward use 'dyntag' mechanism, I need to add filter support for that ingestion method.

edsiper added a commit that referenced this issue Feb 6, 2017
@edsiper
in_kmsg: use string field type and enable filter plugin support (#173)
075ef34 
Signed-off-by: Eduardo Silva <[email protected]>
edsiper added a commit that referenced this issue Feb 6, 2017
@edsiper
in_serial: use string field type and enable filter plugin support (#173)
4acee52 
Signed-off-by: Eduardo Silva <[email protected]>
edsiper added a commit that referenced this issue Feb 6, 2017
@edsiper
in_stin: use string field type and enable filter plugin support (#173)
a09d73a 
Signed-off-by: Eduardo Silva <[email protected]>
edsiper added a commit that referenced this issue Feb 6, 2017
@edsiper
in_stdin: use string field type and enable filter plugin support (#173)
0b32056 
Signed-off-by: Eduardo Silva <[email protected]>
edsiper added a commit that referenced this issue Feb 6, 2017
@edsiper
in_tcp: pack direct map properly and add filtering support (#173)
a314f7c 
Signed-off-by: Eduardo Silva <[email protected]>
@edsiper
Copy link
Member

edsiper commented Feb 6, 2017 •
edited
Loading

Updates:

  • in_forward: pending until filtering support for dyntag plugins is added.
  • in_http: this plugin is not functional yet
  • in_kmsg: fixed.
  • in_serial: fixed.
  • in_stdin: fixed.
  • in_tcp: fixed.
  • in_xbee: should this plugin be deprecated ? (ref: in_xbee: should be deprecated ? #181)

@nokute78
Copy link
Collaborator Author

nokute78 commented Feb 7, 2017

Thank you for fixing.
A little bit more!

@nokute78 nokute78 changed the title Some input plugins doesn't support filter plugin Some input plugins don't support filter plugin Feb 7, 2017
@nokute78 nokute78 mentioned this issue Feb 7, 2017
Closed
@edsiper
Copy link
Member

edsiper commented Feb 13, 2017

closing this issue, remaining ones are:

@edsiper edsiper closed this as completed Feb 13, 2017
fujimotos pushed a commit to fujimotos/fluent-bit that referenced this issue Jul 22, 2019
@Kwelity @edsiper
filter: parser: Reserve_Data and Preserve_Key should be activated in …
df96d05 
…[FILTER]. Preserved_Key only works if Reserved_Key On (fluent#173)

Signed-off-by: Cédric Da Fonseca <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@edsiper @nokute78