repos: Reduce the logging from build report
This commit is contained in:
parent
94458d44ce
commit
333310dc9b
@ -336,11 +336,9 @@ class Repository:
|
||||
preferred = [p for p in providers
|
||||
if p.source_name in prefer]
|
||||
if preferred and len(preferred) < len(providers):
|
||||
LOG.info("Using preferred providers: %s %s",
|
||||
dep, preferred)
|
||||
providers = preferred
|
||||
if len(providers) > 1:
|
||||
LOG.warning('Ambiguous provide: %s (%s)', dep, providers)
|
||||
LOG.debug('Ambiguous provide: %s (%s)', dep, providers)
|
||||
# choose an (almost) random provider
|
||||
p = providers.pop()
|
||||
chroot.add(p)
|
||||
@ -443,6 +441,13 @@ class BuildReporter:
|
||||
self.ignore = ignore
|
||||
self.prefer = prefer
|
||||
self._cache = {} if caching else None
|
||||
self._warned_on = set()
|
||||
|
||||
def _warn(self, text, subject):
|
||||
if subject in self._warned_on:
|
||||
return
|
||||
LOG.warning(text, subject)
|
||||
self._warned_on.add(subject)
|
||||
|
||||
def report(self, source_name):
|
||||
if self._cache:
|
||||
@ -522,15 +527,14 @@ class BuildReporter:
|
||||
for _dep, provider in self.from_repo.providers(dep)
|
||||
if provider.source_name not in self.ignore)
|
||||
if not providers:
|
||||
LOG.warning("No providers for %s", dep)
|
||||
self._warn("No providers for %s", dep)
|
||||
elif len(providers) > 1:
|
||||
LOG.warning("%d provider(s) for %s: %s",
|
||||
len(providers), dep, list(providers))
|
||||
preferred = [p for p in providers
|
||||
if p.source_name in self.prefer]
|
||||
if preferred and len(preferred) < len(providers):
|
||||
LOG.info("Using preferred providers: %s", preferred)
|
||||
providers = preferred
|
||||
if len(providers) > 1:
|
||||
self._warn("Multiple providers for %s", dep)
|
||||
result.update((kind, dep, provider) for provider in providers)
|
||||
return TripletReport([self.from_repo, self.to_repo], result)
|
||||
|
||||
@ -574,7 +578,7 @@ def missing_rating(from_repo, to_repo):
|
||||
try:
|
||||
rbr = reporter.recursive_closure([name])
|
||||
except Exception:
|
||||
LOG.error("Failed to build recursive build report"
|
||||
LOG.error("Failed to build recursive build report "
|
||||
"for %s", name.decode(), exc_info=True)
|
||||
else:
|
||||
for other_name in rbr:
|
||||
|
Loading…
x
Reference in New Issue
Block a user