LB-1455 again (#3039)

* Use MB by default

* For cron job, use mb_conn

* Remove the useless default flag

mbid_mapping/manage.py

@@ -199,7 +199,9 @@ def cron_build_all_mb_caches(ctx):
 @click.pass_context
 def cron_update_all_mb_caches(ctx):
     """ Update all mb entity metadata cache in ListenBrainz. """
-    ctx.invoke(update_canonical_release_data)
+
+    # In this context we want to use mb_conn, not lb_conn, like the functions that follow
+    update_canonical_release_data(False)
     ctx.invoke(update_mb_metadata_cache)
     ctx.invoke(update_mb_artist_metadata_cache)
     ctx.invoke(update_mb_release_group_cache)

mbid_mapping/mapping/canonical_musicbrainz_data.py

@@ -128,7 +128,7 @@ def create_canonical_musicbrainz_data(use_lb_conn: bool):
         log("canonical_musicbrainz_data: done done done!")
 
 
-def update_canonical_release_data(use_lb_conn: bool = True):
+def update_canonical_release_data(use_lb_conn: bool):
     """
         Run only the canonical release data, apart from the other tables.