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

Update aws flavors cpu count. #452

Merged
merged 1 commit into from Jun 21, 2018
Merged

Update aws flavors cpu count. #452

merged 1 commit into from Jun 21, 2018

Conversation

xward
Copy link
Contributor

@xward xward commented Jun 21, 2018

@xward
Copy link
Contributor Author

xward commented Jun 21, 2018

Actually the question is : what is the unit of core ? ecu ou vcpu ?

@geemus
Copy link
Member

geemus commented Jun 21, 2018

Good question. I don't recall for sure (and it seems strange that we would have so many that are off). The old values don't really obviously match with either ecu or vcpu either. So, I'm not sure what it used to be. I think standardizing to vcpu makes sense (especially since so many just have "variable" for ecu). So I'm fine with the change as you've done, thanks!

@geemus geemus merged commit 2993d42 into fog:master Jun 21, 2018
@romaintb romaintb deleted the fix/update_aws_flavor_core_count branch June 21, 2018 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants