-
-
Notifications
You must be signed in to change notification settings - Fork 4.3k
WIP: feat(lambda): Use Lambda Runtime Emulator Interface v2 #12736
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
base: main
Are you sure you want to change the base?
Conversation
Test Results - Preflight, Unit22 145 tests 20 407 ✅ 6m 42s ⏱️ Results for commit 4ea8ce2. ♻️ This comment has been updated with latest results. |
Test Results (MA/MR) - Preflight, Unit22 145 tests 20 407 ✅ 7m 0s ⏱️ Results for commit 4ea8ce2. ♻️ This comment has been updated with latest results. |
Test Results (amd64) - Acceptance0 tests 0 ✅ 0s ⏱️ Results for commit 4ea8ce2. ♻️ This comment has been updated with latest results. |
Test Results (amd64, MA/MR) - Acceptance0 tests 0 ✅ 0s ⏱️ Results for commit 4ea8ce2. ♻️ This comment has been updated with latest results. |
Test Results (MA/MR) - Alternative Providers987 tests 584 ✅ 23m 18s ⏱️ Results for commit 88b413e. ♻️ This comment has been updated with latest results. |
Test Results - Alternative Providers987 tests 584 ✅ 22m 18s ⏱️ Results for commit 88b413e. ♻️ This comment has been updated with latest results. |
Test Results (amd64, MA/MR) - Integration, Bootstrap24 tests 23 ✅ 11m 38s ⏱️ For more details on these failures, see this check. Results for commit 4ea8ce2. ♻️ This comment has been updated with latest results. |
Test Results (amd64) - Integration, Bootstrap24 tests 23 ✅ 11m 57s ⏱️ For more details on these failures, see this check. Results for commit 4ea8ce2. ♻️ This comment has been updated with latest results. |
LocalStack Community integration with Pro 2 files ± 0 2 suites ±0 40s ⏱️ - 1h 42m 14s For more details on these errors, see this check. Results for commit f79554a. ± Comparison against base commit 5783fa3. This pull request removes 4894 and adds 25 tests. Note that renamed tests count towards both.♻️ This comment has been updated with latest results. |
| "errorMessage": "[Errno 13] Permission denied: '/proc/1/environ'", | ||
| "errorType": "PermissionError", | ||
| "requestId": "<uuid:2>", | ||
| "stackTrace": [ | ||
| " File \"/var/task/handler.py\", line 3, in handler\n with open(f\"/proc/{pid}/environ\", mode=\"rt\") as f:\n" | ||
| ] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we parity-test/inspect this, or does the permission prohibit that?
If not, @dfangl might have a better guess. Maybe, try 600 🤷♂️
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the tests makes no longer sense then - it was mean to introspect the environment of the init process, if that is no longer possible in AWS, there is no real point (although we could try to hide it in our containers as well, I don't necessarily see this as a priority)
|
Currently, only patch changes are allowed on main. Your PR labels (aws:lambda, semver: minor) indicate that it cannot be merged into the main at this time. |
Motivation
Currently a WIP. This PR was created to trigger a CI/CD run with the new Lambda RIE v2 enabled by default.
Changes