Skip to content

ES|QL: fix telemetry tests after FORK release #129983

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

luigidellaquila
Copy link
Contributor

Fixing non-snapshot telemetry, after we promoted FORK for release, see #129606

Fixes: #129888

@luigidellaquila luigidellaquila requested a review from astefan June 25, 2025 07:56
@luigidellaquila luigidellaquila added >test Issues or PRs that are addressing/adding tests test-release Trigger CI checks against release build :Analytics/ES|QL AKA ESQL labels Jun 25, 2025
@elasticsearchmachine elasticsearchmachine added Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v9.1.0 labels Jun 25, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-analytical-engine (Team:Analytics)

Copy link
Contributor

@astefan astefan left a comment

Choose a reason for hiding this comment

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

LGTM

@luigidellaquila luigidellaquila enabled auto-merge (squash) June 25, 2025 08:33
@luigidellaquila
Copy link
Contributor Author

Also this #129839 didn't account for release tests

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Analytics/ES|QL AKA ESQL Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests test-release Trigger CI checks against release build v9.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CI] XPackRestIT test {p0=esql/60_usage/Basic ESQL usage output (telemetry) non-snapshot version} failing
3 participants