[TRI-3867] Updating lots of environment variables at once fails with a transaction timeout

[matt-aitken]

When we bulk upsert environment variables we wrap the entire thing in a database transaction. If there are a lot of env vars (e.g. more than 100) then the transaction can timeout.

You can see in the current code that the transaction is wrapped around the for loop:

trigger.dev/apps/webapp/app/v3/environmentVariables/environmentVariablesRepository.server.ts

Line 138 in feb4fcd

const result = await $transaction(this.prismaClient, async (tx) => {

Instead we should move that transaction inside the for loop. We still need the transaction so we always create all the resources for each env var and don't leave any of them dangling.

_TRI-3867

[yadavshubham01]

hey @matt-aitken working in this issue

[yadavshubham01]

@matt-aitken Checkout this #1413