Skip to content
This repository has been archived by the owner on May 17, 2019. It is now read-only.

Add keepalive document #632

Merged
merged 1 commit into from
May 17, 2019
Merged

Conversation

ganmacs
Copy link
Member

@ganmacs ganmacs commented Apr 23, 2019


| type | default | version |
|:----:|:-------:|:-------:|
| bool | fales | 0.14.3 |
Copy link
Contributor

Choose a reason for hiding this comment

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

We should specify concrete version such as 1.4.3 in version column.

Copy link
Member Author

Choose a reason for hiding this comment

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

Thank you for your comment. As current fluentd looks v1.4.2(https://github.com/fluent/fluentd/releases/tag/v1.4.2). I changed the version to 1.4.3.


| type | default | version |
|:----:|:-------:|:-------:|
| time | nil | 0.14.3 |
Copy link
Contributor

Choose a reason for hiding this comment

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

ditto.

Signed-off-by: Yuta Iwama <ganmacs@gmail.com>
@fujimotos fujimotos merged commit 3f8271e into fluent:master May 17, 2019
@fujimotos
Copy link
Member

Merged. Thank you.

@ganmacs ganmacs deleted the add-keepalive-docs branch May 17, 2019 01:35
fujimotos pushed a commit to fujimotos/fluentd-docs-gitbook that referenced this pull request May 17, 2019
This patch contains the changesets made by:

 * fluent/fluentd-docs#632
 * fluent/fluentd-docs#634
 * fluent/fluentd-docs#633

Signed-off-by: Fujimoto Seiji <fujimoto@clear-code.com>
fujimotos pushed a commit to fluent/fluentd-docs-gitbook that referenced this pull request May 17, 2019
This patch contains the changesets made by:

 * fluent/fluentd-docs#632
 * fluent/fluentd-docs#634
 * fluent/fluentd-docs#633

Signed-off-by: Fujimoto Seiji <fujimoto@clear-code.com>
@fujimotos
Copy link
Member

Side note: I ported this change to the new GitBook site in fluent/fluentd-docs-gitbook/pull/23.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants