fdo-mirrors/mesa
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/mesa/mesa.git synced 2025-12-20 20:20:18 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

ci: post gantt: use logging instead of print

Browse source 
Replace the print statements with logging to take advantage of the
built-in logging formatting and levels.

Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/32637>
This commit is contained in:
Deborah Brouwer 2024-12-13 11:49:53 -08:00 • committed by Deb_1543
parent cb756ae8a2
commit 9671de71a3
1 changed files with 10 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
bin/ci/ci_post_gantt.py
View file

@ -10,6 +10,7 @@
import argparse import argparse
import gitlab import gitlab
import re import re
import logging as log
import os import os
import pytz import pytz
import traceback import traceback
@ -100,7 +101,7 @@ def gitlab_post_reply_to_note(gl, event, reply_message):
return reply return reply
except gitlab.exceptions.GitlabError as e: except gitlab.exceptions.GitlabError as e:
print(f"Failed to post a reply to '{event.note['body']}': {e}") log.error(f"Failed to post a reply to '{event.note['body']}': {e}")
return None return None
@ -121,6 +122,7 @@ def parse_args() -> None:
if __name__ == "__main__": if __name__ == "__main__":
args = parse_args() args = parse_args()
log.basicConfig(level=log.INFO)
token = read_token(args.token) token = read_token(args.token)
@ -129,7 +131,7 @@ if __name__ == "__main__":
user = gl.users.get(MARGE_USER_ID) user = gl.users.get(MARGE_USER_ID)
last_event_at = args.since if args.since else read_last_event_date_from_file() last_event_at = args.since if args.since else read_last_event_date_from_file()
print(f"Retrieving Marge messages since {pretty_time(last_event_at)}\n") log.info(f"Retrieving Marge messages since {pretty_time(last_event_at)}\n")
# the "after" only considers the "2023-10-24" part, it doesn't consider the time # the "after" only considers the "2023-10-24" part, it doesn't consider the time
events = user.events.list( events = user.events.list(
@ -154,24 +156,24 @@ if __name__ == "__main__":
match = re.search(r"https://[^ ]+", event.note["body"]) match = re.search(r"https://[^ ]+", event.note["body"])
if match: if match:
try: try:
print("Found message:", event.note["body"]) log.info(f"Found message: {event.note['body']}")
pipeline_url = match.group(0)[:-1] pipeline_url = match.group(0)[:-1]
pipeline, _ = get_gitlab_pipeline_from_url(gl, pipeline_url) pipeline, _ = get_gitlab_pipeline_from_url(gl, pipeline_url)
print("Generating gantt chart...") log.info("Generating gantt chart...")
fig = generate_gantt_chart(pipeline) fig = generate_gantt_chart(pipeline)
file_name = "Gantt.html" file_name = "Gantt.html"
fig.write_html(file_name) fig.write_html(file_name)
print("Uploading gantt file...") log.info("Uploading gantt file...")
file_url = gitlab_upload_file_get_url(gl, event.project_id, file_name) file_url = gitlab_upload_file_get_url(gl, event.project_id, file_name)
print("Posting reply ...\n") log.info("Posting reply ...")
message = compose_message(file_name, file_url) message = compose_message(file_name, file_url)
gitlab_post_reply_to_note(gl, event, message) gitlab_post_reply_to_note(gl, event, message)
except Exception as e: except Exception as e:
print(f"Failed to generate gantt chart, not posting reply.{e}") log.info(f"Failed to generate gantt chart, not posting reply.{e}")
traceback.print_exc() traceback.print_exc()
if not args.since: if not args.since:
print( log.info(
f"Updating last event date to {pretty_time(last_event_at)} on {LAST_MARGE_EVENT_FILE}\n" f"Updating last event date to {pretty_time(last_event_at)} on {LAST_MARGE_EVENT_FILE}\n"
) )
with open(LAST_MARGE_EVENT_FILE, "w") as f: with open(LAST_MARGE_EVENT_FILE, "w") as f: