fix(Site): Serialize enable database access calls

- Since job is fast, we need to check the `is_database_access_enabled`
  field's status in addition to pending and running jobs.
- This will only work if job callback also puts a lock on site doc
  before updating the field

press/press/doctype/agent_job/agent_job.py

@@ -602,6 +602,8 @@ def get_pair_jobs() -> tuple[str]:
 		"Remove Site from Upstream",
 		"Rename Site",
 		"Rename Site on Upstream",
+		"Add User to ProxySQL",
+		"Remove User from ProxySQL",
 	)
 
 

press/press/doctype/site/site.py

@@ -2160,9 +2160,7 @@ def _create_initial_site_plan_change(self, plan):
 			}
 		).insert(ignore_permissions=True)
 
-	@dashboard_whitelist()
-	@site_action(["Active"])
-	def enable_database_access(self, mode="read_only"):
+	def check_db_access_enabling(self):
 		if frappe.db.get_value(
 			"Agent Job",
 			filters={
@@ -2176,8 +2174,19 @@ def enable_database_access(self, mode="read_only"):
 				"Database Access is already being enabled on this site. Please check after a while."
 			)
 
+	def check_db_access_enabled_already(self):
+		if frappe.db.get_value(
+			self.doctype, self.name, "is_database_access_enabled", for_update=True
+		):
+			frappe.throw("Database Access already enabled. Reload the page and try.")
+
+	@dashboard_whitelist()
+	@site_action(["Active"])
+	def enable_database_access(self, mode="read_only"):
 		if not frappe.db.get_value("Site Plan", self.plan, "database_access"):
 			frappe.throw(f"Database Access is not available on {self.plan} plan")
+		self.check_db_access_enabling()
+		self.check_db_access_enabled_already()
 		log_site_activity(self.name, "Enable Database Access")
 
 		server_agent = Agent(self.server)