Arvados

Bug #21417

Updated by Tom Clegg 3 months ago

This part of source:sdk/python/arvados/commands/keepdocker.py should go away so it doesn't crash on new image tarball formats: 

 <pre> 
         json_file = image_tar.extractfile(image_tar.getmember(json_filename)) 
         image_metadata = json.loads(json_file.read().decode('utf-8')) 
         json_file.close() 
         image_tar.close() 
         link_base = {'head_uuid': coll_uuid, 'properties': {}} 
         if 'created' in image_metadata: 
             link_base['properties']['image_timestamp'] = image_metadata['created'] 
 </pre> 

 See #21408 for example. 

 (Tom & Peter discussed offline, came to the conclusion that saving the image timestamp is not important enough to justify maintaining the code.) 

Back