1. Removed DryRun from delete_snapshot

Dave Osment · Dave Osment · commit 8f38cda6d1e4 · 2017-01-27T13:14:22.000Z
2. Excluded all source SnapShots not in yet in a 'completed' state
diff --git a/ebs-snapshot-offsiter.py b/ebs-snapshot-offsiter.py
@@ -14,6 +14,7 @@ def lambda_handler(event, context):
 	snapshots = ec.describe_snapshots(
 		Filters=[
 			{ 'Name': 'tag-key', 'Values': ['DestinationRegion'] },
+            { 'Name': 'status', 'Values': ['completed'] },
 		]
 	)
 
@@ -74,7 +75,6 @@ def lambda_handler(event, context):
 			)
 
 			ec.delete_snapshot(
-				DryRun=True,
 				SnapshotId=snapshot['SnapshotId']
 			)
 

Original file line number	Diff line number	Diff line change
`@@ -14,6 +14,7 @@ def lambda_handler(event, context):`
`14`	`14`	`snapshots = ec.describe_snapshots(`
`15`	`15`	`Filters=[`
`16`	`16`	`{ 'Name': 'tag-key', 'Values': ['DestinationRegion'] },`
	`17`	`+ { 'Name': 'status', 'Values': ['completed'] },`
`17`	`18`	`]`
`18`	`19`	`)`
`19`	`20`
`@@ -74,7 +75,6 @@ def lambda_handler(event, context):`
`74`	`75`	`)`
`75`	`76`
`76`	`77`	`ec.delete_snapshot(`
`77`		`- DryRun=True,`
`78`	`78`	`SnapshotId=snapshot['SnapshotId']`
`79`	`79`	`)`
`80`	`80`