Change manifest delete to mark tag as dead and log

endpoints/v1/tag.py

@@ -12,6 +12,7 @@ from auth.permissions import (ReadRepositoryPermission,
 from data import model
 from endpoints.decorators import anon_protect
 from endpoints.v1 import v1_bp
+from endpoints.trackhelper import track_and_log
 
 
 logger = logging.getLogger(__name__)
@@ -86,6 +87,8 @@ def delete_tag(namespace, repository, tag):
 
   if permission.can():
     model.tag.delete_tag(namespace, repository, tag)
+    track_and_log('delete_tag', model.repository.get_repository(namespace, repository),
+                  tag=tag)
     return make_response('Deleted', 200)
 
   abort(403)

endpoints/v2/manifest.py

@@ -388,9 +388,15 @@ def delete_manifest_by_digest(namespace, repo_name, manifest_ref):
     # Without a tag name to reference, we can't make an attempt to generate the manifest
     raise ManifestUnknown()
 
-  manifest.delete_instance()
+  # Mark the tag as no longer alive.
+  try:
+    model.tag.delete_tag(namespace, repo_name, manifest.tag.name)
+  except model.DataModelException:
+    # Tag is not alive.
+    raise ManifestUnknown()
 
-  # TODO(jschorr): Log this as a new log type.
+  track_and_log('delete_tag', manifest.tag.repository,
+                tag=manifest.tag.name, digest=manifest_ref)
 
   return make_response('', 202)