treestatus: convert logging of Treestatus API errors to info level (Bug 1897044) (#204)

The `error` and `exception` level logging calls cause Sentry to treat
expected states as new issues. Lower the level to `info` to quiet down
Sentry. Issues experienced by the Treestatus API will still show up in
the LandoAPI logs, and at the `info` level we can still debug which
code paths are taken by various requests.

Author: cgsheeh

landoui/treestatus.py

@@ -141,7 +141,7 @@ def update_treestatus(api: TreestatusAPI, update_trees_form: TreeStatusUpdateTre
             json=update_trees_form.to_submitted_json(),
         )
     except LandoAPIError as exc:
-        logger.exception("Request to update trees status failed.")
+        logger.info("Request to update trees status failed.")
         if not exc.detail:
             raise exc
 
@@ -200,7 +200,7 @@ def new_tree_handler(api: TreestatusAPI, form: TreeStatusNewTreeForm):
             },
         )
     except LandoAPIError as exc:
-        logger.exception(f"Could not create new tree {tree}.")
+        logger.info(f"Could not create new tree {tree}.")
 
         if not exc.detail:
             raise exc
@@ -226,14 +226,17 @@ def treestatus_tree(tree: str):
         if not exc.detail or not exc.status_code:
             raise
 
-        error = f"Error received from LandoAPI: {exc.detail}"
-        logger.error(error)
-        flash(error, "error")
+        # Avoid displaying an error for a 404.
+        if exc.status_code != 404:
+            error = f"Error received from LandoAPI: {exc.detail}"
+            logger.info(error)
+            flash(error, "error")
+
         return redirect(request.referrer, code=exc.status_code)
 
     logs = logs_response.get("result")
     if not logs:
-        logger.error(f"Could not retrieve logs for tree {tree}.")
+        logger.info(f"Could not retrieve logs for tree {tree}.")
         flash(
             f"Could not retrieve status logs for {tree} from Lando, try again later.",
             "error",
@@ -294,7 +297,7 @@ def update_change(id: int):
             **action.request_args,
         )
     except LandoAPIError as exc:
-        logger.exception(f"Stack entry {id} failed to update.")
+        logger.info(f"Stack entry {id} failed to update.")
 
         if not exc.detail:
             raise exc
@@ -340,7 +343,7 @@ def update_log(id: int):
             json=json_body,
         )
     except LandoAPIError as exc:
-        logger.exception(f"Log entry {id} failed to update.")
+        logger.info(f"Log entry {id} failed to update.")
 
         if not exc.detail:
             raise exc