Skip to content

Perform lease synchronization in transaction

Cal Pratt requested to merge cpratt34/bot-session-transactions into master

In this MR, the Job class is completely killed in favor of doing everything in the datastore/scheduler.

One goal of these changes is to not heavily change the existing behavior, but to ensure that almost all changes made are being performed in a single transaction. I compared these findings to the Remote Workers API spec and validated if the behavior makes sense.

Question: In the MR I removed some function which used to produce metrics. Notably the update_job and update_lease methods are removed. We're the metrics provided by these generic update methods of any value, or should I look to add them back into various functions?

Edited by Cal Pratt

Merge request reports