Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue number:
#97
Description of changes:
With this change, symlinks to the underlying EBS volumes will be created under
/dev/by-ebs-id
instead of the root of/dev
. This will help prevent collisions with actual devices created by the kernel (as described in this amazonlinux/amazon-ec2-utils#37).As part of this change,
/dev/
will be removed from the device name if it is found after querying the device identifier, so that only the device id is returned to udev.Testing done:
I ran my change in an EC2 instance, and confirmed that the devices are listed under
/dev/by-ebs-id
when the mapping is added:I confirmed that in the workflow that prepends the device ID with
/dev/
, this prefix is removed and the symlink is created under/dev/by-ebs-id/
Terms of contribution:
By submitting this pull request, I agree that this contribution is dual-licensed under the terms of both the Apache License, version 2.0, and the MIT license.