feat: support image digest filtering and bump IG to v0.51.1#111
Conversation
There was a problem hiding this comment.
Pull request overview
Adds support for excluding containers by image digest in micromize’s LocalManager filtering, and updates Inspektor Gadget (and related Go module dependencies) to the versions needed for that capability.
Changes:
- Add
--filter-image-digestCLI flag and wire it into the gadget operator params. - Expose the digest filter via Helm values (
image.digest) and DaemonSet args; update README usage. - Bump Inspektor Gadget to
v0.51.1and refresh Go module dependencies accordingly.
Reviewed changes
Copilot reviewed 5 out of 6 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| go.mod | Updates IG + dependency versions; adds an exclude for an older moby/moby version. |
| go.sum | Syncs module checksums with dependency upgrades/removals. |
| cmd/micromize/root.go | Adds digest filtering flag and passes new LocalManager param at runtime. |
| charts/micromize/values.yaml | Adds image.digest value for configuring digest-based filtering. |
| charts/micromize/templates/daemonset.yaml | Passes --filter-image-digest when image.digest is set. |
| README.md | Documents digest resolution and the new CLI flag. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| if filterImageDigest != "" { | ||
| commonParams["operator.LocalManager.runtime-containerimage-digest"] = "!" + filterImageDigest | ||
| slog.Info("Filtering out containers by image digest", "digest", filterImageDigest) | ||
| } |
| # Image digest for self-exclusion filtering. When set, micromize filters out | ||
| # containers running this digest from monitoring. Resolve with: | ||
| # crane digest ghcr.io/micromize-dev/micromize:<tag> | ||
| digest: "" |
072d58a to
3a91ccb
Compare
Signed-off-by: Dor Serero <dor.serero@gmail.com>
3a91ccb to
4bac32c
Compare
There was a problem hiding this comment.
Pull request overview
Adds image-digest based self-exclusion filtering support while updating Inspektor Gadget (and related dependencies) to newer versions.
Changes:
- Bump
github.com/inspektor-gadget/inspektor-gadgettov0.51.1and refresh a set of Go module dependencies. - Add a new CLI flag
--filter-image-digestand wire it into the gadget/operator parameters. - Extend the Helm chart + README to support/configure the new digest-based filtering.
Reviewed changes
Copilot reviewed 6 out of 7 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| go.mod | Bumps core dependencies (IG, ebpf, docker/cli, x/sync, etc.) and adds an exclude for an older moby/moby module version. |
| go.sum | Updates checksums to reflect the dependency graph changes from the version bumps. |
| cmd/micromize/root.go | Introduces --filter-image-digest and passes digest filtering to the LocalManager operator params; adjusts namespace filter help text. |
| charts/micromize/values.yaml | Adds image.digest value to configure digest-based self-exclusion via Helm. |
| charts/micromize/values.schema.json | Extends chart schema to include the new image.digest field. |
| charts/micromize/templates/daemonset.yaml | Passes --filter-image-digest to the DaemonSet container args when configured. |
| README.md | Documents how to resolve/set the digest for Helm installs and lists the new CLI flag. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.