summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMattia Rizzolo <mattia@debian.org>2015-12-14 18:34:01 +0000
committerHolger Levsen <holger@layer-acht.org>2015-12-14 19:36:29 +0100
commit5b71180cb97a9a79b40b3fd025dfe17a48712848 (patch)
treed49282093ec51204ca25da600644ca7cd78e961c
parent7bb713f14db4e12587241c852fc8910b3affa0cf (diff)
downloadjenkins.debian.net-5b71180cb97a9a79b40b3fd025dfe17a48712848.tar.xz
reproducible: don't save the reschedule reason in the DB
That was actually a bad idea, given that we don't check on input execute the with the given string right away. Also, we're not really interested in this, so once convenient we can also remove the column from the DB.
-rw-r--r--TODO1
-rwxr-xr-xbin/reproducible_remote_scheduler.py3
2 files changed, 2 insertions, 2 deletions
diff --git a/TODO b/TODO
index 883933cc..3e225c02 100644
--- a/TODO
+++ b/TODO
@@ -136,6 +136,7 @@ This is about Debian, below are more todo entries for other projects…
* on SIGTERM, also ssh to remote host and cleanup there! (via ssh &)
* higher prio:
+** remove the rescheduling reason from the DB, that's really not needed
** scheduler should automatically schedule 404 packages
** explain status in plain english on each coreboot/openwrt/netbsd/freebsd page, also on the Debian dashboard plus add an "executive summary about reproducible builds in the free software world"
*** get the content for "<h2>status of $1</h2>" from notes.git/friends.yaml or such
diff --git a/bin/reproducible_remote_scheduler.py b/bin/reproducible_remote_scheduler.py
index 331f7b36..4afe29db 100755
--- a/bin/reproducible_remote_scheduler.py
+++ b/bin/reproducible_remote_scheduler.py
@@ -233,7 +233,6 @@ if amount + len(ids) > 200 and not local:
to_schedule = []
save_schedule = []
artifacts_value = 1 if artifacts else 0
-reason = reason if reason else None
if notify_on_start:
do_notify = 2
elif notify or artifacts:
@@ -242,7 +241,7 @@ else:
do_notify = 0
for id in ids:
to_schedule.append((id, date, artifacts_value, str(do_notify), requester,
- reason))
+ None))
save_schedule.append((id, requester, epoch))
log.debug('Packages about to be scheduled: ' + str(to_schedule))