Skip to content

Commit

Permalink
Merge pull request #224 from ihmeuw/feature/permissions
Browse files Browse the repository at this point in the history
    Category: feature
    JIRA issue: MIC-3389

This mask creates directories with 775 permissions rather than the original 755 permissions.
Testing

Ran single run and parallel run, inspected directory permissions.
  • Loading branch information
hussain-jafari committed Sep 20, 2022
2 parents a5b1aa5 + 10c9e48 commit 1ddae15
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion src/vivarium/interface/cli.py
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,7 @@
"""
import cProfile
import os
import pstats
import shutil
from pathlib import Path
Expand Down Expand Up @@ -106,7 +107,10 @@ def run(
configure_logging_to_terminal(verbose)

start = time()
results_root = get_output_root(results_directory, model_specification, artifact_path)

results_root = get_output_root(results_directory, model_specification)
# Update permissions mask (assign to variable to avoid printing previous value)
_ = os.umask(0o002)
results_root.mkdir(parents=True, exist_ok=False)

configure_logging_to_file(results_root)
Expand Down

0 comments on commit 1ddae15

Please sign in to comment.