Skip to content

Commit

Permalink
Remove experimentation middleware from global middleware list (#4416)
Browse files Browse the repository at this point in the history
While we troubleshoot blocking issues when communicating with our backend experimentation we will remove the experimentation middleware from the global middleware list since we are not running any active experiments.
  • Loading branch information
wbreza authored Oct 8, 2024
1 parent e18ead3 commit 8fb8ca1
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 0 additions & 1 deletion cli/azd/cmd/root.go
Original file line number Diff line number Diff line change
Expand Up @@ -332,7 +332,6 @@ func NewRootCmd(
root.
UseMiddleware("debug", middleware.NewDebugMiddleware).
UseMiddleware("ux", middleware.NewUxMiddleware).
UseMiddleware("experimentation", middleware.NewExperimentationMiddleware).
UseMiddlewareWhen("telemetry", middleware.NewTelemetryMiddleware, func(descriptor *actions.ActionDescriptor) bool {
return !descriptor.Options.DisableTelemetry
})
Expand Down
2 changes: 2 additions & 0 deletions cli/azd/test/functional/experiment_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,8 @@ import (

// Verifies that the assignment context returned is included in the telemetry events we capture.
func Test_CLI_Experiment_AssignmentContextInTelemetry(t *testing.T) {
t.Skip("Skipping while experimentation is not enabled")

// CLI process and working directory are isolated
t.Parallel()
ctx, cancel := newTestContext(t)
Expand Down

0 comments on commit 8fb8ca1

Please sign in to comment.