Merged
Conversation
nddq
force-pushed
the
chore/improve-devcontainer-experience
branch
from
104dc35 to
d106017
Compare
Retina Code Coverage ReportTotal coverage no change |
nddq
force-pushed
the
chore/improve-devcontainer-experience
branch
9 times, most recently
from
00e6787 to
065bf0f
Compare
- Pin devcontainer feature versions for reproducibility - Replace installMoreTools.sh with declarative onCreateCommand - Add LLVM 17 feature for eBPF compilation - Add Go module/build cache volume mounts - Add Kind cluster auto-creation on startup - Configure VS Code settings for golangci-lint and gofumpt - Add port forwarding for Hubble and Retina metrics - Add CI workflow to validate devcontainer builds - Update contributing docs with devcontainer instructions - Fix markdownlint warnings in bpftrace troubleshooting doc Signed-off-by: Quang Nguyen <nguyenquang@microsoft.com>
nddq
force-pushed
the
chore/improve-devcontainer-experience
branch
from
065bf0f to
a8dbd3d
Compare
nddq
changed the title
alexcastilio
approved these changes
Mar 13, 2026
github-merge-queue
bot
removed this pull request from the merge queue due to failed status checks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
The devcontainer is missing several tools needed for development (LLVM/clang, jq), uses a shell script for setup instead of declarative devcontainer features, and has no CI validation.
This PR overhauls the devcontainer:
installMoreTools.shwith declarative devcontainer features (LLVM 17, pinned versions)devcontainer.yamlCI workflow that builds the container and validates toolingRelated Issue
Partially addresses #2027 (lint alignment will be handled in a follow-up PR)
Checklist
git commit -S -s ...).Screenshots (if applicable) or Testing Completed
CI will validate the devcontainer builds and all tools are available via the new
devcontainer.yamlworkflow.