Skip to content

Commit c8e13a8

Browse files
rsaperstaednichols
andauthored
[WX-1448] Add verbose logging and timeout for getm (#7416)
Co-authored-by: Adam Nichols <anichols@broadinstitute.org>
1 parent 2ebd62b commit c8e13a8

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

cromwell-drs-localizer/src/main/scala/drs/localizer/downloaders/BulkAccessUrlDownloader.scala

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,7 @@ case class BulkAccessUrlDownloader(resolvedUrls: List[ResolvedDrsUrl]) extends D
7575
Files.deleteIfExists(getmManifestPath)
7676

7777
def generateGetmCommand(pathToMainfestJson: Path): String =
78-
s"""getm --manifest ${pathToMainfestJson.toString}"""
78+
s"""timeout 24h getm --manifest ${pathToMainfestJson.toString} -vv"""
7979
def runGetm: IO[GetmResult] =
8080
generateJsonManifest(resolvedUrls).flatMap { manifestPath =>
8181
val script = generateGetmCommand(manifestPath)

cromwell-drs-localizer/src/test/scala/drs/localizer/downloaders/BulkAccessUrlDownloaderSpec.scala

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ class BulkAccessUrlDownloaderSpec extends AnyFlatSpec with CromwellTimeoutSpec w
8080
it should "properly construct the invocation command" in {
8181
val downloader = BulkAccessUrlDownloader(oneElement)
8282
val filepath: Path = downloader.generateJsonManifest(threeElements).unsafeRunSync()
83-
val expected = s"""getm --manifest ${filepath.toString}"""
83+
val expected = s"""timeout 24h getm --manifest ${filepath.toString} -vv"""
8484
downloader.generateGetmCommand(filepath) shouldBe expected
8585
}
8686

0 commit comments

Comments
 (0)