Merge pull request #9650 from Budibase/fix/automation-perf-improvement

Automation environment variable fix
This commit is contained in:
Michael Drury 2023-02-10 15:39:33 +00:00 committed by GitHub
commit ccbd7518f9
1 changed files with 13 additions and 13 deletions

View File

@ -464,13 +464,17 @@ export function execute(job: Job, callback: WorkerCallback) {
throw new Error("Unable to execute, event doesn't contain app ID.") throw new Error("Unable to execute, event doesn't contain app ID.")
} }
return context.doInAppContext(appId, async () => { return context.doInAppContext(appId, async () => {
const automationOrchestrator = new Orchestrator(job) const envVars = await sdkUtils.getEnvironmentVariables()
try { // put into automation thread for whole context
const response = await automationOrchestrator.execute() await context.doInEnvironmentContext(envVars, async () => {
callback(null, response) const automationOrchestrator = new Orchestrator(job)
} catch (err) { try {
callback(err) const response = await automationOrchestrator.execute()
} callback(null, response)
} catch (err) {
callback(err)
}
})
}) })
} }
@ -480,11 +484,7 @@ export const removeStalled = async (job: Job) => {
throw new Error("Unable to execute, event doesn't contain app ID.") throw new Error("Unable to execute, event doesn't contain app ID.")
} }
await context.doInAppContext(appId, async () => { await context.doInAppContext(appId, async () => {
const envVars = await sdkUtils.getEnvironmentVariables() const automationOrchestrator = new Orchestrator(job)
// put into automation thread for whole context await automationOrchestrator.stopCron("stalled")
await context.doInEnvironmentContext(envVars, async () => {
const automationOrchestrator = new Orchestrator(job)
await automationOrchestrator.stopCron("stalled")
})
}) })
} }