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

Ensure root device is listed in block device mappings #1146

Merged
merged 1 commit into from
Sep 15, 2017

Conversation

nuwang
Copy link
Contributor

@nuwang nuwang commented Sep 15, 2017

No description provided.

@coveralls
Copy link

coveralls commented Sep 15, 2017

Coverage Status

Coverage remained the same at 93.58% when pulling c116c57 on gvlproject:root_device_mapping into 44bd6e8 on spulec:master.

@JackDanger
Copy link
Collaborator

I'm not very familiar with this API but the Example #1 on this page shows a root device of /dev/sda and then a deviceName of /dev/sda1: https://docs.aws.amazon.com/AWSEC2/latest/APIReference/API_DescribeImages.html

@JackDanger
Copy link
Collaborator

We can merge this PR as-is because Example #2 on that page matches what your changes suggest. I just wonder if either the AWS docs are wrong (which happens) or if there's something else going on here.

@nuwang
Copy link
Contributor Author

nuwang commented Sep 15, 2017

@JackDanger I suspect that the example may be wrong. The docs here state the following:
"Each AMI has a block device mapping that specifies the block devices to attach to an instance when it is launched from the AMI. An AMI that Amazon provides includes a root device only. To add more block devices to an AMI, you must create your own AMI."

The docs here say that "Every time you launch an instance from an AMI, a root storage device is created for that instance. The root storage device contains all the information necessary to boot the instance"

Putting the two things together, I think it's safe to assume that the root device always appears in block device mappings for the AMI, although I couldn't find an unambiguous statement stating this explicitly. It's also consistent with how things work through the console and APIs.

@JackDanger
Copy link
Collaborator

@nuwang Thanks for figuring this out! Once again Moto developers eclipse the AWS docs in accuracy 👍

@JackDanger JackDanger merged commit eab7056 into getmoto:master Sep 15, 2017
nuwang added a commit to CloudVE/cloudbridge that referenced this pull request Sep 17, 2017
The following PRs fix various moto issues:
- getmoto/moto#1140
- getmoto/moto#1146
- getmoto/moto#1148
- getmoto/moto#1149
- getmoto/moto#1150
- getmoto/moto#1152

These has been merged into moto 1.1.10
nuwang added a commit to CloudVE/cloudbridge that referenced this pull request Sep 17, 2017
The following PRs fix various moto issues:
- getmoto/moto#1140
- getmoto/moto#1146
- getmoto/moto#1148
- getmoto/moto#1149
- getmoto/moto#1150
- getmoto/moto#1152

These has been merged into moto 1.1.10
nuwang added a commit to CloudVE/cloudbridge that referenced this pull request Sep 17, 2017
The following PRs fix various moto issues:
- getmoto/moto#1140
- getmoto/moto#1146
- getmoto/moto#1148
- getmoto/moto#1149
- getmoto/moto#1150
- getmoto/moto#1152

These has been merged into moto 1.1.10
nuwang added a commit to CloudVE/cloudbridge that referenced this pull request Sep 17, 2017
The following PRs fix various moto issues:
- getmoto/moto#1140
- getmoto/moto#1146
- getmoto/moto#1148
- getmoto/moto#1149
- getmoto/moto#1150
- getmoto/moto#1152

These has been merged into moto 1.1.10
nuwang added a commit to CloudVE/cloudbridge that referenced this pull request Sep 17, 2017
The following PRs fix various moto issues:
- getmoto/moto#1140
- getmoto/moto#1146
- getmoto/moto#1148
- getmoto/moto#1149
- getmoto/moto#1150
- getmoto/moto#1152

These have been merged into Moto 1.1.10
nuwang added a commit to CloudVE/cloudbridge that referenced this pull request Sep 17, 2017
The following PRs fix various moto issues:
- getmoto/moto#1140
- getmoto/moto#1146
- getmoto/moto#1148
- getmoto/moto#1149
- getmoto/moto#1150
- getmoto/moto#1152

These have been merged into Moto 1.1.10
nuwang added a commit to CloudVE/cloudbridge that referenced this pull request Nov 25, 2017
The following PRs fix various moto issues:
- getmoto/moto#1140
- getmoto/moto#1146
- getmoto/moto#1148
- getmoto/moto#1149
- getmoto/moto#1150
- getmoto/moto#1152

These have been merged into Moto 1.1.10
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.

3 participants