Include plugin name into logged error messages
We already include the plugin name into all logged warnings. Do the
same for errors.
Change-Id: I18ddc34727a69a1eb4476361325d949096608591
Signed-off-by: Edwin Kempin <edwin.kempin@sap.com>
diff --git a/src/main/java/com/googlesource/gerrit/plugins/importer/ImportProject.java b/src/main/java/com/googlesource/gerrit/plugins/importer/ImportProject.java
index bc8061e..ca6efb5 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/importer/ImportProject.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/importer/ImportProject.java
@@ -19,6 +19,7 @@
import com.google.common.base.Strings;
import com.google.gerrit.common.errors.NoSuchAccountException;
+import com.google.gerrit.extensions.annotations.PluginName;
import com.google.gerrit.extensions.annotations.RequiresCapability;
import com.google.gerrit.extensions.restapi.BadRequestException;
import com.google.gerrit.extensions.restapi.ResourceConflictException;
@@ -98,6 +99,7 @@
private static Logger log = LoggerFactory.getLogger(ImportProject.class);
private static Version v2_11 = new Version("2.11");
+ private final String pluginName;
private final ProjectCache projectCache;
private final OpenRepositoryStep openRepoStep;
private final ConfigureRepositoryStep configRepoStep;
@@ -122,6 +124,7 @@
@Inject
ImportProject(
+ @PluginName String pluginName,
ProjectCache projectCache,
OpenRepositoryStep openRepoStep,
ConfigureRepositoryStep configRepoStep,
@@ -135,6 +138,7 @@
ImportLog importLog,
ProjectsCollection projects,
@Assisted Project.NameKey targetProject) {
+ this.pluginName = pluginName;
this.projectCache = projectCache;
this.openRepoStep = openRepoStep;
this.configRepoStep = configRepoStep;
@@ -253,11 +257,11 @@
importLog.onImport((IdentifiedUser) currentUser.get(), srcProject,
targetProject, input.from, e);
String msg = input.from != null
- ? format("Unable to transfer project '%s' from"
+ ? format("[%s] Unable to transfer project '%s' from"
+ " source gerrit host '%s'.",
- srcProject.get(), input.from)
- : format("Unable to copy project '%s'.",
- srcProject.get());
+ pluginName, srcProject.get(), input.from)
+ : format("[%s] Unable to copy project '%s'.",
+ pluginName, srcProject.get());
log.error(msg, e);
throw e;
}
diff --git a/src/main/java/com/googlesource/gerrit/plugins/importer/ReplayChangesStep.java b/src/main/java/com/googlesource/gerrit/plugins/importer/ReplayChangesStep.java
index c7d058f..dcebaff 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/importer/ReplayChangesStep.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/importer/ReplayChangesStep.java
@@ -142,8 +142,8 @@
try {
replayChange(rw, c);
} catch (Exception e) {
- log.error(String.format("Failed to replay change %s.",
- Url.decode(c.id)), e);
+ log.error(String.format("[%s] Failed to replay change %s.",
+ pluginName, Url.decode(c.id)), e);
throw e;
}
pm.update(1);