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

[WIP] initial stab at comparing available instance types #42

Closed
wants to merge 2 commits into from

Conversation

durandom
Copy link
Member

@durandom durandom commented Sep 5, 2016

with those listed at ec2instances.info.

Right now it does not compare all attributes, but at least it finds new instance types.
We could add the content from ec2instances.info to a vcr or even better fetch it live on CI.
This could be a reference for other providers as well.

❯ be rspec spec/models/manageiq/providers/amazon/instance_types_spec.rb
** master - DB vmdb_test on

Randomized with seed 35266
"m2.xlarge" => {
:description => "M2 High Memory Extra Large"
# was: M2 High-Memory Extra Large
}
"m2.2xlarge" => {
:description => "M2 High Memory Double Extra Large"
# was: M2 High-Memory Double Extra Large
}
"m2.4xlarge" => {
:description => "M2 High Memory Quadruple Extra Large"
# was: M2 High-Memory Quadruple Extra Large
}
"cr1.8xlarge" => {
:description => "High Memory Cluster Eight Extra Large"
# was: High-Memory Cluster Eight Extra Large
}
"hi1.4xlarge" => {
:description => "HI1. High I/O Quadruple Extra Large"
# was: High I/O Quadruple Extra Large
}
"t1.micro" => {
:family => "Micro"
# was: Micro Instances
}
"m4.large" => {
:description => "M4 Large"
# was: M4 General Purpose Large
}
"m4.xlarge" => {
:description => "M4 Extra Large"
# was: M4 General Purpose Extra Large
}
"m4.2xlarge" => {
:description => "M4 Double Extra Large"
# was: M4 General Purpose Double Extra Large
}
"m4.4xlarge" => {
:description => "M4 Quadruple Extra Large"
# was: M4 General Purpose Quadruple Extra Large
}
"m4.10xlarge" => {
:description => "M4 Deca Extra Large"
# was: M4 General Purpose Ten Extra Large
}
.

@durandom durandom self-assigned this Sep 5, 2016
@miq-bot
Copy link
Member

miq-bot commented Sep 5, 2016

<github_pr_commenter_batch />Some comments on commit durandom@0b88aab

spec/models/manageiq/providers/amazon/instance_types_spec.rb

  • ⚠️ - 43 - Detected puts. Remove all debugging statements.
  • ⚠️ - 47 - Detected puts. Remove all debugging statements.
  • ⚠️ - 48 - Detected puts. Remove all debugging statements.
  • ⚠️ - 51 - Detected puts. Remove all debugging statements.
  • ⚠️ - 54 - Detected puts. Remove all debugging statements.

@chessbyte chessbyte changed the title [wip] intial stab at comparing available instance types [WIP] initial stab at comparing available instance types Oct 26, 2016
with those listed at ec2instances.info
@miq-bot
Copy link
Member

miq-bot commented Mar 8, 2017

Some comments on commits durandom/manageiq-providers-amazon@903167b~...154b35b

spec/models/manageiq/providers/amazon/instance_types_spec.rb

  • ⚠️ - 109 - Detected puts. Remove all debugging statements.
  • ⚠️ - 161 - Detected puts. Remove all debugging statements.

@miq-bot
Copy link
Member

miq-bot commented Mar 8, 2017

Checked commits durandom/manageiq-providers-amazon@903167b~...154b35b with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
2 files checked, 12 offenses detected

spec/models/manageiq/providers/amazon/instance_types_spec.rb

@durandom
Copy link
Member Author

durandom commented Mar 9, 2017

closing in favour of #170

@durandom durandom closed this Mar 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants