-
Notifications
You must be signed in to change notification settings - Fork 695
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
Devcontainer: enable coredumps #7523
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #7523 +/- ##
==========================================
- Coverage 89.61% 89.51% -0.10%
==========================================
Files 282 282
Lines 60375 60373 -2
Branches 7522 7522
==========================================
- Hits 54105 54045 -60
- Misses 4118 4148 +30
- Partials 2152 2180 +28 |
.devcontainer/devcontainer.json
Outdated
@@ -1,9 +1,12 @@ | |||
{ | |||
"image": "ghcr.io/citusdata/citus-devcontainer:main", | |||
"image": "ghcr.io/citusdata/citus-devcontainer:devcontainer-coredumps", |
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.
TODO: before merging, revert image tag
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.
Looks good to me. I successfully obtained the core-dump file and opened it using the added extension.
Co-authored-by: Jelte Fennema-Nio <[email protected]>
b6be08e
to
6fd5cbe
Compare
Add configuration for coredumps and document how to make sure they are enabled when developing in a devcontainer. --------- Co-authored-by: Jelte Fennema-Nio <[email protected]>
Add configuration for coredumps and document how to make sure they are enabled.