use ssd image types on Amazon EC2
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
juju-core |
Fix Released
|
High
|
Ian Booth |
Bug Description
Amazon uses a differently named image type for the new SSD machines. Instead of deploying an "ebs" root volume, or an "instance" based volume, you use an "ssd" volume.
You can see them in the streams data here:
http://
Such as:
"apne1hs": {
"virt": "hvm",
"crsn": "ap-northeast-1",
"id": "ami-35e2b434"
},
as contrasted with:
"apne1he": {
"virt": "hvm",
"crsn": "ap-northeast-1",
"id": "ami-3d79003c"
},
or
"apne1hi": {
"virt": "hvm",
"crsn": "ap-northeast-1",
"id": "ami-b9067fb8"
},
We currently have 2 places in our code that seem to filter for exact matches to "ebs", which we need to figure out what we can do. AIUI Amazon would *like* us to just switch over to "ssd" completely (that was my understanding from Ben Howard). However, I'd rather we move to a "use ssd if we can, else fall back to ebs".
You can grep the code for "ebsStorage" where we save that constant, but it is in StartInstance (passing it as the Storage parameter to findInstanceSpec), and in filterImages, where we again do a:
// For now, we only want images with "ebs" storage.
if image.Storage == ebsStorage {
}
I'm assuming the loop is easy enough to change to:
if image.Storage == ebsStorage || image.Storage == ssdStorage
but I don't quite know how to fix the findInstanceSpec since it only lets us put one entry in the InstanceConstraint document.
I also don't quite understand why InstanceConstraint is getting the Storage parameter, and passing it on to imagemetadata.
I'm guessing the value passed in is just ignored everywhere, but there are a lot of layers to dig through to get there.
Changed in juju-core: | |
milestone: | none → 1.21-alpha2 |
Changed in juju-core: | |
assignee: | nobody → Ian Booth (wallyworld) |
status: | Triaged → In Progress |
Changed in juju-core: | |
status: | In Progress → Fix Committed |
Changed in juju-core: | |
status: | Fix Committed → Fix Released |
Is this a duplicate of bug 1333117