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

Fix typo in netflow module docs #18992

Merged
merged 2 commits into from
Aug 10, 2020
Merged

Fix typo in netflow module docs #18992

merged 2 commits into from
Aug 10, 2020

Conversation

rruckley
Copy link
Contributor

@rruckley rruckley commented Jun 5, 2020

What does this PR do?

Fixes typo in netflow config documentation (s/find/bind/).

Why is it important?

Sloppy spelling gives bad impression for Elastic.

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • [ x ] 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.

@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

1 similar comment
@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Jun 5, 2020
@cla-checker-service
Copy link

cla-checker-service bot commented Jun 5, 2020

💚 CLA has been signed

@elasticmachine
Copy link
Collaborator

Pinging @elastic/siem (Team:SIEM)

@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Jun 5, 2020
@andrewkroh andrewkroh added needs CLA User must sign the Elastic Contributor License before review. docs Filebeat Filebeat labels Jun 5, 2020
@rruckley
Copy link
Contributor Author

rruckley commented Jun 9, 2020

I have now signed the contributor agreement. All this to fix a typo.

@rruckley
Copy link
Contributor Author

rruckley commented Jun 9, 2020

@andrewkroh I've signed the CLA. I would be chuffed if my single character typo fix could make it through the paperwork but I feel like this process is not really set up for such trivial things!

@marc-gr
Copy link
Contributor

marc-gr commented Jun 9, 2020

jenkins run tests

@marc-gr marc-gr removed the needs CLA User must sign the Elastic Contributor License before review. label Jun 9, 2020
@marc-gr
Copy link
Contributor

marc-gr commented Jun 9, 2020

Thanks for your contribution @rruckley! When changing documentation we need to re-generate some files, you can do so by running mage fmt update inside filebeat folder and commiting the modified files. Please let us know if you encounter any issue during the process 😄

@adriansr
Copy link
Contributor

adriansr commented Jun 9, 2020

Also you'll need to point to PR to the master branch instead of 7.7.

@dedemorton
Copy link
Contributor

Thanks for catching this! Sorry this easy fix is so complicated. I will apply your commit to master and run mage fmt update.

@rruckley
Copy link
Contributor Author

Thank you, I wasn't really looking to become a full developer but it's nice to get these little things fixed.

Fixed typo, line 47, Address to bind to
@andrewkroh andrewkroh changed the base branch from 7.7 to master August 10, 2020 21:42
@andrewkroh andrewkroh requested review from a team as code owners August 10, 2020 21:42
@andrewkroh
Copy link
Member

run tests

@elasticmachine
Copy link
Collaborator

elasticmachine commented Aug 10, 2020

❕ Build Aborted

The PR is not allowed to run in the CI yet

Pipeline View Test View Changes Artifacts

Expand to view the summary

Build stats

  • Build Cause: [Pull request #18992 updated]

  • Reason: The PR is not allowed to run in the CI yet

  • Start Time: 2020-08-10T22:02:27.177+0000

  • Duration: 4 min 35 sec

  • Commit: 3cc5c64

Steps errors

Expand to view the steps failures

  • Name: Error signal
    • Description: githubPrCheckApproved: The PR is not allowed to run in the CI yet. (Only users with write permission

    • Duration: 0 min 0 sec

    • Start Time: 2020-08-10T22:05:58.048+0000

    • log

Log output

Expand to view the last 100 lines of log output

[2020-08-10T22:05:59.607Z] Stage "Elastic Agent x-pack Windows" skipped due to earlier failure(s)
[2020-08-10T22:05:59.608Z] Stage "Elastic Agent Mac OS X" skipped due to earlier failure(s)
[2020-08-10T22:05:59.609Z] Stage "Filebeat oss" skipped due to earlier failure(s)
[2020-08-10T22:05:59.610Z] Stage "Filebeat x-pack" skipped due to earlier failure(s)
[2020-08-10T22:05:59.610Z] Stage "Filebeat Mac OS X" skipped due to earlier failure(s)
[2020-08-10T22:05:59.611Z] Stage "Filebeat x-pack Mac OS X" skipped due to earlier failure(s)
[2020-08-10T22:05:59.612Z] Stage "Filebeat Windows" skipped due to earlier failure(s)
[2020-08-10T22:05:59.613Z] Stage "Filebeat x-pack Windows" skipped due to earlier failure(s)
[2020-08-10T22:05:59.614Z] Stage "Heartbeat" skipped due to earlier failure(s)
[2020-08-10T22:05:59.614Z] Stage "Auditbeat oss Linux" skipped due to earlier failure(s)
[2020-08-10T22:05:59.615Z] Stage "Auditbeat crosscompile" skipped due to earlier failure(s)
[2020-08-10T22:05:59.616Z] Stage "Auditbeat oss Mac OS X" skipped due to earlier failure(s)
[2020-08-10T22:05:59.616Z] Stage "Auditbeat oss Windows" skipped due to earlier failure(s)
[2020-08-10T22:05:59.617Z] Stage "Auditbeat x-pack" skipped due to earlier failure(s)
[2020-08-10T22:05:59.618Z] Stage "Auditbeat x-pack Mac OS X" skipped due to earlier failure(s)
[2020-08-10T22:05:59.619Z] Stage "Auditbeat x-pack Windows" skipped due to earlier failure(s)
[2020-08-10T22:05:59.620Z] Stage "Libbeat" skipped due to earlier failure(s)
[2020-08-10T22:05:59.620Z] Stage "Libbeat x-pack" skipped due to earlier failure(s)
[2020-08-10T22:05:59.621Z] Stage "Metricbeat OSS Unit tests" skipped due to earlier failure(s)
[2020-08-10T22:05:59.622Z] Stage "Metricbeat OSS Go Integration tests" skipped due to earlier failure(s)
[2020-08-10T22:05:59.623Z] Stage "Metricbeat OSS Python Integration tests" skipped due to earlier failure(s)
[2020-08-10T22:05:59.624Z] Stage "Metricbeat x-pack" skipped due to earlier failure(s)
[2020-08-10T22:05:59.625Z] Stage "Metricbeat crosscompile" skipped due to earlier failure(s)
[2020-08-10T22:05:59.625Z] Stage "Metricbeat Mac OS X" skipped due to earlier failure(s)
[2020-08-10T22:05:59.626Z] Stage "Metricbeat x-pack Mac OS X" skipped due to earlier failure(s)
[2020-08-10T22:05:59.627Z] Stage "Metricbeat Windows" skipped due to earlier failure(s)
[2020-08-10T22:05:59.628Z] Stage "Metricbeat x-pack Windows" skipped due to earlier failure(s)
[2020-08-10T22:05:59.629Z] Stage "Packetbeat OSS" skipped due to earlier failure(s)
[2020-08-10T22:05:59.629Z] Stage "dockerlogbeat" skipped due to earlier failure(s)
[2020-08-10T22:05:59.630Z] Stage "Winlogbeat" skipped due to earlier failure(s)
[2020-08-10T22:05:59.630Z] Stage "Winlogbeat Windows x-pack" skipped due to earlier failure(s)
[2020-08-10T22:05:59.631Z] Stage "Functionbeat" skipped due to earlier failure(s)
[2020-08-10T22:05:59.632Z] Stage "Journalbeat" skipped due to earlier failure(s)
[2020-08-10T22:05:59.632Z] Stage "Generators" skipped due to earlier failure(s)
[2020-08-10T22:05:59.633Z] Stage "Kubernetes" skipped due to earlier failure(s)
[2020-08-10T22:05:59.706Z] Stage "Heartbeat" skipped due to earlier failure(s)
[2020-08-10T22:05:59.707Z] Stage "Libbeat" skipped due to earlier failure(s)
[2020-08-10T22:05:59.708Z] Stage "Metricbeat x-pack" skipped due to earlier failure(s)
[2020-08-10T22:05:59.709Z] Stage "Packetbeat OSS" skipped due to earlier failure(s)
[2020-08-10T22:05:59.711Z] Stage "dockerlogbeat" skipped due to earlier failure(s)
[2020-08-10T22:05:59.712Z] Stage "Winlogbeat" skipped due to earlier failure(s)
[2020-08-10T22:05:59.712Z] Stage "Functionbeat" skipped due to earlier failure(s)
[2020-08-10T22:05:59.713Z] Stage "Journalbeat" skipped due to earlier failure(s)
[2020-08-10T22:05:59.714Z] Stage "Generators" skipped due to earlier failure(s)
[2020-08-10T22:06:00.261Z] Failed in branch Elastic Agent x-pack
[2020-08-10T22:06:00.262Z] Failed in branch Elastic Agent x-pack Windows
[2020-08-10T22:06:00.263Z] Failed in branch Elastic Agent Mac OS X
[2020-08-10T22:06:00.263Z] Failed in branch Filebeat oss
[2020-08-10T22:06:00.264Z] Failed in branch Filebeat x-pack
[2020-08-10T22:06:00.264Z] Failed in branch Filebeat Mac OS X
[2020-08-10T22:06:00.265Z] Failed in branch Filebeat x-pack Mac OS X
[2020-08-10T22:06:00.266Z] Failed in branch Filebeat Windows
[2020-08-10T22:06:00.266Z] Failed in branch Filebeat x-pack Windows
[2020-08-10T22:06:00.267Z] Failed in branch Auditbeat oss Linux
[2020-08-10T22:06:00.267Z] Failed in branch Auditbeat crosscompile
[2020-08-10T22:06:00.268Z] Failed in branch Auditbeat oss Mac OS X
[2020-08-10T22:06:00.268Z] Failed in branch Auditbeat oss Windows
[2020-08-10T22:06:00.271Z] Failed in branch Auditbeat x-pack
[2020-08-10T22:06:00.272Z] Failed in branch Auditbeat x-pack Mac OS X
[2020-08-10T22:06:00.273Z] Failed in branch Auditbeat x-pack Windows
[2020-08-10T22:06:00.273Z] Failed in branch Libbeat x-pack
[2020-08-10T22:06:00.274Z] Failed in branch Metricbeat OSS Unit tests
[2020-08-10T22:06:00.274Z] Failed in branch Metricbeat OSS Go Integration tests
[2020-08-10T22:06:00.275Z] Failed in branch Metricbeat OSS Python Integration tests
[2020-08-10T22:06:00.276Z] Failed in branch Metricbeat crosscompile
[2020-08-10T22:06:00.276Z] Failed in branch Metricbeat Mac OS X
[2020-08-10T22:06:00.277Z] Failed in branch Metricbeat x-pack Mac OS X
[2020-08-10T22:06:00.277Z] Failed in branch Metricbeat Windows
[2020-08-10T22:06:00.278Z] Failed in branch Metricbeat x-pack Windows
[2020-08-10T22:06:00.278Z] Failed in branch Winlogbeat Windows x-pack
[2020-08-10T22:06:00.279Z] Failed in branch Kubernetes
[2020-08-10T22:06:00.563Z] Stage "Heartbeat" skipped due to earlier failure(s)
[2020-08-10T22:06:00.565Z] Stage "Libbeat" skipped due to earlier failure(s)
[2020-08-10T22:06:00.566Z] Stage "Metricbeat x-pack" skipped due to earlier failure(s)
[2020-08-10T22:06:00.567Z] Stage "Packetbeat OSS" skipped due to earlier failure(s)
[2020-08-10T22:06:00.568Z] Stage "Winlogbeat" skipped due to earlier failure(s)
[2020-08-10T22:06:00.568Z] Stage "Functionbeat" skipped due to earlier failure(s)
[2020-08-10T22:06:00.569Z] Stage "Generators" skipped due to earlier failure(s)
[2020-08-10T22:06:00.612Z] Failed in branch dockerlogbeat
[2020-08-10T22:06:00.614Z] Failed in branch Journalbeat
[2020-08-10T22:06:00.849Z] Stage "Heartbeat" skipped due to earlier failure(s)
[2020-08-10T22:06:00.851Z] Stage "Libbeat" skipped due to earlier failure(s)
[2020-08-10T22:06:00.852Z] Stage "Packetbeat OSS" skipped due to earlier failure(s)
[2020-08-10T22:06:00.853Z] Stage "Functionbeat" skipped due to earlier failure(s)
[2020-08-10T22:06:00.854Z] Stage "Generators" skipped due to earlier failure(s)
[2020-08-10T22:06:00.894Z] Failed in branch Metricbeat x-pack
[2020-08-10T22:06:00.895Z] Failed in branch Winlogbeat
[2020-08-10T22:06:01.127Z] Failed in branch Heartbeat
[2020-08-10T22:06:01.128Z] Failed in branch Libbeat
[2020-08-10T22:06:01.129Z] Failed in branch Packetbeat OSS
[2020-08-10T22:06:01.130Z] Failed in branch Functionbeat
[2020-08-10T22:06:01.130Z] Stage "Generators" skipped due to earlier failure(s)
[2020-08-10T22:06:01.219Z] Failed in branch Generators
[2020-08-10T22:06:01.533Z] Running on Jenkins in /var/lib/jenkins/workspace/Beats_beats_PR-18992
[2020-08-10T22:06:01.653Z] [INFO] getVaultSecret: Getting secrets
[2020-08-10T22:06:01.737Z] Masking supported pattern matches of $VAULT_ADDR or $VAULT_ROLE_ID or $VAULT_SECRET_ID
[2020-08-10T22:06:02.557Z] + chmod 755 generate-build-data.sh
[2020-08-10T22:06:02.557Z] + ./generate-build-data.sh https://beats-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/Beats/beats/PR-18992/ https://beats-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/Beats/beats/PR-18992/runs/5 ABORTED 215117
[2020-08-10T22:06:02.557Z] INFO: curl https://beats-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/Beats/beats/PR-18992/runs/5/steps/?limit=10000 -o steps-info.json
[2020-08-10T22:06:03.108Z] INFO: curl https://beats-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/Beats/beats/PR-18992/runs/5/tests/?status=FAILED -o tests-errors.json

@andrewkroh andrewkroh changed the title Updated netflow.asciidoc Fix typo in netflow module docs Aug 10, 2020
@andrewkroh andrewkroh merged commit 091f66d into elastic:master Aug 10, 2020
v1v added a commit to v1v/beats that referenced this pull request Aug 12, 2020
…ne-2.0

* upstream/master: (39 commits)
  [ITs] Revert healthcheck for elasticsearchssl service to the previous behaviour (elastic#20558)
  [Heartbeat] Fix packaging (elastic#20566)
  [Heartbeat] Add Magefile to X-Pack (elastic#20549)
  [Packetbeat] Add "network" to event.category (elastic#20392)
  fix typo in docs (elastic#20541)
  Add service resource in k8s cluster role (elastic#20546)
  Update Golang version to 1.14.7 (elastic#20508)
  Add missing inputs to filebeat spec (elastic#20388)
  add warning log in aws and googlecloud module for API cost (elastic#20523)
  Fix fortinet.firewall.mem value to be interpreted as integer (elastic#19335)
  [CI] add more resilience (elastic#20505)
  [JJBB] fix credentials with a service account for golang-crossbuild (elastic#20537)
  [ITs] change healthcheck for elasticsearch (elastic#20514)
  [JJBB] fix credentials with a service account (elastic#20535)
  chore(ci): use build step for checking if is PR (elastic#20536)
  [CI] runbld project name (elastic#20466)
  Add panw.panos.endreason field (elastic#18705)
  [Filebeat] Fix PANW field spelling "veredict" to "verdict" (elastic#18808)
  Fix typo in netflow module docs (elastic#18992)
  Modified auditd ingest pipeline to handle node=hostname (elastic#19659)
  ...
melchiormoulin pushed a commit to melchiormoulin/beats that referenced this pull request Oct 14, 2020
* Updated netflow.asciidoc

Fixed typo, line 47, Address to bind to

* Update source file for netflow docs

Co-authored-by: Andrew Kroh <andrew.kroh@elastic.co>
jsoriano pushed a commit to jsoriano/beats that referenced this pull request Feb 16, 2021
* Updated netflow.asciidoc

Fixed typo, line 47, Address to bind to

* Update source file for netflow docs

Co-authored-by: Andrew Kroh <andrew.kroh@elastic.co>
(cherry picked from commit 091f66d)
jsoriano added a commit that referenced this pull request Feb 17, 2021
(cherry picked from commit 091f66d)

Co-authored-by: Andrew Kroh <andrew.kroh@elastic.co>
Co-authored-by: Ryan <rruckley@gmail.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.

8 participants