]> git.ipfire.org Git - thirdparty/bind9.git/commitdiff
Wait for "sending notifies" for step3.zsk-prepub
authorMatthijs Mekking <matthijs@isc.org>
Wed, 31 Dec 2025 10:40:42 +0000 (11:40 +0100)
committerMatthijs Mekking (GitLab job 6682718) <matthijs@isc.org>
Fri, 2 Jan 2026 08:13:31 +0000 (08:13 +0000)
Commit c17ac426082b2eca802dd1b2e1bb9b4b4b291199 changed some tests to
wait for "zone_needdump" messages instead of "sending notifies", because
notifies are rate limited and "zone_needdump" happen on every change.

However, inspecting the logs, the "zone_needdump" changes happen more
than once (likely because the re-signing is done in batches):

    received control channel command 'sign step3.zsk-prepub.manual'
    zone_journal: zone step3.zsk-prepub.manual/IN (signed): enter
    zone_needdump: zone step3.zsk-prepub.manual/IN (signed): enter
    zone_journal: zone step3.zsk-prepub.manual/IN (signed): enter
    zone_needdump: zone step3.zsk-prepub.manual/IN (signed): enter
    zone_journal: zone step3.zsk-prepub.manual/IN (signed): enter
    zone_needdump: zone step3.zsk-prepub.manual/IN (signed): enter
    zone step3.zsk-prepub.manual/IN (signed): sending notifies

This means we are running the rollover step checks too fast in some
test runs.

Revert the wait for log change for the rollover-zsk-prepub test.

(cherry picked from commit 22c02a4df9fefeb9626807e47da3892143aebfc0)

bin/tests/system/rollover-zsk-prepub/tests_rollover_zsk_prepublication.py

index 7d3be511b313ac6e1a42a7ef3bb664c33709fafd..31e0b99e43e6cf56481755ab42494ac8e7a52398 100644 (file)
@@ -254,7 +254,7 @@ def test_zsk_prepub_step3(tld, alg, size, ns3):
     # Force full resign and check all signatures have been replaced.
     with ns3.watch_log_from_here() as watcher:
         ns3.rndc(f"sign {zone}")
-        watcher.wait_for_line(f"zone_needdump: zone {zone}/IN (signed): enter")
+        watcher.wait_for_line(f"zone {zone}/IN (signed): sending notifies")
 
     step["smooth"] = False
     step["nextev"] = Iret(CONFIG, smooth=False)