Skip to content

panw: update mapping of zone fields in visualisations#3361

Merged
efd6 merged 2 commits intoelastic:mainfrom
efd6:3321-panw-bis
May 17, 2022
Merged

panw: update mapping of zone fields in visualisations#3361
efd6 merged 2 commits intoelastic:mainfrom
efd6:3321-panw-bis

Conversation

@efd6
Copy link
Contributor

@efd6 efd6 commented May 17, 2022

What does this PR do?

These were missed when the source and destination zone ingestion was
removed.

See b6a86b2#diff-59b7510d3715bf1d6cc9619d4ae36134571ae31e2bb54c717b653287ac4cc288L59-L62 for details.

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.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.

Author's Checklist

  • [ ]

How to test this PR locally

Related issues

Screenshots

These were missed when the source and destination zone ingestion was
removed.
@efd6 efd6 added bug Something isn't working, use only for issues Team:Security-External Integrations Integration:panw Palo Alto Next-Gen Firewall labels May 17, 2022
@efd6 efd6 self-assigned this May 17, 2022
@efd6 efd6 requested a review from a team as a code owner May 17, 2022 00:19
@elasticmachine
Copy link

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

@elasticmachine
Copy link

elasticmachine commented May 17, 2022

💚 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 preview

Expand to view the summary

Build stats

  • Start Time: 2022-05-17T00:20:23.344+0000

  • Duration: 18 min 34 sec

Test stats 🧪

Test Results
Failed 0
Passed 33
Skipped 0
Total 33

🤖 GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

@elasticmachine
Copy link

🌐 Coverage report

Name Metrics % (covered/total) Diff
Packages 100.0% (1/1) 💚
Files 100.0% (6/6) 💚 3.589
Classes 100.0% (6/6) 💚 3.589
Methods 100.0% (34/34) 💚 11.714
Lines 87.059% (814/935) 👎 -1.973
Conditionals 100.0% (0/0) 💚

@efd6 efd6 requested a review from P1llus May 17, 2022 01:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working, use only for issues Integration:panw Palo Alto Next-Gen Firewall

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants