Skip to content

feat: scan containers on host during daemon initialisation - #623

Merged
VaibhavUpreti merged 2 commits into
mainfrom
fix_unwrap_otel
Dec 19, 2025
Merged

feat: scan containers on host during daemon initialisation#623
VaibhavUpreti merged 2 commits into
mainfrom
fix_unwrap_otel

Conversation

@VaibhavUpreti

Copy link
Copy Markdown
Member

🧪 Testing

To install tracer with this version/branch, run:

curl -sSL https://install.tracer.cloud | CLI_BRANCH="fix_unwrap_otel" sh

To use the installer of tracer with this version/branch, run:

curl -sSL https://install.tracer.cloud | INS_BRANCH="fix_unwrap_otel" sh

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Dec 18, 2025

Copy link
Copy Markdown

Deploying tracer-client with  Cloudflare Pages  Cloudflare Pages

Latest commit: e77cf21
Status: ✅  Deploy successful!
Preview URL: https://fe6e2f3a.tracer-client.pages.dev
Branch Preview URL: https://fix-unwrap-otel.tracer-client.pages.dev

View logs

@MicheleVerriello MicheleVerriello left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you can merge if you want

Comment thread src/tracer/src/extracts/containers/docker_watcher/watcher.rs Outdated
.as_ref()
.and_then(|cfg| cfg.image.clone())
.unwrap_or_default(),
ip: inspect.network_settings.and_then(|net| {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we don't really use these, so we may completely remove them from the ContainerEvent

@VaibhavUpreti
VaibhavUpreti merged commit d77e92b into main Dec 19, 2025
15 checks passed
@VaibhavUpreti
VaibhavUpreti deleted the fix_unwrap_otel branch December 19, 2025 19:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants