-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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: master
Are you sure you want to change the base?
Conversation
Test Results (MA/MR) - Preflight, Unit21 613 tests 19 958 ✅ 6m 14s ⏱️ Results for commit 88b413e. ♻️ This comment has been updated with latest results. |
Test Results (amd64, MA/MR) - Acceptance7 tests 5 ✅ 3m 12s ⏱️ Results for commit 88b413e. ♻️ 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, Bootstrap 5 files 5 suites 2h 21m 12s ⏱️ For more details on these failures, see this check. Results for commit 88b413e. ♻️ This comment has been updated with latest results. |
Test Results (amd64) - Integration, Bootstrap 5 files 5 suites 2h 25m 50s ⏱️ For more details on these failures, see this check. Results for commit 88b413e. ♻️ This comment has been updated with latest results. |
LocalStack Community integration with Pro 2 files ±0 2 suites ±0 1h 44m 40s ⏱️ + 1m 22s For more details on these failures, see this check. Results for commit 88b413e. ± Comparison against base commit 3a5f08b. ♻️ 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)
Motivation
Currently a WIP. This PR was created to trigger a CI/CD run with the new Lambda RIE v2 enabled by default.
Changes