diff --git a/law/job/base.py b/law/job/base.py index 50981f6d..be0e2ab0 100644 --- a/law/job/base.py +++ b/law/job/base.py @@ -1324,7 +1324,6 @@ def __init__(self, path, copy=None, share=None, forward=None, increment=None, po render_local = maybe_set(render_local, False) # store attributes, apply residual defaults - # TODO: move to job rendering by default self.path = os.path.abspath(os.path.expandvars(os.path.expanduser(get_path(path)))) self.copy = True if copy is None else bool(copy) self.share = False if share is None else bool(share) @@ -1356,7 +1355,7 @@ def __init__(self, path, copy=None, share=None, forward=None, increment=None, po "but rendering is enabled which has no effect".format(self.path), ) if self.share and self.render_local: - logger.warning( + logger.error( "input file at {} is configured to be shared across jobs but local rendering is " "active, potentially resulting in wrong file content".format(self.path), ) @@ -1365,6 +1364,11 @@ def __init__(self, path, copy=None, share=None, forward=None, increment=None, po "input file at {} is configured to be rendered locally and within the job, which " "is likely unnecessary".format(self.path), ) + if not self.render_local and not self.render_job: + logger.error( + "input file at {} is configured not to be rendered at all, which will most likely " + "cause errors on remote worker nodes".format(self.path), + ) # different path variants as seen by jobs self.path_sub_abs = None