Skip to content

Conversation

@TotalDwarf03
Copy link
Contributor

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update

What

The lambda has been failing to run on AWS due to running out of memory.
As a temporary fix, I have increased from 128MB to 256MB. This works fine.

As an aside in this PR, I will look to improve its memory usage and make it more efficient (TODO in another PR).

Testing

Have any new tests been added as part of this issue? If not, try to explain why test coverage is not needed here.

  • Yes
  • No
    Please write a brief description of why test coverage is not necessary here.
  • Not as part of this ticket. (Could be done at a later point)

Documentation

Has any new documentation been written as part of this issue? We should try to keep documentation up to date
as new code is added, rather than leaving it for the future.

  • Yes
  • No
    Please write a brief description of why documentation is not necessary here.
  • Not as part of this ticket. (Could be done at a later point)

Related issues

KEH-1795 (Jira)

How to review

Describe the steps required to test the changes.

@TotalDwarf03 TotalDwarf03 self-assigned this Dec 17, 2025
@TotalDwarf03 TotalDwarf03 added the bug Something isn't working label Dec 17, 2025
@TotalDwarf03 TotalDwarf03 requested a review from a team December 17, 2025 17:36
@github-actions
Copy link
Contributor

🦙 MegaLinter status: ⚠️ WARNING

Descriptor Linter Files Fixed Errors Warnings Elapsed time
⚠️ BASH bash-exec 6 4 0 0.04s
✅ BASH shellcheck 6 0 0 0.14s
✅ BASH shfmt 6 0 0 0.01s
✅ CSHARP csharpier 1 0 0 0.57s
✅ DOCKERFILE hadolint 1 0 0 0.12s
✅ JSON jsonlint 2 0 0 0.2s
✅ JSON prettier 2 0 0 0.59s
✅ JSON v8r 2 0 0 3.15s
✅ MARKDOWN markdownlint 6 0 0 1.0s
✅ MARKDOWN markdown-table-formatter 6 0 0 0.43s
✅ REPOSITORY checkov yes no no 28.44s
✅ REPOSITORY dustilock yes no no 0.02s
✅ REPOSITORY gitleaks yes no no 7.61s
✅ REPOSITORY grype yes no no 55.14s
✅ REPOSITORY kics yes no no 49.04s
✅ REPOSITORY secretlint yes no no 1.11s
✅ REPOSITORY syft yes no no 3.78s
✅ REPOSITORY trivy yes no no 16.26s
✅ REPOSITORY trivy-sbom yes no no 0.56s
✅ TERRAFORM terraform-fmt 6 0 0 0.42s
✅ YAML prettier 8 0 0 1.17s
✅ YAML v8r 8 0 0 7.36s
✅ YAML yamllint 8 0 0 0.59s

See detailed report in MegaLinter reports

You could have same capabilities but better runtime performances if you request a new MegaLinter flavor.

MegaLinter is graciously provided by OX Security

Copy link
Member

@smstone0 smstone0 left a comment

Choose a reason for hiding this comment

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

LGTM.

I assume this is a recent bug - has anything changed with the lambda that could cause this?

@TotalDwarf03
Copy link
Contributor Author

LGTM.

I assume this is a recent bug - has anything changed with the lambda that could cause this?

The lambda is just working with more data

@TotalDwarf03 TotalDwarf03 merged commit 63b8dd4 into main Dec 23, 2025
5 checks passed
@TotalDwarf03 TotalDwarf03 deleted the keh-1795-lambda-memory branch December 23, 2025 10:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants