From 9a141658513df33df2c7d477939b2a136e3e79ff Mon Sep 17 00:00:00 2001
From: Ian McEwen <mian@cyverse.org>
Date: Tue, 21 Jul 2026 13:04:43 -0700
Subject: [PATCH] Add MountDataStore default-true to Job constructors

MountDataStore already existed on the Job struct but was always its zero
value (false) when constructed via New() or NewAnalysis(). Set it to true
in both constructors so that jobs created from JSON with the field absent
behave as if the data store is to be mounted, matching the historical default.
---
 jobs.go | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/jobs.go b/jobs.go
index b9eef26..c84f817 100644
--- a/jobs.go
+++ b/jobs.go
@@ -138,6 +138,7 @@ func New(cfg *viper.Viper) *Job {
 		NowDate:        n,
 		SubmissionDate: n,
 		ArchiveLogs:    true,
+		MountDataStore: true,
 		CondorLogPath:  lp,
 		FilterFiles:    paths,
 		IRODSBase:      irodsBase,
@@ -174,6 +175,7 @@ func NewAnalysis(cfg *AnalysisConfig, data []byte) (*Analysis, error) {
 		NowDate:        n,
 		SubmissionDate: n,
 		ArchiveLogs:    true,
+		MountDataStore: true,
 		CondorLogPath:  cfg.LogPath,
 		FilterFiles:    cfg.FilterFiles,
 		IRODSBase:      cfg.IRODSBase,