Mercurial > hg > tc-symbol-server
annotate agent/src/jetbrains/buildServer/symbols/SymbolsIndexer.java @ 26:87b24e4efcc3
fixed multiple indexing of the same pdb file
author | Evgeniy.Koshkin |
---|---|
date | Wed, 07 Aug 2013 17:29:18 +0400 |
parents | f4e0050e2a2f |
children | 76ed2d58c871 |
rev | line source |
---|---|
2
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
1 package jetbrains.buildServer.symbols; |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
2 |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
3 import com.intellij.execution.configurations.GeneralCommandLine; |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
4 import jetbrains.buildServer.ExecResult; |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
5 import jetbrains.buildServer.SimpleCommandLineProcessRunner; |
3 | 6 import jetbrains.buildServer.agent.*; |
2
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
7 import jetbrains.buildServer.agent.artifacts.ArtifactsWatcher; |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
8 import jetbrains.buildServer.agent.impl.artifacts.ArtifactsBuilderAdapter; |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
9 import jetbrains.buildServer.agent.impl.artifacts.ArtifactsCollection; |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
10 import jetbrains.buildServer.agent.plugins.beans.PluginDescriptor; |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
11 import jetbrains.buildServer.util.EventDispatcher; |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
12 import jetbrains.buildServer.util.FileUtil; |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
13 import org.apache.log4j.Logger; |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
14 import org.jetbrains.annotations.NotNull; |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
15 import org.jetbrains.annotations.Nullable; |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
16 |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
17 import java.io.File; |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
18 import java.io.IOException; |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
19 import java.util.*; |
26
87b24e4efcc3
fixed multiple indexing of the same pdb file
Evgeniy.Koshkin
parents:
25
diff
changeset
|
20 import java.util.concurrent.CopyOnWriteArrayList; |
2
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
21 |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
22 /** |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
23 * @author Evgeniy.Koshkin |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
24 */ |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
25 public class SymbolsIndexer extends ArtifactsBuilderAdapter { |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
26 |
7 | 27 private static final Logger LOG = Logger.getLogger(SymbolsIndexer.class); |
2
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
28 |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
29 public static final String PDB_FILE_EXTENSION = "pdb"; |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
30 |
5 | 31 public static final String SYMBOLS_EXE = "JetBrains.CommandLine.Symbols.exe"; |
2
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
32 public static final String DUMP_SYMBOL_SIGN_CMD = "dumpSymbolSign"; |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
33 |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
34 @NotNull private final ArtifactsWatcher myArtifactsWatcher; |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
35 @NotNull private final File myNativeToolPath; |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
36 @Nullable private AgentRunningBuild myBuild; |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
37 @Nullable private Collection<File> mySymbolsToProcess; |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
38 |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
39 public SymbolsIndexer(@NotNull final PluginDescriptor pluginDescriptor, @NotNull final EventDispatcher<AgentLifeCycleListener> agentDispatcher, @NotNull final ArtifactsWatcher artifactsWatcher) { |
12 | 40 myArtifactsWatcher = artifactsWatcher; |
2
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
41 myNativeToolPath = new File(new File(pluginDescriptor.getPluginRoot(), "bin"), SYMBOLS_EXE); |
12 | 42 |
2
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
43 agentDispatcher.addListener(new AgentLifeCycleAdapter() { |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
44 @Override |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
45 public void buildStarted(@NotNull final AgentRunningBuild runningBuild) { |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
46 myBuild = runningBuild; |
26
87b24e4efcc3
fixed multiple indexing of the same pdb file
Evgeniy.Koshkin
parents:
25
diff
changeset
|
47 mySymbolsToProcess = new CopyOnWriteArrayList<File>(); |
2
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
48 } |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
49 |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
50 @Override |
5 | 51 public void afterAtrifactsPublished(@NotNull AgentRunningBuild build, @NotNull BuildFinishedStatus buildStatus) { |
52 super.afterAtrifactsPublished(build, buildStatus); | |
12 | 53 if (myBuild == null || mySymbolsToProcess == null) return; |
3 | 54 if(myBuild.getBuildFeaturesOfType(SymbolsConstants.BUILD_FEATURE_TYPE).isEmpty()) return; |
55 | |
2
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
56 if (mySymbolsToProcess.isEmpty()) { |
3 | 57 myBuild.getBuildLogger().warning("Symbols weren't found in artifacts to be published."); |
2
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
58 LOG.debug("Symbols weren't found in artifacts to be published."); |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
59 } else { |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
60 try { |
12 | 61 final File symbolSignaturesFile = dumpSymbolSignatures(mySymbolsToProcess, myBuild.getBuildTempDirectory(), myBuild.getBuildLogger()); |
5 | 62 if(symbolSignaturesFile.exists()){ |
63 myArtifactsWatcher.addNewArtifactsPath(symbolSignaturesFile + "=>" + ".teamcity/symbols"); | |
64 } | |
2
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
65 } catch (IOException e) { |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
66 LOG.error("Error while dumping symbols/binaries signatures.", e); |
5 | 67 myBuild.getBuildLogger().error("Error while dumping symbols/binaries signatures."); |
68 myBuild.getBuildLogger().exception(e); | |
2
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
69 } |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
70 } |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
71 mySymbolsToProcess = null; |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
72 myBuild = null; |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
73 } |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
74 }); |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
75 } |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
76 |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
77 @Override |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
78 public void afterCollectingFiles(@NotNull List<ArtifactsCollection> artifacts) { |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
79 super.afterCollectingFiles(artifacts); |
12 | 80 if(myBuild == null || mySymbolsToProcess == null) return; |
18
d1c5ce19d0f7
added warning in case source info was not found in pdb
Evgeniy.Koshkin
parents:
14
diff
changeset
|
81 final BuildProgressLogger buildLogger = myBuild.getBuildLogger(); |
2
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
82 if(myBuild.getBuildFeaturesOfType(SymbolsConstants.BUILD_FEATURE_TYPE).isEmpty()){ |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
83 LOG.debug(SymbolsConstants.BUILD_FEATURE_TYPE + " build feature disabled. No indexing performed."); |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
84 return; |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
85 } |
5 | 86 LOG.debug(SymbolsConstants.BUILD_FEATURE_TYPE + " build feature enabled. Searching for suitable files."); |
20 | 87 final Collection<File> pdbFiles = getArtifactPathsByFileExtension(artifacts, PDB_FILE_EXTENSION); |
25 | 88 if(pdbFiles.isEmpty()) return; |
89 | |
90 final FileUrlProvider urlProvider = getUrlProviderForBuild(myBuild, buildLogger); | |
91 if(urlProvider == null) return; | |
92 | |
93 final PdbFilePatcher pdbFilePatcher = new PdbFilePatcher(myBuild.getBuildTempDirectory(), new SrcSrvStreamBuilder(urlProvider)); | |
12 | 94 for(File pdbFile : pdbFiles){ |
26
87b24e4efcc3
fixed multiple indexing of the same pdb file
Evgeniy.Koshkin
parents:
25
diff
changeset
|
95 if(mySymbolsToProcess.contains(pdbFile)){ |
87b24e4efcc3
fixed multiple indexing of the same pdb file
Evgeniy.Koshkin
parents:
25
diff
changeset
|
96 LOG.debug(String.format("File %s already processed. Skipped.", pdbFile.getAbsolutePath())); |
87b24e4efcc3
fixed multiple indexing of the same pdb file
Evgeniy.Koshkin
parents:
25
diff
changeset
|
97 continue; |
87b24e4efcc3
fixed multiple indexing of the same pdb file
Evgeniy.Koshkin
parents:
25
diff
changeset
|
98 } |
12 | 99 try { |
18
d1c5ce19d0f7
added warning in case source info was not found in pdb
Evgeniy.Koshkin
parents:
14
diff
changeset
|
100 buildLogger.message("Indexing sources appeared in file " + pdbFile.getAbsolutePath()); |
d1c5ce19d0f7
added warning in case source info was not found in pdb
Evgeniy.Koshkin
parents:
14
diff
changeset
|
101 pdbFilePatcher.patch(pdbFile, buildLogger); |
12 | 102 mySymbolsToProcess.add(pdbFile); |
103 } catch (Throwable e) { | |
104 LOG.error("Error occurred while patching symbols file " + pdbFile, e); | |
18
d1c5ce19d0f7
added warning in case source info was not found in pdb
Evgeniy.Koshkin
parents:
14
diff
changeset
|
105 buildLogger.error("Error occurred while patching symbols file " + pdbFile); |
d1c5ce19d0f7
added warning in case source info was not found in pdb
Evgeniy.Koshkin
parents:
14
diff
changeset
|
106 buildLogger.exception(e); |
12 | 107 } |
108 } | |
2
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
109 } |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
110 |
25 | 111 @Nullable |
112 private FileUrlProvider getUrlProviderForBuild(@NotNull AgentRunningBuild build, BuildProgressLogger buildLogger) { | |
113 String serverUrl = build.getSharedConfigParameters().get(SymbolsConstants.SERVER_URL_PARAMETER_NAME); | |
114 if(serverUrl == null){ | |
115 final String message = String.format("Configuration parameter %s was not set. ", SymbolsConstants.SERVER_URL_PARAMETER_NAME); | |
116 LOG.warn(message); | |
117 buildLogger.warning(message); | |
118 return null; | |
119 } | |
120 return new FileUrlProvider(serverUrl, build.getBuildId(), build.getCheckoutDirectory()); | |
121 } | |
122 | |
2
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
123 private Collection<File> getArtifactPathsByFileExtension(List<ArtifactsCollection> artifactsCollections, String fileExtension){ |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
124 final Collection<File> result = new HashSet<File>(); |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
125 for(ArtifactsCollection artifactsCollection : artifactsCollections){ |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
126 if(artifactsCollection.isEmpty()) continue; |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
127 for (File artifact : artifactsCollection.getFilePathMap().keySet()){ |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
128 if(FileUtil.getExtension(artifact.getPath()).equalsIgnoreCase(fileExtension)) |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
129 result.add(artifact); |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
130 } |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
131 } |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
132 return result; |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
133 } |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
134 |
3 | 135 private File dumpSymbolSignatures(Collection<File> files, File targetDir, BuildProgressLogger buildLogger) throws IOException { |
2
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
136 final File tempFile = FileUtil.createTempFile(targetDir, "symbol-signatures-", ".xml", false); |
12 | 137 runNativeTool(DUMP_SYMBOL_SIGN_CMD, files, tempFile, buildLogger); |
2
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
138 return tempFile; |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
139 } |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
140 |
12 | 141 private void runNativeTool(String cmd, Collection<File> files, File output, BuildProgressLogger buildLogger){ |
2
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
142 final GeneralCommandLine commandLine = new GeneralCommandLine(); |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
143 commandLine.setExePath(myNativeToolPath.getPath()); |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
144 commandLine.addParameter(cmd); |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
145 commandLine.addParameter(String.format("/o=\"%s\"", output.getPath())); |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
146 for(File file : files){ |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
147 commandLine.addParameter(file.getPath()); |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
148 } |
5 | 149 buildLogger.message(String.format("Running command %s", commandLine.getCommandLineString())); |
2
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
150 final ExecResult execResult = SimpleCommandLineProcessRunner.runCommand(commandLine, null); |
21 | 151 final String stdout = execResult.getStdout(); |
152 if(!stdout.isEmpty()){ | |
153 buildLogger.message("Stdout: " + stdout); | |
154 } | |
2
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
155 if (execResult.getExitCode() == 0) return; |
5 | 156 buildLogger.warning(String.format("%s ends with non-zero exit code.", SYMBOLS_EXE)); |
21 | 157 buildLogger.warning("Stdout: " + stdout); |
158 buildLogger.warning("Stderr: " + execResult.getStderr()); | |
14
5856cab1ca05
logged exception end output of the native symbols utility
Evgeniy.Koshkin
parents:
12
diff
changeset
|
159 buildLogger.exception(execResult.getException()); |
2
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
160 } |
c6ca0a26d93e
initial implementation of agent-side symbol indexer
Evgeniy.Koshkin
parents:
diff
changeset
|
161 } |