From a351ec0010242f16ee5505f94ee1a95d8a006fee Mon Sep 17 00:00:00 2001 From: AdaAibaby Date: Fri, 29 May 2026 11:34:42 +0800 Subject: [PATCH] feat(orchestrator): always include execution metrics in sandbox webhook events --- packages/orchestrator/pkg/server/sandboxes.go | 8 ++------ packages/shared/pkg/featureflags/flags.go | 1 - 2 files changed, 2 insertions(+), 7 deletions(-) diff --git a/packages/orchestrator/pkg/server/sandboxes.go b/packages/orchestrator/pkg/server/sandboxes.go index f0ad1901cc..870be9d5a8 100644 --- a/packages/orchestrator/pkg/server/sandboxes.go +++ b/packages/orchestrator/pkg/server/sandboxes.go @@ -456,9 +456,7 @@ func (s *Server) Delete(ctxConn context.Context, in *orchestrator.SandboxDeleteR }() teamID, buildId, eventData := s.prepareSandboxEventData(ctx, sbx) - if s.featureFlags.BoolFlag(ctx, featureflags.ExecutionMetricsOnWebhooksFlag) { - eventData[executionEventDataKey] = s.getSandboxExecutionData(sbx) - } + eventData[executionEventDataKey] = s.getSandboxExecutionData(sbx) eventType := events.SandboxKilledEventPair go s.sbxEventsService.Publish( @@ -524,9 +522,7 @@ func (s *Server) Pause(ctx context.Context, in *orchestrator.SandboxPauseRequest s.uploadSnapshotAsync(ctx, sbx, res) teamID, buildId, eventData := s.prepareSandboxEventData(ctx, sbx) - if s.featureFlags.BoolFlag(ctx, featureflags.ExecutionMetricsOnWebhooksFlag) { - eventData[executionEventDataKey] = s.getSandboxExecutionData(sbx) - } + eventData[executionEventDataKey] = s.getSandboxExecutionData(sbx) eventType := events.SandboxPausedEventPair go s.sbxEventsService.Publish( diff --git a/packages/shared/pkg/featureflags/flags.go b/packages/shared/pkg/featureflags/flags.go index d8d5279c72..8ab3e8b4db 100644 --- a/packages/shared/pkg/featureflags/flags.go +++ b/packages/shared/pkg/featureflags/flags.go @@ -120,7 +120,6 @@ var ( PeerToPeerAsyncCheckpointFlag = NewBoolFlag("peer-to-peer-async-checkpoint", false) PersistentVolumesFlag = NewBoolFlag("can-use-persistent-volumes", env.IsDevelopment()) - ExecutionMetricsOnWebhooksFlag = NewBoolFlag("execution-metrics-on-webhooks", false) // TODO: Remove NLT 20250315 SandboxLabelBasedSchedulingFlag = NewBoolFlag("sandbox-label-based-scheduling", false) OptimisticResourceAccountingFlag = NewBoolFlag("sandbox-placement-optimistic-resource-accounting", false) FreePageReportingFlag = NewBoolFlag("free-page-reporting", false)