From 1e17b222abbc2e5898b805fa3805bfd5fcc86b52 Mon Sep 17 00:00:00 2001 From: Seth Schoen Date: Wed, 18 Jul 2012 19:38:00 -0700 Subject: [PATCH] document priority inversion bug --- server-ca/daemon.py | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/server-ca/daemon.py b/server-ca/daemon.py index e5a19116c..4b2b382f0 100644 --- a/server-ca/daemon.py +++ b/server-ca/daemon.py @@ -229,6 +229,12 @@ def issue(session): # session nonetheless died for some reason unrelated to failing # challenges before the cert could be issued. Normally, this # should never happen. + # + # TODO: This can definitely happen when there are at least as many + # sessions stuck in testchallenge state as there are daemon processes + # to handle them, because all sessions in testchallenge have absolute + # priority over all sessions in issue, from each individual daemon's + # point of view! This is a bug. if debug: print "removing expired (issue-state!?) session", session r.lrem("pending-requests", session) return