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

fix a bug introduced by aws cli change of CreateSnapshot Response #247 #70

Closed
wants to merge 1 commit into from

Conversation

mingliangfeng
Copy link
Contributor

See boto/botocore#247.

Tags key was removed from CreateSnapshot response, which means one column is removed from the result of "aws ec2 create-snapshot". To get the right snapshot id, need to use "cut -f 4"; "cut -f 5" is getting the timestamp, not snapshot id.

@colinbjohnson
Copy link
Collaborator

@mingliangfeng - thank you for the PR - looks like it can't be merged, but I've taken your advice into consideration - I've modified the code to use aws ec2 create-snapshot --region $region --description $ec2_snapshot_description --volume-id $ebs_selected --output text --query SnapshotId - where --query will allow me to select the return value of a given command - in this case, I only get SnapshotId back, so I don't need to use cut at all.

@mingliangfeng
Copy link
Contributor Author

Good to see it is fixed.

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.

2 participants