Use the log_path from the confguration #284
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Requires DUNE-DAQ/confmodel#51
This fixes: #257 and #200 (change the log file extension).
This uses the following logic:
In cases 1. and 2., the log will be non overwriting (i.e. they will have a timestamp assigned to them). In the case 3. overwriting logs is controlled by the
boot
's--override-logs/--no-override-logs
switch.Note that there is logic to check that the path exists if you are running on
localhost
:Say you configured your
log_path
to/bananas/cucumbers/potatoes
,drunc
will only be able to say whether this path exists (and therefore thrown a reasonable exception) if you are running onlocalhost
.There is no check if that path is writable, or if the disk has space.