Skip to content

[Azure] Add geo.name and result_description fields in platformlogs#3211

Merged
kaiyan-sheng merged 3 commits intoelastic:mainfrom
kaiyan-sheng:platformlogs
Apr 29, 2022
Merged

[Azure] Add geo.name and result_description fields in platformlogs#3211
kaiyan-sheng merged 3 commits intoelastic:mainfrom
kaiyan-sheng:platformlogs

Conversation

@kaiyan-sheng
Copy link
Copy Markdown

@kaiyan-sheng kaiyan-sheng commented Apr 27, 2022

What does this PR do?

Add missing field geo.name and azure.platformlogs.result_description field. Also when ingesting, make sure to convert durationMs to integer first if it's a string in the original log.

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.

@kaiyan-sheng kaiyan-sheng requested a review from a team as a code owner April 27, 2022 18:07
@kaiyan-sheng kaiyan-sheng self-assigned this Apr 27, 2022
@elasticmachine
Copy link
Copy Markdown

elasticmachine commented Apr 27, 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-04-29T16:36:16.281+0000

  • Duration: 13 min 16 sec

Test stats 🧪

Test Results
Failed 0
Passed 86
Skipped 0
Total 86

🤖 GitHub comments

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

  • /test : Re-trigger the build.

@elasticmachine
Copy link
Copy Markdown

elasticmachine commented Apr 27, 2022

🌐 Coverage report

Name Metrics % (covered/total) Diff
Packages 100.0% (6/6) 💚
Files 78.571% (11/14) 👎 -18.63
Classes 78.571% (11/14) 👎 -18.63
Methods 76.724% (89/116) 👎 -11.507
Lines 80.46% (1540/1914) 👎 -8.473
Conditionals 100.0% (0/0) 💚

@kaiyan-sheng kaiyan-sheng merged commit 715f939 into elastic:main Apr 29, 2022
@kaiyan-sheng kaiyan-sheng deleted the platformlogs branch April 29, 2022 18:20
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.

3 participants