Skip to content

[panw] Normalize log.level to lowercase - #10270

Merged
taylor-swanson merged 3 commits into
elastic:mainfrom
taylor-swanson:bugfix/panw-log-level-ignorecase
Jun 26, 2024
Merged

[panw] Normalize log.level to lowercase#10270
taylor-swanson merged 3 commits into
elastic:mainfrom
taylor-swanson:bugfix/panw-log-level-ignorecase

Conversation

@taylor-swanson

@taylor-swanson taylor-swanson commented Jun 26, 2024 •
edited
Loading

Copy link
Copy Markdown
Contributor

Proposed commit message

  • Ensure that log.level has been normalized to lowercase, both for consistency and for the conditionals for the event.severity processors.

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.

How to test this PR locally

cd packages/panw
elastic-package test

Related issues
@taylor-swanson
[panw] Normalize log.level to lowercase
25ca4a3 
- Ensure that log.level has been normalized to lowercase, both
for consistency and for the conditionals for the event.severity
processors.
@taylor-swanson taylor-swanson added bug Something isn't working, use only for issues Integration:panw Palo Alto Next-Gen Firewall Team:Security-Deployment and Devices DEPRECATED Deployment and Devices Security team [elastic/sec-deployment-and-devices] labels Jun 26, 2024
@taylor-swanson taylor-swanson self-assigned this Jun 26, 2024
@elasticmachine

elasticmachine commented Jun 26, 2024

Copy link
Copy Markdown

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport
@elasticmachine

elasticmachine commented Jun 26, 2024

Copy link
Copy Markdown

💚 Build Succeeded

History

cc @taylor-swanson
@elastic-sonarqube

elastic-sonarqube Bot commented Jun 26, 2024

Copy link
Copy Markdown

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube
@taylor-swanson
taylor-swanson marked this pull request as ready for review June 26, 2024 14:25
@taylor-swanson
taylor-swanson requested a review from a team as a code owner June 26, 2024 14:25
@elasticmachine

elasticmachine commented Jun 26, 2024

Copy link
Copy Markdown

Pinging @elastic/sec-deployment-and-devices (Team:Security-Deployment and Devices)
pkoutsovasilis
pkoutsovasilis approved these changes Jun 26, 2024
View reviewed changes

@pkoutsovasilis pkoutsovasilis left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM
@taylor-swanson
taylor-swanson merged commit 35fc7a1 into elastic:main Jun 26, 2024
@taylor-swanson
taylor-swanson deleted the bugfix/panw-log-level-ignorecase branch June 26, 2024 17:02
@elasticmachine

elasticmachine commented Jun 26, 2024

Copy link
Copy Markdown

Package panw - 3.26.2 containing this change is available at https://epr.elastic.co/search?package=panw
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 Team:Security-Deployment and Devices DEPRECATED Deployment and Devices Security team [elastic/sec-deployment-and-devices]

3 participants

@taylor-swanson @elasticmachine @pkoutsovasilis