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 fields for Netflow module #697

Merged
merged 2 commits into from
Mar 4, 2021

Conversation

andrewkroh
Copy link
Member

@andrewkroh andrewkroh commented Feb 16, 2021

What does this PR do?

Updates the vendor fields as a result of elastic/beats#23773.

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.

Related issues

@elasticmachine
Copy link

Pinging @elastic/security-external-integrations (Team:Security-External Integrations)

@elasticmachine
Copy link

elasticmachine commented Feb 16, 2021

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: Branch indexing

  • Start Time: 2021-03-02T22:28:23.548+0000

  • Duration: 40 min 31 sec

  • Commit: 149d964

Test stats 🧪

Test Results
Failed 0
Passed 1842
Skipped 3
Total 1845

Trends 🧪

Image of Build Times

Image of Tests

@andrewkroh andrewkroh force-pushed the feature/netflow/vendor-fields branch from 152c572 to 149d964 Compare February 18, 2021 13:52
@mtojek
Copy link
Contributor

mtojek commented Feb 24, 2021

The build is failing due to this: elastic/beats#24198 . If it's not an urgent issue I would wait for the correct Docker images.

@andrewkroh andrewkroh merged commit 903d237 into elastic:master Mar 4, 2021
eyalkraft pushed a commit to build-security/integrations that referenced this pull request Mar 30, 2022
* Update fields for Netflow module

Updates the vendor fields as a result of elastic/beats#23773.

* Update changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants