Follow-up from "Add broken master label nudger automation"
The following discussion from !1839 (merged) should be addressed:
-
@ddieulivol started a discussion: (+4 comments) the 5min delay hasn't been respected at all either: it triggered as soon as I closed the issue, which shouldn't happen I think
I think there's a bug in how BrokenMasterLabelNudgerJob
checks for whether labels are missing -- despite of waiting for 5 minutes to add the ping, it's still relying only the original event attributes from 5 minutes ago to decide whether labels are missing or not -- what should happen is that the BrokenMasterLabelNudgerJob
should just directly make a request to the issue object and check the labels, to ensure the result is reflecting the latest.
Edited by Jennifer Li