Disable entire ensure call

oxidecomputer · Sep 24, 2024 · 0c8838b · 0c8838b
1 parent 76359f6
commit 0c8838b
Showing 1 changed file with 12 additions and 12 deletions.
diff --git a/cio/src/configs.rs b/cio/src/configs.rs
@@ -547,18 +547,18 @@ impl UserConfig {
             }
         }
 
-        // Add the user to their GitHub teams and the org.
-        if !new_user.github.is_empty() {
-            // Add them to the org and any teams they need to be added to.
-            // We don't return an id here.
-            match github.ensure_user(db, company, &new_user, config).await {
-                Ok(id) => Ok(id),
-                Err(err) => {
-                    warn!("Failed to ensure GitHub user `{}`: {}", new_user.id, err);
-                    Err(err)
-                }
-            }?;
-        }
+        // // Add the user to their GitHub teams and the org.
+        // if !new_user.github.is_empty() {
+        //     // Add them to the org and any teams they need to be added to.
+        //     // We don't return an id here.
+        //     match github.ensure_user(db, company, &new_user, config).await {
+        //         Ok(id) => Ok(id),
+        //         Err(err) => {
+        //             warn!("Failed to ensure GitHub user `{}`: {}", new_user.id, err);
+        //             Err(err)
+        //         }
+        //     }?;
+        // }
 
         match ramp.ensure_user(db, company, &new_user, config).await {
             Ok(ramp_id) => {