Skip to content

Commit b7da50b

Browse files
sitolee2b-bot[bot]
authored andcommitted
feat(shared): send service, environment and version to LaunchDarkly as application metadata
GitOrigin-RevId: ac77659d42ece651465debdae97a7e5dcf996dec
1 parent 0269182 commit b7da50b

27 files changed

Lines changed: 127 additions & 49 deletions

File tree

‎packages/api/internal/handlers/sandbox_create_fcgate_test.go‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ func TestPostSandboxes_FsOnlyAutoPauseVersionGate(t *testing.T) {
4040
teamID := testutils.CreateTestTeam(t, db)
4141
teamSlug := testutils.GetTeamSlug(t, ctx, db, teamID)
4242

43-
flags, err := featureflags.NewClientWithLogLevel("", "", ldlog.Error)
43+
flags, err := featureflags.NewClientWithLogLevel("", "", "", ldlog.Error)
4444
require.NoError(t, err)
4545

4646
store := &APIStore{

‎packages/api/internal/orchestrator/evictor/evict_test.go‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ func TestEvictSandbox_ReasonByAction(t *testing.T) {
3131
// Offline flags client: resolution falls back to the built-in
3232
// FirecrackerVersionMap, so each declared version resolves within its
3333
// own line — exactly the degrade semantics under test.
34-
flags, err := featureflags.NewClientWithLogLevel("", "", ldlog.Error)
34+
flags, err := featureflags.NewClientWithLogLevel("", "", "", ldlog.Error)
3535
require.NoError(t, err)
3636

3737
counter, err := telemetry.GetCounter(noop.NewMeterProvider().Meter("github.com/e2b-dev/infra/packages/api/internal/orchestrator/evictor"), telemetry.ApiEvictorFsOnlyAutoPause)
@@ -308,7 +308,7 @@ func TestEvictSandbox_DegradeDecision(t *testing.T) {
308308

309309
var flags *featureflags.Client
310310
if sw.budgetMs == nil {
311-
flags, err = featureflags.NewClientWithLogLevel("", "", ldlog.Error)
311+
flags, err = featureflags.NewClientWithLogLevel("", "", "", ldlog.Error)
312312
} else {
313313
td := ldtestdata.DataSource()
314314
td.Update(td.Flag(featureflags.AutoPauseOverstayBudgetMs.Key()).ValueForAll(ldvalue.Int(*sw.budgetMs)))

‎packages/api/main.go‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -414,7 +414,7 @@ func run() int {
414414
return redisClient.Close()
415415
})
416416

417-
featureFlags, err := featureflags.NewClient(config.DeploymentEnvironment, serviceName)
417+
featureFlags, err := featureflags.NewClient(config.DeploymentEnvironment, serviceName, serviceVersion)
418418
if err != nil {
419419
logger.L().Fatal(ctx, "failed to create feature flags client", zap.Error(err))
420420
}

‎packages/client-proxy/main.go‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,7 @@ func run() int {
100100

101101
l.Info(ctx, "Starting client proxy", zap.String("commit", commitSHA), zap.String("instance_id", instanceID))
102102

103-
featureFlagsClient, err := featureflags.NewClient(config.DeploymentEnvironment, serviceName)
103+
featureFlagsClient, err := featureflags.NewClient(config.DeploymentEnvironment, serviceName, version)
104104
if err != nil {
105105
l.Error(ctx, "Failed to create feature flags client", zap.Error(err))
106106

‎packages/dashboard-api/main.go‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -145,7 +145,7 @@ func run() int {
145145
}
146146
defer authDB.Close()
147147

148-
featureFlags, err := featureflags.NewClient(config.DeploymentEnvironment, serviceName)
148+
featureFlags, err := featureflags.NewClient(config.DeploymentEnvironment, serviceName, serviceVersion)
149149
if err != nil {
150150
l.Error(ctx, "Initializing feature flags client", zap.Error(err))
151151

‎packages/orchestrator/benchmarks/benchmark_test.go‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -157,7 +157,7 @@ func BenchmarkBaseImageLaunch(b *testing.B) {
157157
assert.NoError(b, err)
158158
})
159159

160-
featureFlags, err := featureflags.NewClient("", "")
160+
featureFlags, err := featureflags.NewClient("", "", "")
161161
require.NoError(b, err)
162162
b.Cleanup(func() {
163163
ctx := context.WithoutCancel(b.Context())

‎packages/orchestrator/benchmarks/concurrent_benchmark_test.go‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -235,7 +235,7 @@ func BenchmarkConcurrentResume(b *testing.B) {
235235
assert.NoError(b, devicePool.Close(ctx))
236236
})
237237

238-
featureFlags, err := featureflags.NewClient("", "")
238+
featureFlags, err := featureflags.NewClient("", "", "")
239239
require.NoError(b, err)
240240
b.Cleanup(func() {
241241
ctx := context.WithoutCancel(b.Context())

‎packages/orchestrator/cmd/clean-nfs-cache/main.go‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -129,7 +129,7 @@ func configure(ctx context.Context) (cleaner.Options, logger.Logger, *telemetry.
129129
return opts, nil, nil, nil, nil, fmt.Errorf("could not parse feature flags config: %w", err)
130130
}
131131

132-
ffc, err := featureflags.NewClient(ffConfig.DeploymentEnvironment, serviceName)
132+
ffc, err := featureflags.NewClient(ffConfig.DeploymentEnvironment, serviceName, serviceVersion)
133133
if err != nil {
134134
return opts, nil, nil, nil, nil, err
135135
}

‎packages/orchestrator/cmd/create-build/main.go‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -290,7 +290,7 @@ func doBuild(
290290
if verbose {
291291
logLevel = ldlog.Info
292292
}
293-
featureFlags, _ := featureflags.NewClientWithLogLevel(builderConfig.DeploymentEnvironment, "", logLevel)
293+
featureFlags, _ := featureflags.NewClientWithLogLevel(builderConfig.DeploymentEnvironment, "", "", logLevel)
294294

295295
sandboxes := sandbox.NewSandboxesMap()
296296

‎packages/orchestrator/cmd/inspect-build/validate.go‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -178,7 +178,7 @@ func openChunker(ctx context.Context, storagePath, buildID, artifact string, h *
178178
return nil, nil, 0, nil, fmt.Errorf("get data size: %w", err)
179179
}
180180

181-
flags, err := featureflags.NewClient("", "")
181+
flags, err := featureflags.NewClient("", "", "")
182182
if err != nil {
183183
return nil, nil, 0, nil, fmt.Errorf("feature flags: %w", err)
184184
}

0 commit comments

Comments
 (0)