switch missing builds to detailed excuse info

Also remove the list of old binaries from the excuse.
ubuntu/rebased
Ivo De Decker 5 years ago
parent e25ca70427
commit 03856a3876

@ -452,8 +452,8 @@ class ExcuseFinder(object):
(quote(arch), quote(src), quote(source_u.version), arch, oodtxt) (quote(arch), quote(src), quote(source_u.version), arch, oodtxt)
else: else:
text = "missing build on <a href=\"https://buildd.debian.org/status/logs.php?" \ text = "missing build on <a href=\"https://buildd.debian.org/status/logs.php?" \
"arch=%s&pkg=%s&ver=%s\" target=\"_blank\">%s</a>: %s" % \ "arch=%s&pkg=%s&ver=%s\" target=\"_blank\">%s</a>" % \
(quote(arch), quote(src), quote(source_u.version), arch, oodtxt) (quote(arch), quote(src), quote(source_u.version), arch)
if arch in self.options.outofsync_arches: if arch in self.options.outofsync_arches:
text = text + " (but %s isn't keeping up, so nevermind)" % (arch) text = text + " (but %s isn't keeping up, so nevermind)" % (arch)
@ -463,13 +463,17 @@ class ExcuseFinder(object):
if uptodatebins: if uptodatebins:
if self.options.ignore_cruft: if self.options.ignore_cruft:
text = text + " (but ignoring cruft, so nevermind)" text = text + " (but ignoring cruft, so nevermind)"
excuse.add_detailed_info(text)
else: else:
excuse.policy_verdict = PolicyVerdict.REJECTED_PERMANENTLY excuse.policy_verdict = PolicyVerdict.REJECTED_PERMANENTLY
excuse.addreason("cruft")
excuse.add_verdict_info(excuse.policy_verdict, text)
else: else:
excuse.policy_verdict = PolicyVerdict.REJECTED_CANNOT_DETERMINE_IF_PERMANENT excuse.policy_verdict = PolicyVerdict.REJECTED_CANNOT_DETERMINE_IF_PERMANENT
excuse.missing_build_on_arch(arch) excuse.missing_build_on_arch(arch)
excuse.addreason("missingbuild")
excuse.addhtml(text) excuse.add_verdict_info(excuse.policy_verdict, text)
excuse.add_detailed_info("old binaries on %s: %s" % (arch, oodtxt))
# if the source package has no binaries, set is_valid to False to block the update # if the source package has no binaries, set is_valid to False to block the update
if not source_u.binaries: if not source_u.binaries:

Loading…
Cancel
Save