To make names clear, we should probably change some names. In find_keydir_entry, iterating should be called use_snapshot, get_from_snapshot or similar. This is not from your PR. I wrote that one. My bad :). A similar name should be given to iterating_status.