Skip to content

Add enclave host execution telemetry - #5

Merged
vreff merged 12 commits into
mainfrom
opentel
Jul 23, 2026
Merged

Add enclave host execution telemetry#5
vreff merged 12 commits into
mainfrom
opentel

Conversation

@cfal

@cfal cfal commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Adds OpenTelemetry metrics to the Nitro enclave host. The host now reports execution latency, total in-flight executions, and per-workflow active state, using static environment-provided authentication and graceful client shutdown.

@cfal
cfal requested a review from a team as a code owner July 20, 2026 12:47
@cfal
cfal marked this pull request as draft July 20, 2026 12:47
@github-actions

github-actions Bot commented Jul 20, 2026

Copy link
Copy Markdown

AI dashboard check

@cfal the AI Stupid Level dashboard is not currently showing any tracked models at score 55 or lower in the last 24 hours.

Tracked models currently not flagged
  • GPT-5.4: gpt-5.4 (score 62, status warning, trend down)
  • Claude Opus 4.5: claude-opus-4-5-20251101 (score 63, status warning, trend down)
  • Claude Opus 4.6: claude-opus-4-6 (score 68, status good, trend stable)
  • Claude Opus 4.7: claude-opus-4-7 (score 69, status good, trend up)
  • Claude Sonnet 4.5: claude-sonnet-4-5-20250929 (score 62, status warning, trend up)
  • Claude Sonnet 4.6: claude-sonnet-4-6 (score 65, status good, trend stable)

To rerun this check on demand, comment /rerun-ai-dashboard-warning on this PR.

Snapshot: 2026-07-21T10:33:03.953Z from https://aistupidlevel.info (24h view).

@github-actions

Copy link
Copy Markdown

👋 cfal, thanks for creating this pull request!

To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team.

Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks!

@cfal
cfal force-pushed the opentel branch 2 times, most recently from 154c4f3 to 834e911 Compare July 21, 2026 12:40
@cfal
cfal marked this pull request as ready for review July 21, 2026 12:51
Comment thread enclave/nitro/host/host.go Outdated
)

// Leave process-teardown headroom within Kubernetes' default 30-second grace period.
const gracefulShutdownTimeout = 25 * time.Second

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

nit: this could be an argument/envar

@cfal
cfal requested a review from vreff July 22, 2026 09:54
@cfal
cfal marked this pull request as draft July 22, 2026 10:27
@vreff

vreff commented Jul 23, 2026

Copy link
Copy Markdown
Collaborator

@cfal there seems to be a vulnerability in the PR.

<h1>Dependency Review</h1>
The following issues were found:<ul><li>❌ 1 vulnerable package(s)</li></ul>
See the Details below.<h2>Vulnerabilities</h2>
<h4><em>enclave/nitro/host/go.mod</em></h4>
<markdown-accessiblity-table data-catalyst="">
Name | Version | Vulnerability | Severity
-- | -- | -- | --
google.golang.org/grpc | 1.80.0 | gRPC-Go: xDS RBAC and HTTP/2 Vulnerabilities | high

</markdown-accessiblity-table>
<blockquote>Only included vulnerabilities with severity <strong>high</strong> or higher</blockquote>Dependency Review
The following issues were found:

    ❌ 1 vulnerable package(s)

See the Details below.
Vulnerabilities
enclave/nitro/host/go.mod
Name	Version	Vulnerability	Severity
[google.golang.org/grpc](https://github.com/grpc/grpc-go)	1.80.0	[gRPC-Go: xDS RBAC and HTTP/2 Vulnerabilities](https://github.com/advisories/GHSA-hrxh-6v49-42gf)	high

    Only included vulnerabilities with severity high or higher

@cfal
cfal force-pushed the opentel branch 2 times, most recently from e7e8d45 to 1ee2014 Compare July 23, 2026 18:53
@cfal
cfal marked this pull request as ready for review July 23, 2026 19:12
@vreff
vreff merged commit d430a07 into main Jul 23, 2026
39 checks passed
@vreff
vreff deleted the opentel branch July 23, 2026 22:26
vreff added a commit that referenced this pull request Jul 23, 2026
Add enclave host execution telemetry
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