it doesn't need to be a transaction anymore

Author: david-crespo

nexus/db-queries/src/db/datastore/scim_provider_store.rs

@@ -1464,16 +1464,7 @@ impl<'a> ProviderStore for CrdbScimProviderStore<'a> {
         let err: OptionalError<ProviderStoreError> = OptionalError::new();
 
         let users = self
-            .datastore
-            .transaction_retry_wrapper("scim_list_users")
-            .transaction(&conn, |conn| {
-                let err = err.clone();
-                let filter = filter.clone();
-
-                async move {
-                    self.list_users_with_groups_in_txn(&conn, err, filter).await
-                }
-            })
+            .list_users_with_groups_in_txn(&conn, err.clone(), filter)
             .await
             .map_err(|e| {
                 if let Some(e) = err.take() {