christianlight/tutor
6
0
Fork 0
mirror of https://github.com/ChristianLight/tutor.git synced 2024-09-26 19:29:03 +00:00
Code Issues Releases Activity

fix: double config prompting during upgrade

Browse Source
This commit is contained in:
Régis Behmo 2023-05-30 16:17:07 +02:00
parent 189bfb2a7a
commit 64f36d910d
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
tutor/commands/compose.py
View File

@ -93,6 +93,8 @@ def launch(
run_for_prod = context_name != "dev" run_for_prod = context_name != "dev"
utils.warn_macos_docker_memory() utils.warn_macos_docker_memory()
# Upgrade has to run before configuration
interactive_upgrade(context, not non_interactive, run_for_prod) interactive_upgrade(context, not non_interactive, run_for_prod)
interactive_configuration(context, not non_interactive, run_for_prod) interactive_configuration(context, not non_interactive, run_for_prod)
@ -163,10 +165,11 @@ Are you sure you want to continue?"""
) )
# Update env and configuration # Update env and configuration
interactive_configuration(context, interactive, run_for_prod) # Don't run in interactive mode, otherwise users gets prompted twice.
interactive_configuration(context, False, run_for_prod)
# Post upgrade # Post upgrade
if run_upgrade_from_release and interactive: if interactive:
question = f"""Your platform is being upgraded from {run_upgrade_from_release.capitalize()}. question = f"""Your platform is being upgraded from {run_upgrade_from_release.capitalize()}.
If you run custom Docker images, you must rebuild them now by running the following command in a different shell: If you run custom Docker images, you must rebuild them now by running the following command in a different shell: