Skip to content

Conversation

dakrone
Copy link
Member

@dakrone dakrone commented May 9, 2022

The telemetry for data tiers was using the size in bytes, however, for the frozen tier using
searchable snapshots, this was the disk usage rather than the size of the actual data.

This commit changes the telemetry to use total_data_set_size as introduced in #70625 so that the
telemetry is correct.

Resolves #86055

The telemetry for data tiers was using the size in bytes, however, for the frozen tier using
searchable snapshots, this was the disk usage rather than the size of the actual data.

This commit changes the telemetry to use `total_data_set_size` as introduced in elastic#70625 so that the
telemetry is correct.

Resolves elastic#86055
@dakrone dakrone added >bug :Data Management/Stats Statistics tracking and retrieval APIs v8.3.0 v8.2.1 v7.17.4 labels May 9, 2022
@elasticmachine elasticmachine added the Team:Data Management Meta label for data/management team label May 9, 2022
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-data-management (Team:Data Management)

@elasticsearchmachine
Copy link
Collaborator

Hi @dakrone, I've created a changelog YAML for you.

@dakrone
Copy link
Member Author

dakrone commented May 9, 2022

@elasticmachine run elasticsearch-ci/part-2

@gmarouli gmarouli self-requested a review May 11, 2022 06:53
Copy link
Contributor

@gmarouli gmarouli left a comment

Choose a reason for hiding this comment

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

LGTM :)

@dakrone
Copy link
Member Author

dakrone commented May 12, 2022

@elasticmachine update branch

@dakrone dakrone added auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) auto-backport-and-merge labels May 12, 2022
@elasticsearchmachine elasticsearchmachine merged commit d5a4586 into elastic:master May 12, 2022
@dakrone dakrone deleted the use-total-data-set-for-tier-telemetry branch May 12, 2022 15:26
dakrone added a commit to dakrone/elasticsearch that referenced this pull request May 12, 2022
…c#86580)

The telemetry for data tiers was using the size in bytes, however, for
the frozen tier using searchable snapshots, this was the disk usage
rather than the size of the actual data.

This commit changes the telemetry to use `total_data_set_size` as
introduced in elastic#70625 so that the telemetry is correct.

Resolves elastic#86055
dakrone added a commit to dakrone/elasticsearch that referenced this pull request May 12, 2022
…c#86580)

The telemetry for data tiers was using the size in bytes, however, for
the frozen tier using searchable snapshots, this was the disk usage
rather than the size of the actual data.

This commit changes the telemetry to use `total_data_set_size` as
introduced in elastic#70625 so that the telemetry is correct.

Resolves elastic#86055
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
8.2
7.17

elasticsearchmachine pushed a commit that referenced this pull request May 12, 2022
#86749)

The telemetry for data tiers was using the size in bytes, however, for
the frozen tier using searchable snapshots, this was the disk usage
rather than the size of the actual data.

This commit changes the telemetry to use `total_data_set_size` as
introduced in #70625 so that the telemetry is correct.

Resolves #86055
elasticsearchmachine pushed a commit that referenced this pull request May 12, 2022
#86748)

The telemetry for data tiers was using the size in bytes, however, for
the frozen tier using searchable snapshots, this was the disk usage
rather than the size of the actual data.

This commit changes the telemetry to use `total_data_set_size` as
introduced in #70625 so that the telemetry is correct.

Resolves #86055
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) >bug :Data Management/Stats Statistics tracking and retrieval APIs Team:Data Management Meta label for data/management team v7.17.4 v8.2.1 v8.3.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Data tiers usage is 0 for partially mounted indices
4 participants