improve provider error messages during ingestion

Signed-off-by: Fredrik Adelöw <freben@gmail.com>
This commit is contained in:
Fredrik Adelöw
2023-09-22 10:00:39 +02:00
parent 58f7ee0a8f
commit 890e3b5ad4
14 changed files with 66 additions and 13 deletions
+14
View File
@@ -0,0 +1,14 @@
---
'@backstage/plugin-catalog-backend-module-bitbucket-server': patch
'@backstage/plugin-catalog-backend-module-bitbucket-cloud': patch
'@backstage/plugin-catalog-backend-module-puppetdb': patch
'@backstage/plugin-catalog-backend-module-msgraph': patch
'@backstage/plugin-catalog-backend-module-gerrit': patch
'@backstage/plugin-catalog-backend-module-github': patch
'@backstage/plugin-catalog-backend-module-gitlab': patch
'@backstage/plugin-catalog-backend-module-azure': patch
'@backstage/plugin-catalog-backend-module-ldap': patch
'@backstage/plugin-catalog-backend-module-aws': patch
---
Make sure to include the error message when ingestion fails
@@ -131,7 +131,10 @@ export class AwsS3EntityProvider implements EntityProvider {
try {
await this.refresh(logger);
} catch (error) {
logger.error(`${this.getProviderName()} refresh failed`, error);
logger.error(
`${this.getProviderName()} refresh failed, ${error}`,
error,
);
}
},
});
@@ -123,7 +123,10 @@ export class AzureDevOpsEntityProvider implements EntityProvider {
try {
await this.refresh(logger);
} catch (error) {
logger.error(`${this.getProviderName()} refresh failed`, error);
logger.error(
`${this.getProviderName()} refresh failed, ${error}`,
error,
);
}
},
});
@@ -152,7 +152,10 @@ export class BitbucketCloudEntityProvider
try {
await this.refresh(logger);
} catch (error) {
logger.error(`${this.getProviderName()} refresh failed`, error);
logger.error(
`${this.getProviderName()} refresh failed, ${error}`,
error,
);
}
},
});
@@ -130,7 +130,10 @@ export class BitbucketServerEntityProvider implements EntityProvider {
try {
await this.refresh(logger);
} catch (error) {
logger.error(`${this.getProviderName()} refresh failed`, error);
logger.error(
`${this.getProviderName()} refresh failed, ${error}`,
error,
);
}
},
});
@@ -129,7 +129,10 @@ export class GerritEntityProvider implements EntityProvider {
try {
await this.refresh(logger);
} catch (error) {
logger.error(`${this.getProviderName()} refresh failed`, error);
logger.error(
`${this.getProviderName()} refresh failed, ${error}`,
error,
);
}
},
});
@@ -163,7 +163,10 @@ export class GithubEntityProvider implements EntityProvider, EventSubscriber {
try {
await this.refresh(logger);
} catch (error) {
logger.error(`${this.getProviderName()} refresh failed`, error);
logger.error(
`${this.getProviderName()} refresh failed, ${error}`,
error,
);
}
},
});
@@ -831,7 +831,10 @@ export class GithubMultiOrgEntityProvider implements EntityProvider {
try {
await this.read({ logger });
} catch (error) {
logger.error(`${this.getProviderName()} refresh failed`, error);
logger.error(
`${this.getProviderName()} refresh failed, ${error}`,
error,
);
}
},
});
@@ -584,7 +584,10 @@ export class GithubOrgEntityProvider
try {
await this.read({ logger });
} catch (error) {
logger.error(`${this.getProviderName()} refresh failed`, error);
logger.error(
`${this.getProviderName()} refresh failed, ${error}`,
error,
);
}
},
});
@@ -133,7 +133,10 @@ export class GitlabDiscoveryEntityProvider implements EntityProvider {
try {
await this.refresh(logger);
} catch (error) {
logger.error(`${this.getProviderName()} refresh failed`, error);
logger.error(
`${this.getProviderName()} refresh failed, ${error}`,
error,
);
}
},
});
@@ -155,7 +155,10 @@ export class GitlabOrgDiscoveryEntityProvider implements EntityProvider {
try {
await this.refresh(logger);
} catch (error) {
logger.error(`${this.getProviderName()} refresh failed`, error);
logger.error(
`${this.getProviderName()} refresh failed, ${error}`,
error,
);
}
},
});
@@ -225,7 +225,10 @@ export class LdapOrgEntityProvider implements EntityProvider {
try {
await this.read({ logger });
} catch (error) {
logger.error(`${this.getProviderName()} refresh failed`, error);
logger.error(
`${this.getProviderName()} refresh failed, ${error}`,
error,
);
}
},
});
@@ -352,7 +352,10 @@ export class MicrosoftGraphOrgEntityProvider implements EntityProvider {
try {
await this.read({ logger });
} catch (error) {
logger.error(`${this.getProviderName()} refresh failed`, error);
logger.error(
`${this.getProviderName()} refresh failed, ${error}`,
error,
);
}
},
});
@@ -148,7 +148,10 @@ export class PuppetDbEntityProvider implements EntityProvider {
try {
await this.refresh(logger);
} catch (error) {
logger.error(`${this.getProviderName()} refresh failed`, error);
logger.error(
`${this.getProviderName()} refresh failed, ${error}`,
error,
);
}
},
});