InstanceTypes should be present in only one boto module #543

Open
rubic opened this Issue Feb 2, 2012 · 1 comment

Comments

Projects
None yet
2 participants
@rubic
Contributor

rubic commented Feb 2, 2012

InstanceTypes is defined more than once in boto:

  • boto/manager/server.py
  • boto/ec2/buyreservation.py
  • boto/mashups/order.py

There should be one authoritative place for InstanceTypes. Also, cc2.8xlarge needs to be added to boto:

InstanceTypes = [
    't1.micro',
    'm1.small',
    'm2.xlarge',
    'm2.2xlarge',
    'm2.4xlarge',
    'c1.medium',
    'c1.xlarge',
    'cc1.4xlarge',
    'cc2.8xlarge',
    ]
@ryansb

This comment has been minimized.

Show comment Hide comment
@ryansb

ryansb Mar 25, 2012

Contributor

Fixed this in pull request #661

Contributor

ryansb commented Mar 25, 2012

Fixed this in pull request #661

dyzsasd pushed a commit to dailymotion/boto that referenced this issue May 9, 2017

Merge pull request #543 from glasnt/topic/syntax
Correct indentation on code example
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment