Skip to content
This repository was archived by the owner on Apr 25, 2025. It is now read-only.

Commit d5e7a6f

Browse files
authored
Merge pull request #9 from nfvs/fix_dryrun_parameter
Fix -dryrun parameter (it was being ignored before)
2 parents db80f1d + 44678f5 commit d5e7a6f

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

main.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -153,12 +153,12 @@ def delete_images(ecr_client, deletesha,deletetag, id, name):
153153
if __name__ == '__main__':
154154
request = {"None": "None"}
155155
parser = argparse.ArgumentParser(description='Deletes stale ECR images')
156-
parser.add_argument('-dryrun', help='Prints the repository to be deleted without deleting them', default=True, action='store', dest='dryrun')
156+
parser.add_argument('-dryrun', help='Prints the repository to be deleted without deleting them', default='true', action='store', dest='dryrun')
157157
parser.add_argument('-imagestokeep', help='Number of image tags to keep', default=100, action='store', dest='imagestokeep')
158158
parser.add_argument('-region', help='ECR/ECS region', default=None, action='store', dest='region')
159159

160160
args = parser.parse_args()
161161
REGION = args.region
162-
DRYRUN = args.dryrun
162+
DRYRUN = args.dryrun.lower() != 'false'
163163
IMAGES_TO_KEEP = int(args.imagestokeep)
164164
handler(request, None)

0 commit comments

Comments
 (0)