Switch BackgroundJob to regular methods

Instead of using a class method for run(), a regular method is used for
this purpose. This gives the possibility to add more convenience methods
in the future, e.g. for interacting with the job object or for logging,
as implemented for scripts.

docs/plugins/development/background-tasks.md

@@ -19,9 +19,8 @@ class MyTestJob(BackgroundJob):
     class Meta:
         name = "My Test Job"
 
-    @classmethod
-    def run(cls, job, *args, **kwargs):
-        obj = job.object
+    def run(self, *args, **kwargs):
+        obj = self.job.object
         # your logic goes here
 ```
 
@@ -62,8 +61,7 @@ class MyHousekeepingJob(BackgroundJob):
     class Meta:
         name = "Housekeeping"
 
-    @classmethod
-    def run(cls, job, *args, **kwargs):
+    def run(self, *args, **kwargs):
         # your logic goes here
 ```
 ```python title="__init__.py"

netbox/core/jobs.py

@@ -17,9 +17,8 @@ class SyncDataSourceJob(BackgroundJob):
     class Meta:
         name = 'Synchronization'
 
-    @classmethod
-    def run(cls, job, *args, **kwargs):
-        datasource = DataSource.objects.get(pk=job.object_id)
+    def run(self, *args, **kwargs):
+        datasource = DataSource.objects.get(pk=self.job.object_id)
 
         try:
             datasource.sync()

netbox/extras/jobs.py

@@ -26,14 +26,12 @@ class ScriptJob(BackgroundJob):
         # where jobs other than this one are used. Therefore, it is hidden, resulting in a cleaner job table overview.
         name = ''
 
-    @staticmethod
-    def run_script(script, job, request, data, commit):
+    def run_script(self, script, request, data, commit):
         """
         Core script execution task. We capture this within a method to allow for conditionally wrapping it with the
         event_tracking context manager (which is bypassed if commit == False).
 
         Args:
-            job: The Job associated with this execution
             request: The WSGI request associated with this execution (if any)
             data: A dictionary of data to be passed to the script upon execution
             commit: Passed through to Script.run()
@@ -80,10 +78,9 @@ class ScriptJob(BackgroundJob):
         # Update the job data regardless of the execution status of the job. Successes should be reported as well as
         # failures.
         finally:
-            job.data = script.get_job_data()
+            self.job.data = script.get_job_data()
 
-    @classmethod
-    def run(cls, job, data, request=None, commit=True, **kwargs):
+    def run(self, data, request=None, commit=True, **kwargs):
         """
         Run the script.
 
@@ -93,7 +90,7 @@ class ScriptJob(BackgroundJob):
             request: The WSGI request associated with this execution (if any)
             commit: Passed through to Script.run()
         """
-        script = ScriptModel.objects.get(pk=job.object_id).python_class()
+        script = ScriptModel.objects.get(pk=self.job.object_id).python_class()
 
         # Add files to form data
         if request:
@@ -107,4 +104,4 @@ class ScriptJob(BackgroundJob):
         # Execute the script. If commit is True, wrap it with the event_tracking context manager to ensure we process
         # change logging, event rules, etc.
         with event_tracking(request) if commit else nullcontext():
-            cls.run_script(script, job, request, data, commit)
+            self.run_script(script, request, data, commit)

netbox/utilities/jobs.py

@@ -27,13 +27,19 @@ class BackgroundJob(ABC):
     class Meta:
         pass
 
+    def __init__(self, job):
+        """
+        Args:
+            job: The specific `Job` this `BackgroundJob` helper class is executing.
+        """
+        self.job = job
+
     @classproperty
     def name(cls):
         return getattr(cls.Meta, 'name', cls.__name__)
 
-    @classmethod
     @abstractmethod
-    def run(cls, *args, **kwargs):
+    def run(self, *args, **kwargs):
         """
         Run the job.
 
@@ -51,7 +57,7 @@ class BackgroundJob(ABC):
         """
         try:
             job.start()
-            cls.run(job, *args, **kwargs)
+            cls(job).run(*args, **kwargs)
             job.terminate()
 
         except Exception as e:

netbox/utilities/tests/test_jobs.py

@@ -10,8 +10,7 @@ from core.choices import JobStatusChoices
 
 
 class TestBackgroundJob(BackgroundJob):
-    @classmethod
-    def run(cls, *args, **kwargs):
+    def run(self, *args, **kwargs):
         pass
 
 
@@ -54,9 +53,8 @@ class BackgroundJobTest(BackgroundJobTestCase):
         class ErroredBackgroundJob(TestBackgroundJob):
             EXP = Exception('Test error')
 
-            @classmethod
-            def run(cls, *args, **kwargs):
-                raise cls.EXP
+            def run(self, *args, **kwargs):
+                raise self.EXP
 
         job = ErroredBackgroundJob.enqueue(immediate=True)