Fix the tutorial's user events

endpoints/index.py

@@ -114,13 +114,11 @@ def create_user():
       # Mark that the user was logged in.
       event = userevents.get_event(username)
       event.publish_event_data('docker-cli', {'action': 'login'})
-
       return success
     else:
       # Mark that the login failed.
       event = userevents.get_event(username)
       event.publish_event_data('docker-cli', {'action': 'loginfailure'})
-
       abort(400, error_message, issue='login-failure')
 
   elif not features.USER_CREATION:
@@ -231,6 +229,16 @@ def create_repository(namespace, repository):
     repo = model.create_repository(namespace, repository,
                                    get_authenticated_user())
 
+    if get_authenticated_user():
+      user_event_data = {
+        'action': 'push_start',
+        'repository': repository,
+        'namespace': namespace
+      }
+
+      event = userevents.get_event(get_authenticated_user().username)
+      event.publish_event_data('docker-cli', user_event_data)
+
   return make_response('Created', 201)
 
 
@@ -248,20 +256,6 @@ def update_images(namespace, repository):
       # Make sure the repo actually exists.
       abort(404, message='Unknown repository', issue='unknown-repo')
 
-    if get_authenticated_user():
-      logger.debug('Publishing push event')
-      username = get_authenticated_user().username
-
-      # Mark that the user has pushed the repo.
-      user_data = {
-        'action': 'pushed_repo',
-        'repository': repository,
-        'namespace': namespace
-      }
-
-      event = userevents.get_event(username)
-      event.publish_event_data('docker-cli', user_data)
-
     logger.debug('GCing repository')
     model.garbage_collect_repository(namespace, repository)
 
@@ -272,6 +266,7 @@ def update_images(namespace, repository):
     event_data = {
       'updated_tags': updated_tags,
     }
+
     track_and_log('push_repo', repo)
     spawn_notification(repo, 'repo_push', event_data)
     return make_response('Updated', 204)