Commit 21f3ab9
committed
pool: add firefly onStart marker for MoverProtocol-based transfers
AbstractMoverProtocolTransferService, which handles remote/TPC transfers
(e.g., RemoteHttpTransferService for FTS third-party copies), never
called transferLifeCycle.onStart(). This meant that firefly flow-start
UDP markers were never emitted for TPC movers, even though onEnd markers
were correctly sent via DefaultPostTransferService.
Only NettyTransferService (direct client connections) called onStart(),
so pool-side SciTag flow markers for TPC transfers were incomplete:
ESnet Stardust dashboards received end markers with correct experiment
and activity IDs from transfer tags, but no corresponding start markers.
Add a startTransferLifeCycle() call in MoverTask.run() that derives the
local endpoint from the remote address using NetworkUtils.getLocalAddress
(the same approach used by DefaultPostTransferService.deriveLocalEndpoint
for onEnd) and invokes transferLifeCycle.onStart() before I/O begins.
Motivation: ATLAS AGLT2 pool logs showed 194 TPC onEnd markers with
classifier=transfer-tag and correct activityIds (11, 15, 16) on a
typical day, but zero TPC onStart markers. Direct-read start markers
(from NettyTransferService) were present but all carried activityId=1
(FQAN fallback) since worker nodes do not send SciTag headers.
Signed-off-by: Shawn McKee <smckee@umich.edu>1 parent 5fe6ec9 commit 21f3ab9
1 file changed
Lines changed: 54 additions & 1 deletion
File tree
- modules/dcache/src/main/java/org/dcache/pool/classic
Lines changed: 54 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | | - | |
| 3 | + | |
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
| |||
20 | 20 | | |
21 | 21 | | |
22 | 22 | | |
| 23 | + | |
23 | 24 | | |
24 | 25 | | |
25 | 26 | | |
| |||
29 | 30 | | |
30 | 31 | | |
31 | 32 | | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
32 | 36 | | |
33 | 37 | | |
34 | 38 | | |
| |||
43 | 47 | | |
44 | 48 | | |
45 | 49 | | |
| 50 | + | |
46 | 51 | | |
47 | 52 | | |
48 | 53 | | |
| |||
143 | 148 | | |
144 | 149 | | |
145 | 150 | | |
| 151 | + | |
146 | 152 | | |
147 | 153 | | |
148 | 154 | | |
| |||
183 | 189 | | |
184 | 190 | | |
185 | 191 | | |
| 192 | + | |
| 193 | + | |
| 194 | + | |
| 195 | + | |
| 196 | + | |
| 197 | + | |
| 198 | + | |
| 199 | + | |
| 200 | + | |
| 201 | + | |
| 202 | + | |
| 203 | + | |
| 204 | + | |
| 205 | + | |
| 206 | + | |
| 207 | + | |
| 208 | + | |
| 209 | + | |
| 210 | + | |
| 211 | + | |
| 212 | + | |
| 213 | + | |
| 214 | + | |
| 215 | + | |
| 216 | + | |
| 217 | + | |
| 218 | + | |
| 219 | + | |
| 220 | + | |
| 221 | + | |
| 222 | + | |
| 223 | + | |
| 224 | + | |
| 225 | + | |
| 226 | + | |
| 227 | + | |
| 228 | + | |
| 229 | + | |
| 230 | + | |
| 231 | + | |
| 232 | + | |
| 233 | + | |
| 234 | + | |
| 235 | + | |
| 236 | + | |
| 237 | + | |
| 238 | + | |
186 | 239 | | |
187 | 240 | | |
188 | 241 | | |
| |||
0 commit comments