Keep run interval in hours

Signed-off-by: Keshav Priyadarshi <git@keshav.space>

Author: keshav-space

vulnerabilities/migrations/0093_alter_pipelineschedule_run_interval.py

@@ -0,0 +1,71 @@
+# Generated by Django 4.2.22 on 2025-06-24 16:33
+
+import django.core.validators
+from django.db import migrations, models
+from datetime import datetime
+from datetime import timezone
+
+from aboutcode.pipeline import LoopProgress
+
+class Migration(migrations.Migration):
+    def update_old_interval(apps, schema_editor):
+        Pipeline = apps.get_model("vulnerabilities", "PipelineSchedule")
+        pipeline_schedule = Pipeline.objects.all()
+
+        log(f"\nUpdating interval for {pipeline_schedule.count():,d} pipeline")
+        progress = LoopProgress(
+            total_iterations=pipeline_schedule.count(),
+            progress_step=10,
+            logger=log,
+        )
+        for pipeline in progress.iter(pipeline_schedule.iterator()):
+            pipeline.run_interval = pipeline.run_interval * 24
+            pipeline.save()
+    
+    def reverse_update_old_interval(apps, schema_editor):
+        Pipeline = apps.get_model("vulnerabilities", "PipelineSchedule")
+        pipeline_schedule = Pipeline.objects.all()
+
+        log(f"\nReverse interval for {pipeline_schedule.count():,d} pipeline")
+        progress = LoopProgress(
+            total_iterations=pipeline_schedule.count(),
+            progress_step=10,
+            logger=log,
+        )
+        for pipeline in progress.iter(pipeline_schedule.iterator()):
+            pipeline.run_interval = pipeline.run_interval / 24
+            pipeline.save()
+
+    dependencies = [
+        ("vulnerabilities", "0092_pipelineschedule_pipelinerun"),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name="pipelineschedule",
+            name="run_interval",
+            field=models.PositiveSmallIntegerField(
+                default=24,
+                help_text="Number of hours to wait between run of this pipeline.",
+                validators=[
+                    django.core.validators.MinValueValidator(
+                        1, message="Interval must be at least 1 hour."
+                    ),
+                    django.core.validators.MaxValueValidator(
+                        8760, message="Interval must be at most 8760 hours."
+                    ),
+                ],
+            ),
+        ),
+        migrations.RunPython(
+            code=update_old_interval,
+            reverse_code=reverse_update_old_interval,
+        ),
+    ]
+
+
+def log(message):
+    now_local = datetime.now(timezone.utc).astimezone()
+    timestamp = now_local.strftime("%Y-%m-%d %H:%M:%S.%f")[:-3]
+    message = f"{timestamp} {message}"
+    print(message)

vulnerabilities/models.py

@@ -2141,11 +2141,11 @@ class PipelineSchedule(models.Model):
 
     run_interval = models.PositiveSmallIntegerField(
         validators=[
-            MinValueValidator(1, message="Interval must be at least 1 day."),
-            MaxValueValidator(365, message="Interval must be at most 365 days."),
+            MinValueValidator(1, message="Interval must be at least 1 hour."),
+            MaxValueValidator(8760, message="Interval must be at most 8760 hours."),
         ],
-        default=1,
-        help_text=("Number of days to wait between run of this pipeline."),
+        default=24,
+        help_text=("Number of hours to wait between run of this pipeline."),
     )
 
     schedule_work_id = models.CharField(

vulnerabilities/schedules.py

@@ -28,7 +28,7 @@ def schedule_execution(pipeline_schedule, execute_now=False):
     if not execute_now:
         first_execution = pipeline_schedule.next_run_date
 
-    interval_in_seconds = pipeline_schedule.run_interval * 24 * 60 * 60
+    interval_in_seconds = pipeline_schedule.run_interval * 60 * 60
 
     job = scheduler.schedule(
         scheduled_time=first_execution,

vulnerabilities/templates/pipeline_dashboard.html

@@ -72,7 +72,9 @@ <h1>Pipeline Dashboard</h1>
                                 <div class="columns px-1 is-mobile is-vcentered">
                                     <div class="column is-one-quarter">{{ schedule.pipeline_id }}</div>
                                     <div class="column is-one-eighth has-text-grey">{{ schedule.is_active|yesno:"Yes,No" }}</div>
-                                    <div class="column is-one-eighth has-text-grey">{{ schedule.run_interval }}day</div>
+                                    <div class="column is-one-eighth has-text-grey">
+                                        {{ schedule.run_interval }} hour{{ schedule.run_interval|pluralize }}
+                                    </div>
                                     <div class="column is-one-eighth">
                                         <span class="is-flex is-align-items-center">
                                             {% include "includes/job_status.html" with status=schedule.status %}