Skip to content

Conversation

@cosmo0920
Copy link
Contributor

Corresponding to fluent/fluent-bit#9301.

Signed-off-by: Hiroshi Hatake <hiroshi@chronosphere.io>
Copy link
Contributor

@lockewritesdocs lockewritesdocs left a comment

Choose a reason for hiding this comment

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

Suggested a few changes, but approving to keep the wheels of industry turning 🌐

Co-authored-by: Adam Locke <adam.locke@chronosphere.io>
Signed-off-by: Hiroshi Hatake <hiroshi@chronosphere.io>
@cosmo0920 cosmo0920 force-pushed the cosmo0920-add-description-for-use_integer-parameter branch from ddcb704 to 4e411cf Compare September 4, 2024 01:15
Signed-off-by: Hiroshi Hatake <hiroshi@chronosphere.io>
Copy link
Contributor

@lockewritesdocs lockewritesdocs left a comment

Choose a reason for hiding this comment

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

LGTM 🦖

@cosmo0920 cosmo0920 changed the title out_influxdb: Add a description for Use_Integer parameter out_influxdb: Add a description for Add_Integer_Suffix parameter Sep 5, 2024
@lecaros lecaros merged commit b24b537 into master Nov 9, 2024
@lecaros lecaros deleted the cosmo0920-add-description-for-use_integer-parameter branch November 9, 2024 23:40
TomlinfreeGit pushed a commit to TomlinfreeGit/fluent-bit-docs that referenced this pull request Oct 28, 2025
…ent#1451)

* out_influxdb: Add a description for Use_Integer parameter

Signed-off-by: Hiroshi Hatake <hiroshi@chronosphere.io>

* Update pipeline/outputs/influxdb.md

Co-authored-by: Adam Locke <adam.locke@chronosphere.io>
Signed-off-by: Hiroshi Hatake <hiroshi@chronosphere.io>

* out_influxdb: Sync the parameter name change

Signed-off-by: Hiroshi Hatake <hiroshi@chronosphere.io>

---------

Signed-off-by: Hiroshi Hatake <hiroshi@chronosphere.io>
Co-authored-by: Hiroshi Hatake <hiroshi@choronosphere.io>
Co-authored-by: Adam Locke <adam.locke@chronosphere.io>
Signed-off-by: Tom <yao.lin@siemens.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants