log errors when tech insights throws an error

Signed-off-by: goenning <me@goenning.net>
This commit is contained in:
goenning
2022-01-07 10:25:42 +00:00
parent cfca607777
commit 19f0f93504
2 changed files with 24 additions and 9 deletions
+5
View File
@@ -0,0 +1,5 @@
---
'@backstage/plugin-tech-insights-backend': patch
---
log errors on tech insights fact retriever
@@ -16,6 +16,7 @@
import {
FactRetriever,
FactRetrieverContext,
TechInsightFact,
TechInsightsStore,
} from '@backstage/plugin-tech-insights-node';
import { FactRetrieverRegistry } from './FactRetrieverRegistry';
@@ -108,15 +109,24 @@ export class FactRetrieverEngine {
this.logger.info(
`Retrieving facts for fact retriever ${factRetriever.id}`,
);
const facts = await factRetriever.handler({
...this.factRetrieverContext,
entityFilter: factRetriever.entityFilter,
});
if (this.logger.isDebugEnabled()) {
this.logger.debug(
`Retrieved ${facts.length} facts for fact retriever ${
factRetriever.id
} in ${duration(startTimestamp)}`,
let facts: TechInsightFact[] = [];
try {
facts = await factRetriever.handler({
...this.factRetrieverContext,
entityFilter: factRetriever.entityFilter,
});
if (this.logger.isDebugEnabled()) {
this.logger.debug(
`Retrieved ${facts.length} facts for fact retriever ${
factRetriever.id
} in ${duration(startTimestamp)}`,
);
}
} catch (e) {
this.logger.error(
`Failed to retrieve facts for retriever ${factRetriever.id}`,
e,
);
}