Add more deployment telemetry

Author: Samze

app/actions/deployment_create.rb

@@ -25,7 +25,7 @@ def create(app:, user_audit_info:, message:)
 
           if target_state.rollback_target_revision
             revision = RevisionResolver.rollback_app_revision(app, target_state.rollback_target_revision, user_audit_info)
-            log_rollback_event(app.guid, user_audit_info.user_guid, target_state.rollback_target_revision.guid, message.strategy)
+            log_rollback_event(app.guid, user_audit_info.user_guid, target_state.rollback_target_revision.guid, message.strategy, message.max_in_flight, message.canary_steps)
           else
             revision = RevisionResolver.update_app_revision(app, user_audit_info)
           end
@@ -251,15 +251,19 @@ def starting_process_instances(deployment, desired_instances)
         [deployment.max_in_flight, starting_process_count].min
       end
 
-      def log_rollback_event(app_guid, user_id, revision_id, strategy)
+      def log_rollback_event(app_guid, user_id, revision_id, strategy, max_in_flight, canary_steps)
         TelemetryLogger.v3_emit(
           'rolled-back-app',
           {
             'app-id' => app_guid,
             'user-id' => user_id,
             'revision-id' => revision_id
           },
-          { 'strategy' => strategy }
+          {
+            'strategy' => strategy,
+            'max_in_flight' => max_in_flight,
+            'canary_steps' => canary_steps
+          }
         )
       end
     end

app/controllers/v3/deployments_controller.rb

@@ -58,7 +58,11 @@ def create
           'app-id' => app.guid,
           'user-id' => current_user.guid
         },
-        { 'strategy' => deployment.strategy }
+        {
+          'strategy' => deployment.strategy,
+          'max_in_flight' => deployment.max_in_flight,
+          'canary_steps' => deployment.canary_steps
+        }
       )
     rescue DeploymentCreate::Error => e
       unprocessable!(e.message)