Skip to content

[AWS][Lambda] Fix AWS Invocations Count Issue#9669

Merged
agithomas merged 6 commits intoelastic:mainfrom
agithomas:issue-1371-invocations-sum-bugfix
Apr 24, 2024
Merged

[AWS][Lambda] Fix AWS Invocations Count Issue#9669
agithomas merged 6 commits intoelastic:mainfrom
agithomas:issue-1371-invocations-sum-bugfix

Conversation

@agithomas
Copy link
Copy Markdown
Contributor

@agithomas agithomas commented Apr 23, 2024 •
edited
Loading

  • Bug

Proposed commit message

Update aggregation function for AWS lambda invocation metric.

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

  • Dashboard validation
  • Discover view validation
  • Field mapping validation
  • Package upgrade test

How to test this PR locally

  • elastic-package build
  • elastic-package stack up -v -d --services package-registry

Screenshots

image image

@elasticmachine
Copy link
Copy Markdown

elasticmachine commented Apr 23, 2024

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@agithomas agithomas marked this pull request as ready for review April 23, 2024 07:37
@agithomas agithomas requested review from a team as code owners April 23, 2024 07:37
dej611
dej611 approved these changes Apr 23, 2024
View reviewed changes
Copy link
Copy Markdown

@dej611 dej611 left a comment

Choose a reason for hiding this comment

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

Code review only 👍

@niraj-elastic
Copy link
Copy Markdown
Contributor

niraj-elastic commented Apr 23, 2024

Let's update the screen shot as well.

@agithomas
Copy link
Copy Markdown
Contributor Author

agithomas commented Apr 23, 2024

Let's update the screen shot as well.

Updated the screenshot.

zmoog
zmoog approved these changes Apr 23, 2024
View reviewed changes
Copy link
Copy Markdown
Contributor

@zmoog zmoog left a comment

Choose a reason for hiding this comment

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

LGTM

@elasticmachine
Copy link
Copy Markdown

elasticmachine commented Apr 23, 2024

💚 Build Succeeded

History

@elastic-sonarqube
Copy link
Copy Markdown

elastic-sonarqube bot commented Apr 23, 2024

Quality Gate passed Quality Gate passed

Issues
Image 0 New issues
Image 0 Fixed issues
Image 0 Accepted issues

Measures
Image 0 Security Hotspots
Image No data about Coverage
Image No data about Duplication

See analysis details on SonarQube

@agithomas agithomas merged commit 87ca061 into elastic:main Apr 24, 2024
@elasticmachine
Copy link
Copy Markdown

elasticmachine commented Apr 24, 2024

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

gizas pushed a commit that referenced this pull request Apr 26, 2024
@agithomas @gizas
[AWS][Lambda] Fix AWS Invocations Count Issue (#9669)
57bb5c6 
* Updated aggregation function for AWS lambda invocation metric.
qcorporation pushed a commit that referenced this pull request Feb 3, 2025
@agithomas @gizas
[AWS][Lambda] Fix AWS Invocations Count Issue (#9669)
e0d1914 
* Updated aggregation function for AWS lambda invocation metric.
qcorporation pushed a commit that referenced this pull request Feb 4, 2025
@agithomas @gizas
[AWS][Lambda] Fix AWS Invocations Count Issue (#9669)
0938574 
* Updated aggregation function for AWS lambda invocation metric.
@agithomas agithomas deleted the issue-1371-invocations-sum-bugfix branch February 12, 2025 04:59
@andrewkroh andrewkroh added the dashboard Relates to a Kibana dashboard bug, enhancement, or modification. label Mar 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@zmoog zmoog zmoog approved these changes

@dej611 dej611 dej611 approved these changes

@muthu-mps muthu-mps muthu-mps approved these changes

+1 more reviewer

@niraj-elastic niraj-elastic niraj-elastic approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

dashboard Relates to a Kibana dashboard bug, enhancement, or modification. Integration:aws AWS

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@agithomas @elasticmachine @niraj-elastic @zmoog @dej611 @muthu-mps @andrewkroh