Skip to content

Commit

Permalink
Clarify CLI message when db needs to be upgraded (#1386)
Browse files Browse the repository at this point in the history
* Clarify that Covalent needs to be started after a db upgrade

* Update changelog
  • Loading branch information
cjao authored Oct 25, 2022
1 parent 6fb49de commit eb94356
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 2 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

- Enabling logging by default
- Removed debugging output
- Clarify cli output when `covalent db migrate` needs to be run

### Changed

Expand Down
6 changes: 4 additions & 2 deletions covalent_dispatcher/_cli/service.py
Original file line number Diff line number Diff line change
Expand Up @@ -49,8 +49,10 @@
UI_LOGFILE = get_config("user_interface.log_dir") + "/covalent_ui.log"
UI_SRVDIR = f"{os.path.dirname(os.path.abspath(__file__))}/../../covalent_ui"

MIGRATION_WARNING_MSG = "There have been changes applied to the database."
MIGRATION_COMMAND_MSG = ' (use "covalent db migrate" to run database migrations)'
MIGRATION_WARNING_MSG = "Covalent not started. The database needs to be upgraded."
MIGRATION_COMMAND_MSG = (
' (use "covalent db migrate" to run database migrations and then retry "covalent start")'
)


def _read_pid(filename: str) -> int:
Expand Down

0 comments on commit eb94356

Please sign in to comment.