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

[Doc] Fix Filebeat MSSQL module doc #17744

Merged
merged 1 commit into from
Apr 21, 2020
Merged

Conversation

renshuki
Copy link
Contributor

What does this PR do?

Fix Filebeat MSSQL module doc where access should be log.

Checklist

- [ ] My code follows the style guidelines of this project
- [ ] I have commented my code, particularly in hard-to-understand areas

  • I have made corresponding changes to the documentation
    - [ ] I have made corresponding change to the default configuration files
    - [ ] I have added tests that prove my fix is effective or that my feature works
    - [ ] I have added an entry in CHANGELOG.next.asciidoc or CHANGELOG-developer.next.asciidoc.

Related issues

@renshuki renshuki added the docs label Apr 16, 2020
Copy link
Contributor

@kaiyan-sheng kaiyan-sheng left a comment

Choose a reason for hiding this comment

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

Thanks for picking up this issue! This filebeat/docs/modules/mssql.asciidoc file is actually generated, not suppose to manually change it. Could you fix it in beats/x-pack/filebeat/module/mssql/_meta/docs.asciidoc please? Thank you!

@kaiyan-sheng kaiyan-sheng self-assigned this Apr 16, 2020
@kaiyan-sheng kaiyan-sheng added needs_backport PR is waiting to be backported to other branches. Team:Services (Deprecated) Label for the former Integrations-Services team labels Apr 16, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/integrations-services (Team:Services)

@renshuki renshuki force-pushed the renshuki-filebeat-mssql-doc branch from 2c949ea to b04f74c Compare April 17, 2020 00:37
@renshuki
Copy link
Contributor Author

Oops, sorry @kaiyan-sheng didn't notice that.
I fixed the file you mentioned instead, should be all good! 🙏
Thanks.

@kaiyan-sheng
Copy link
Contributor

Ahh sorry @renshuki I forgot to mention, could you run mage fmt update under elastic/beats/filebeat directory please? That will update the generated file to make ci pass.

@renshuki renshuki force-pushed the renshuki-filebeat-mssql-doc branch from b04f74c to 1c2ec10 Compare April 18, 2020 03:00
@renshuki
Copy link
Contributor Author

Thanks @kaiyan-sheng, done! 👍

@kaiyan-sheng
Copy link
Contributor

kaiyan-sheng commented Apr 20, 2020

CI is failing caused by old Kafka version in CI. Will merge after #17819.

@kaiyan-sheng
Copy link
Contributor

Hi @renshuki, could you rebase this PR on top of master branch please? There was a problem in CI just got fixed. Thanks!

@renshuki renshuki force-pushed the renshuki-filebeat-mssql-doc branch from 1c2ec10 to ba7c255 Compare April 21, 2020 07:14
@renshuki
Copy link
Contributor Author

Hey @kaiyan-sheng, I did the rebase. Thanks.

@kaiyan-sheng
Copy link
Contributor

Thank you for fixing this!! I will merge this PR and cherrypick it into previous branches.

@kaiyan-sheng kaiyan-sheng merged commit 1621c0f into master Apr 21, 2020
@kaiyan-sheng kaiyan-sheng deleted the renshuki-filebeat-mssql-doc branch April 21, 2020 14:01
@kaiyan-sheng kaiyan-sheng added v7.8.0 and removed needs_backport PR is waiting to be backported to other branches. labels Apr 21, 2020
kaiyan-sheng added a commit that referenced this pull request Apr 21, 2020
(cherry picked from commit 1621c0f)

Co-authored-by: François-Clément Brossard <[email protected]>
kaiyan-sheng added a commit that referenced this pull request Apr 21, 2020
(cherry picked from commit 1621c0f)

Co-authored-by: François-Clément Brossard <[email protected]>
kaiyan-sheng added a commit that referenced this pull request Apr 22, 2020
(cherry picked from commit 1621c0f)

Co-authored-by: François-Clément Brossard <[email protected]>
@renshuki
Copy link
Contributor Author

Thanks for the merge and backports!

leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
(cherry picked from commit 3cd2d76)

Co-authored-by: François-Clément Brossard <[email protected]>
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
(cherry picked from commit 3cd2d76)

Co-authored-by: François-Clément Brossard <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Team:Services (Deprecated) Label for the former Integrations-Services team v7.6.3 v7.7.0 v7.8.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Docs]It seems like the document is wrong, so please request a correction. (Filebeat)
4 participants