Skip to content

chore: add gdb debugging support to dev container#1188

Merged
gaoxueyu merged 1 commit intoIvorySQL:masterfrom
rophy:feat/dev-container
Feb 10, 2026
Merged

chore: add gdb debugging support to dev container#1188
gaoxueyu merged 1 commit intoIvorySQL:masterfrom
rophy:feat/dev-container

Conversation

@rophy
Copy link
Copy Markdown
Contributor

@rophy rophy commented Feb 8, 2026 •
edited by coderabbitai bot
Loading

Useful when debugging (Claude knows how to use it).

Summary by CodeRabbit

  • Chores
    • Added debugging support and tools to the development container environment.
    • Updated development artifact ignore patterns for improved consistency.
    • Enhanced development environment configuration to support advanced debugging capabilities.

@rophy
chore: add gdb debugging support to dev container
a72b7c3 
- Add gdb package to Dockerfile
- Enable SYS_PTRACE and seccomp:unconfined in docker-compose
- Fix .gitignore for tmp_install
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Feb 8, 2026 •
edited
Loading

📝 Walkthrough

Walkthrough

This pull request updates the development container configuration to enable debugging capabilities. Changes include installing gdb in the Dockerfile, updating the git ignore pattern for temporary install artifacts, and configuring the docker-compose dev service to build locally with ptrace and security options enabled.

Changes

Cohort / File(s) Summary
Dev Container Configuration
.devcontainer/Dockerfile, docker-compose.yaml		 Added gdb installation for debugging; enabled SYS_PTRACE capability and disabled seccomp security constraints in dev service to support debugger functionality.
Repository Configuration
.gitignore		 Simplified pattern to ignore tmp_install artifacts at repository root level regardless of whether they are files or directories.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • feat: dev container #982: Modifies dev container setup (Dockerfile and docker-compose.yaml dev service) with related configuration changes for the development environment.

Poem

🐰 A burrow debugged with gdb so fine,
SYS_PTRACE unleashed, the code shall shine!
No seccomp walls to hold us back,
Just gdb's power on the dev track. 🔍

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The pull request title accurately reflects the main objective of adding gdb debugging support to the dev container, which is confirmed by changes to Dockerfile, docker-compose.yaml, and .gitignore.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

No actionable comments were generated in the recent review. 🎉

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 YAMLlint (1.38.0)
docker-compose.yaml

[Errno 2] No such file or directory: 'docker-compose.yaml'

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@yuanyl630 yuanyl630 self-requested a review February 9, 2026 00:49
@yuanyl630
Copy link
Copy Markdown
Collaborator

yuanyl630 commented Feb 9, 2026

Thank you again for your contribution to IvorySQL. We willl review your codes soon.

@yuanyl630
Copy link
Copy Markdown
Collaborator

yuanyl630 commented Feb 10, 2026

LGTM, will merge soon.

@gaoxueyu gaoxueyu merged commit 4cab735 into IvorySQL:master Feb 10, 2026
6 checks passed
@yuanyl630 yuanyl630 mentioned this pull request Mar 19, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@yuanyl630 yuanyl630 Awaiting requested review from yuanyl630

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@rophy @yuanyl630 @gaoxueyu