sayanchowdhury commented on the pull-request: `Fix the clean ami script to remove the amis based on date` that you are following: ``
Looks good to me, one minor change may required: we might want to make NIGHTLY lowercase at https://pagure.io/fork/sayanchowdhury/releng/blob/0ce51d0036c4b779e71efdc4fd... , since result of fedfind is somewhat like this for rawhide and F30 nightly:
compose_info = fedfind.release.get_release(cid='Fedora-Rawhide-20190213.n.0') print compose_info.type
nightly
Yeah, `NIGHTLY` is the variable. The value of it is `nightly`.
Also, this won't delete any of Fedora updates and updates testing AMIs, since for both cases compose_info.label starts with Update-*
Yes, This is to prevent from deleting the 2WA Host released AMIs. I could not find a way to distinguish from the other AMIs. ``
To reply, visit the link below or just reply to this email https://pagure.io/releng/pull-request/8146