robur/builder-web
4
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Improve builder-web setup error messages

Browse source 
Give hints what to do in case the database file does not exist, or when
the database is not of the expected version.

Addresses #82
This commit is contained in:
Reynir Björnsson 2022-07-15 12:50:32 +02:00
parent 1310c35256
commit f666c9d0d1
1 changed files with 11 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
bin/builder_web_app.ml
View file

@ -126,14 +126,23 @@ let setup_app level influx port host datadir cachedir configdir run_batch_viz_fl
run_batch_viz ~cachedir ~datadir ~configdir run_batch_viz ~cachedir ~datadir ~configdir
in in
match Builder_web.init dbpath datadir with match Builder_web.init dbpath datadir with
| exception Sqlite3.Error e ->
Format.eprintf "Error: @[@,%s.\
@,Does the database file exist? Create with `builder-db migrate`.@]\n%!"
e;
exit 2
| Error (#Caqti_error.load as e) -> | Error (#Caqti_error.load as e) ->
Format.eprintf "Error: %a\n%!" Caqti_error.pp e; Format.eprintf "Error: %a\n%!" Caqti_error.pp e;
exit 2 exit 2
| Error (`Wrong_version _ as e) ->
Format.eprintf "Error: @[@,%a.\
@,Migrate database version with `builder-migrations`,\
@,or start with a fresh database with `builder-db migrate`.@]\n%!"
Builder_web.pp_error e;
| Error ( | Error (
#Caqti_error.connect #Caqti_error.connect
| #Caqti_error.call_or_retrieve | #Caqti_error.call_or_retrieve
| `Msg _ | `Msg _ as e
| `Wrong_version _ as e
) -> ) ->
Format.eprintf "Error: %a\n%!" Builder_web.pp_error e; Format.eprintf "Error: %a\n%!" Builder_web.pp_error e;
exit 1 exit 1