Skip to content

Telemetry test cases support 8.10 and later Kibana versions#7150

Merged
rhr323 merged 6 commits intoelastic:mainfrom
rhr323:fix-telemetry-e2e-test
Sep 7, 2023
Merged

Telemetry test cases support 8.10 and later Kibana versions#7150
rhr323 merged 6 commits intoelastic:mainfrom
rhr323:fix-telemetry-e2e-test

Conversation

@rhr323
Copy link
Copy Markdown
Contributor

@rhr323 rhr323 commented Sep 4, 2023

This PR updates the Telemetry test suite.

Since 8.10, the telemetry API URLs have changed: elastic/kibana#159839

Old URL: /api/telemetry/%s/clusters/_stats
New URL: /internal/telemetry/clusters/_stats

There's a simple branching which this PR adds that will generate the appropriate URL based on the stack version. For the new API a few extra headers are also added to the HTTP request.

@botelastic botelastic bot added the triage label Sep 4, 2023
@rhr323
Copy link
Copy Markdown
Contributor Author

rhr323 commented Sep 4, 2023

buildkite test this -f p=kind,t=TestTelemetry

@rhr323
Copy link
Copy Markdown
Contributor Author

rhr323 commented Sep 5, 2023

buildkite test this -f p=gke,t=TestTelemetry,s=8.10.0-SNAPSHOT

@thbkrkr thbkrkr added >bug Something isn't working >test Related to unit/integration/e2e tests and removed triage labels Sep 5, 2023
@rhr323
Copy link
Copy Markdown
Contributor Author

rhr323 commented Sep 5, 2023

buildkite test this -f p=kind,t=TestTelemetry,s=8.10.0-SNAPSHOT

@rhr323
Copy link
Copy Markdown
Contributor Author

rhr323 commented Sep 6, 2023

buildkite test this -f p=kind,t=TestTelemetry,s=8.10.0-SNAPSHOT

@rhr323
Copy link
Copy Markdown
Contributor Author

rhr323 commented Sep 6, 2023

buildkite test this -f p=kind,t=TestTelemetry,s=8.10.0-SNAPSHOT

@rhr323 rhr323 marked this pull request as ready for review September 6, 2023 10:02
@rhr323 rhr323 changed the title [WIP] Telemetry API url supports 8.10 and later Kibana versions Telemetry test cases support 8.10 and later Kibana versions Sep 6, 2023
return StackStats{}, err
}

// a few extra headers a required by Kibana for this internal API
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
// a few extra headers a required by Kibana for this internal API
// a few extra headers are required by Kibana for this internal API

@thbkrkr thbkrkr added the v2.10.0 label Sep 6, 2023
@rhr323 rhr323 merged commit 9c06d76 into elastic:main Sep 7, 2023
@rhr323 rhr323 deleted the fix-telemetry-e2e-test branch September 7, 2023 07:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>bug Something isn't working >test Related to unit/integration/e2e tests v2.10.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants