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

Update docs.asciidoc #4095

Merged
merged 1 commit into from
Apr 24, 2017
Merged

Update docs.asciidoc #4095

merged 1 commit into from
Apr 24, 2017

Conversation

lisakaymera
Copy link

fsstats doesn't work anymore because of a previous merge.
Would be helpful to change fsstats -> fsstat to avoid confusion.

fsstats doesn't work anymore because of a previous merge. 
Would be helpful to change fsstats -> fsstat to avoid confusion.
@elasticmachine
Copy link
Collaborator

Jenkins standing by to test this. If you aren't a maintainer, you can ignore this comment. Someone with commit access, please review this and clear it for Jenkins to run.

1 similar comment
@elasticmachine
Copy link
Collaborator

Jenkins standing by to test this. If you aren't a maintainer, you can ignore this comment. Someone with commit access, please review this and clear it for Jenkins to run.

@tsg tsg added docs needs_backport PR is waiting to be backported to other branches. labels Apr 24, 2017
@tsg
Copy link
Contributor

tsg commented Apr 24, 2017

Good catch! May I ask you to sign the CLA, please? We need that even for docs changes, but it should only take a few minutes. If you don't want to bother with that, I'd be happy to re-add this change in another PR.

@lisakaymera
Copy link
Author

I clicked on the link, signed it, and confirmed the signature. Is that enough?

@ruflin
Copy link
Member

ruflin commented Apr 24, 2017

@lisakaymera Normally yes. Did you use the same username and email you use on github to sign it or a different one?

@ruflin ruflin merged commit bcc5614 into elastic:5.3 Apr 24, 2017
@ruflin
Copy link
Member

ruflin commented Apr 24, 2017

@lisakaymera Seems to have worked now. Thanks for the contribution.

dedemorton pushed a commit to dedemorton/beats that referenced this pull request May 1, 2017
fsstats doesn't work anymore because of a previous merge. 
Would be helpful to change fsstats -> fsstat to avoid confusion.
dedemorton pushed a commit to dedemorton/beats that referenced this pull request May 1, 2017
fsstats doesn't work anymore because of a previous merge. 
Would be helpful to change fsstats -> fsstat to avoid confusion.
tsg pushed a commit that referenced this pull request May 2, 2017
* Fix config example

* Add rsbeat to communitybeats doc (#4108)

* Update docs.asciidoc (#4095)

fsstats doesn't work anymore because of a previous merge. 
Would be helpful to change fsstats -> fsstat to avoid confusion.
ruflin pushed a commit that referenced this pull request May 2, 2017
fsstats doesn't work anymore because of a previous merge. 
Would be helpful to change fsstats -> fsstat to avoid confusion.
@dedemorton dedemorton removed the needs_backport PR is waiting to be backported to other branches. label May 6, 2017
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
fsstats doesn't work anymore because of a previous merge. 
Would be helpful to change fsstats -> fsstat to avoid confusion.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants