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

[AWS] Add metric type to VPN, Firewall and Transit Gateway data streams #6365

Merged

Conversation

constanca-m
Copy link
Contributor

What does this PR do?

Add metric type to VPN, Firewall and Transit Gateway data streams.

All metrics are gauge, since they go up and down and might not relate to the previous values. For example, some metrics may have a specific time window of 1 minute. For that minute, all values are summed to obtain the final value of a field, but as soon as that time window finishes, the value restarts again. There is no guarantee that the next value will be equal or higher than the previous value.

Author's Checklist

  • Verification of data in visualisation after enabling TSDB flag in kibana
  • Verify if field mapping is correct in the data stream template.

How to test this PR locally

Refer to #6293.

Related issues

Relates to #6293.

Signed-off-by: constanca-m <[email protected]>
Signed-off-by: constanca-m <[email protected]>
Signed-off-by: constanca-m <[email protected]>
@constanca-m constanca-m requested a review from a team as a code owner May 30, 2023 09:46
Signed-off-by: constanca-m <[email protected]>
@elasticmachine
Copy link

elasticmachine commented May 30, 2023

💚 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: 2023-06-05T11:37:24.921+0000

  • Duration: 49 min 0 sec

Test stats 🧪

Test Results
Failed 0
Passed 188
Skipped 4
Total 192

🤖 GitHub comments

Expand to view the GitHub comments

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

  • /test : Re-trigger the build.

Signed-off-by: constanca-m <[email protected]>
@elasticmachine
Copy link

elasticmachine commented May 30, 2023

🌐 Coverage report

Name Metrics % (covered/total) Diff
Packages 100.0% (15/15) 💚
Files 93.75% (15/16) 👎 -6.25
Classes 93.75% (15/16) 👎 -6.25
Methods 86.131% (236/274) 👎 -13.869
Lines 85.925% (7387/8597) 👎 -9.419
Conditionals 100.0% (0/0) 💚

@constanca-m
Copy link
Contributor Author

/test

@constanca-m constanca-m merged commit 69dbddf into elastic:main Jun 5, 2023
@constanca-m constanca-m deleted the aws-add-metric-type-vpn-firewall-transit branch June 5, 2023 12:27
@elasticmachine
Copy link

Package aws - 1.38.2 containing this change is available at https://epr.elastic.co/search?package=aws

sodhikirti07 pushed a commit that referenced this pull request Jun 15, 2023
…ms (#6365)

* Add metric type to VPN.

Signed-off-by: constanca-m <[email protected]>

* Add metric type to Firewall.

Signed-off-by: constanca-m <[email protected]>

* Add metric type to Transit gateway.

Signed-off-by: constanca-m <[email protected]>

* Update version.

Signed-off-by: constanca-m <[email protected]>

* Update changelog.

Signed-off-by: constanca-m <[email protected]>

* Update md files

Signed-off-by: constanca-m <[email protected]>

---------

Signed-off-by: constanca-m <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants