[aws] Fix http.response.status_code to accept 000#3285

Merged

kaiyan-sheng merged 3 commits intoelastic:mainfrom

kaiyan-sheng:cloudfront_logs

May 9, 2022

Merged

[aws] Fix http.response.status_code to accept 000#3285
kaiyan-sheng merged 3 commits intoelastic:mainfrom
kaiyan-sheng:cloudfront_logs

Conversation

Copy link

Contributor

kaiyan-sheng commentedMay 5, 2022•
edited
Loading

What does this PR do?

Based onAWS docs, incloudfront logs,sc-status field allows000 to be one of the acceptable values. With this, we need to change the grok pattern we have when parsing to accept not only positive integers but also just integers.

Checklist

I have reviewedtips 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'schangelog.yml file.
I have verified that Kibana version constraints are current according toguidelines.

Fix http.response.bytes to accept 000

c47135f

kaiyan-sheng requested a review froma team as acode owner

May 5, 2022 16:38

kaiyan-sheng self-assigned this

May 5, 2022

kaiyan-sheng added the Team:Cloud-MonitoringLabel for the Cloud Monitoring team label

May 5, 2022

add changelog

5998726

Copy link

elasticmachine commentedMay 5, 2022•
edited
Loading

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2022-05-05T20:00:11.288+0000
Duration: 28 min 20 sec

Test stats 🧪

Test	Results
Failed	0
Passed	328
Skipped	0
Total	328

🤖 GitHub comments

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

/test : Re-trigger the build.

Copy link

elasticmachine commentedMay 5, 2022•
edited
Loading

🌐 Coverage report

Name	Metrics % (`covered/total`)	Diff
Packages	100.0% (`11/11`)	💚
Files	91.667% (`11/12`)	👎 -4.684
Classes	91.667% (`11/12`)	👎 -4.684
Methods	82.439% (`169/205`)	👎 -5.67
Lines	92.673% (`2087/2252`)	👍 3.873
Conditionals	100.0% (`0/0`)	💚