From 84e4027f90c5dc053a597ce643c16699db24ecd9 Mon Sep 17 00:00:00 2001 From: Mario Limonciello Date: Thu, 23 Jan 2025 10:44:10 -0600 Subject: [PATCH] trivial: contrib: wrap the process in try/except to catch server issues If lore has load problems, don't bail, just wrap them and try again on the next cycle. Signed-off-by: Mario Limonciello --- contrib/process_linux_firmware.py | 18 +++++++++++------- 1 file changed, 11 insertions(+), 7 deletions(-) diff --git a/contrib/process_linux_firmware.py b/contrib/process_linux_firmware.py index 8c024016..5bb1fd35 100755 --- a/contrib/process_linux_firmware.py +++ b/contrib/process_linux_firmware.py @@ -376,19 +376,23 @@ if __name__ == "__main__": logging.getLogger("").addHandler(console) while True: - conn = sqlite3.connect(args.database) - # update the database - update_database(conn, args.url) - if args.dry: remote = "" else: remote = args.remote - # process the database - process_database(conn, remote) + try: + conn = sqlite3.connect(args.database) - conn.close() + # update the database + update_database(conn, args.url) + # process the database + process_database(conn, remote) + + except urllib.error.HTTPError as e: + logging.error("Failed to fetch URL: {}".format(e)) + finally: + conn.close() if args.refresh_cycle: logging.info("Sleeping for {} minutes".format(args.refresh_cycle))