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

block device examples updated #136

Merged
merged 3 commits into from
Jun 9, 2015
Merged

Conversation

gmiranda23
Copy link

The block device examples are incorrect. Adjusted them to reflect a useable config.

@tyler-ball
Copy link
Contributor

Hey @gmiranda23 - I actually get errors trying to use this PR. What is the error you're seeing?

Although, while troubleshooting I did come across an unexpected exception trying to call logger from the InstanceGenerator class

@gmiranda23
Copy link
Author

It's not so much an error as misleading info. I don't typically come across EBS volumes with block device trailing numbers for anything other than root volumes. And even then, most newer instance types have problems with them. I wasn't quite sure why, but your question prompted me to look it up.

I got some clarification on this from the EBS Block Device Mapping Concepts doc.

"Trailing numbers on device names can sometimes be used, but may cause conflicts. [...] Hardware virtual machine (HVM) AMIs don't support the use of trailing numbers on device names (xvd[a-p][1-15])."

So perhaps to make that usage point, the example should have one definition with a trailing device number and one without?

@tyler-ball
Copy link
Contributor

If it is suggested to use non-numbered device names, then I'm all for updating the documentation. I'm not very familiar with the concept of disc device naming.

I would just like to make sure the example does work. Can you update the AMIs in the example at the bottom of the README so specifying /dev/sda instead of /dev/sda1 will create it correctly? Perhaps just use Ubuntu 15.04 HVM (ami-cf09e8a4) and Centos 7 HVM (ami-96a818fe). Will you try those AMIs and test the full example to make sure it works?

If you update the root device mapping, TK should issue an info level log message stating this.

@gmiranda23
Copy link
Author

After some research, I put together a full working .kitchen.yml example using non-standard settings (us-west-2), with updated fields where appropriate, a mix of using platforms defined in amis.json, and custom platforms with different settings. Makes usage described in #153 clear.

region: us-east-1
availability_zone: b
region: us-west-2
availability_zone: us-west-2b
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I know this was broken earlier, but I would like to encourage people to use the single-letter form of this config to DRY this yaml up a bit. Would you try b with the latest version of kitchen-ec2 (0.9.5+)?

@tyler-ball
Copy link
Contributor

Other than my last minor comment, this is looking awesome. Thanks @gmiranda23! Once that is addressed I'll merge this.

@gmiranda23
Copy link
Author

I updated it to the long format because single letter was busted. I agree that's a better form. Tested on kitchen-ec2 0.9.5 and it works again. LGTM!

tyler-ball added a commit that referenced this pull request Jun 9, 2015
block device examples updated
@tyler-ball tyler-ball merged commit bd30050 into test-kitchen:master Jun 9, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants