Skip to content

Commit

Permalink
Replacing the check with 200 OK for HTTP/2 200
Browse files Browse the repository at this point in the history
As we can see we are able to get the data as before, see: https://github.com/operator-framework/operator-controller/actions/runs/12322365651/job/34395777693?pr=1475

On the discussions where defined that we can move forward for HTTP/2
  • Loading branch information
camilamacedo86 committed Dec 13, 2024
1 parent 5342759 commit 21eb405
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion test/e2e/metrics_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -87,5 +87,5 @@ func TestOperatorControllerMetricsExportedEndpoint(t *testing.T) {
"curl", "-v", "-k", "-H", "Authorization: Bearer "+token, metricsURL)
output, err = curlCmd.CombinedOutput()
require.NoError(t, err, "Error calling metrics endpoint: %s", string(output))
require.Contains(t, string(output), "200 OK", "Metrics endpoint did not return 200 OK")
require.Contains(t, string(output), "HTTP/2 200", "Metrics endpoint did not return HTTP/2 200")
}

0 comments on commit 21eb405

Please sign in to comment.