Skip to content

Commit

Permalink
Add log messages for subprocesses and log file location
Browse files Browse the repository at this point in the history
  • Loading branch information
Justin Campbell committed Sep 30, 2020
1 parent 48a1510 commit f978479
Showing 1 changed file with 14 additions and 7 deletions.
21 changes: 14 additions & 7 deletions utils/venv-manager.py
Original file line number Diff line number Diff line change
Expand Up @@ -142,15 +142,16 @@ def create_environment() -> int:
int: Exit code
"""

logger.info(f"Creating virtual environment {VENV_NAME} at {VENV_DIR}")

# Check for an existing virtual environment
try:
os.mkdir(VENV_DIR)
except FileExistsError:
logger.warning(f"The directory {VENV_DIR} already exists. Exiting")
return 5

# Create virtual environment
logger.info(f"Creating virtual environment {VENV_NAME} at {VENV_DIR}")
# Create virtual environmentc
create_env_returncode, _ = run_logged_subprocess(f"cd {API_DIR} && python3 -m venv {VENV_NAME}", shell=True)
if create_env_returncode == 0:
logger.info(f"Virtual environment {VENV_NAME} created at {VENV_DIR}")
Expand Down Expand Up @@ -199,12 +200,14 @@ def delete_environment() -> int:
Returns:
int: Exit code
"""
logger.info(f"Deleting virtual environment {VENV_NAME} at {VENV_DIR}")

delete_venv_returncode, _ = run_logged_subprocess(f"rm -rf {VENV_DIR}")
if delete_venv_returncode == 0:
logger.info(f"Successfully deleted virtual environment {VENV_DIR} at {VENV_DIR}")
logger.info(f"Successfully deleted virtual environment {VENV_NAME} at {VENV_DIR}")
return 0
else:
logger.critical(f"Failed to delete virtual environment {VENV_DIR} at {VENV_DIR}. Exiting")
logger.critical(f"Failed to delete virtual environment {VENV_NAME} at {VENV_DIR}. Exiting")
return 5

def reset_environment() -> int:
Expand All @@ -218,20 +221,24 @@ def reset_environment() -> int:
Returns:
int: Exit code
"""
logger.info(f"Resetting virtual environment {VENV_NAME} at {VENV_DIR}")

delete_returncode = delete_environment()
if delete_returncode != 0:
logger.critical(f"Failed to reset virtual environment {VENV_DIR} at {VENV_DIR}. Exiting")
logger.critical(f"Failed to reset virtual environment {VENV_NAME} at {VENV_DIR}. Exiting")
return 5

create_returncode = create_environment()
if create_returncode != 0:
logger.critical(f"Failed to reset virtual environment {VENV_DIR} at {VENV_DIR}. Exiting")
logger.critical(f"Failed to reset virtual environment {VENV_NAME} at {VENV_DIR}. Exiting")
return 10

logger.info(f"Successfully reset virtual environment {VENV_DIR} at {VENV_DIR}. Exiting")
logger.info(f"Successfully reset virtual environment {VENV_NAME} at {VENV_DIR}. Exiting")


if __name__ == "__main__":
logger.info(f"Starting venv-manager. Log file available at {log_file_path}")

args = get_args()
action = args.action

Expand Down

0 comments on commit f978479

Please sign in to comment.