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

Cherry-pick #6130 to 6.2: Change kubernetes.node.cpu.allocatable.cores to float #6709

Merged
merged 1 commit into from
Apr 3, 2018

Conversation

ruflin
Copy link
Member

@ruflin ruflin commented Mar 30, 2018

Cherry-pick of PR #6130 to 6.2 branch. Original message:

Closes #6123

@jsoriano
Copy link
Member

jsoriano commented Apr 3, 2018

Could we have used scaled_float here?

@ruflin
Copy link
Member Author

ruflin commented Apr 3, 2018

@jsoriano Probably yes. I'm hesitant to change it in the backport but then would rather merge this one, open a second PR against master and backport this too.

I wonder if we should treat float to scaled_float as a breaking change. In general I would say no, so we go have float in 6.2 and do the scaled_float improvement in 6.3 to save space.

@jsoriano
Copy link
Member

jsoriano commented Apr 3, 2018

Ok, merging this then

@jsoriano jsoriano merged commit bf5e16f into elastic:6.2 Apr 3, 2018
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants