Skip to content
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

Autoscaling capacity should be in bytes #66351

Conversation

henningandersen
Copy link
Contributor

The autoscaling storage and memory capacities returned were
inadvertently returned as capacities with units. This imposes a burden
on the caller to be able to parse ES byte-size units. Fixed this to
just return the bytes as numbers.

The autoscaling storage and memory capacities returned were
inadvertently returned as capacities with units. This imposes a burden
on the caller to be able to parse ES byte-size units. Fixed this to
just return the bytes as numbers.
@elasticmachine elasticmachine added the Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. label Dec 15, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed (Team:Distributed)

Copy link
Member

@jasontedor jasontedor left a comment

Choose a reason for hiding this comment

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

LGTM.

@henningandersen henningandersen merged commit bd85855 into elastic:master Dec 15, 2020
henningandersen added a commit that referenced this pull request Dec 15, 2020
The autoscaling storage and memory capacities returned were
inadvertently returned as capacities with units. This imposes a burden
on the caller to be able to parse ES byte-size units. Fixed this to
just return the bytes as numbers.
barkbay added a commit to barkbay/cloud-on-k8s that referenced this pull request Dec 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Distributed Coordination/Autoscaling >non-issue Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. v7.11.0 v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants