chore: log format

joint_teapot/workers/canvas.py

@@ -23,12 +23,12 @@ class Canvas:
                 raise Exception(
                     f"Unable to gather students' {attr}, please contact the Canvas site admin"
                 )
-        logger.debug(f"Canvas students loaded.")
+        logger.debug(f"Canvas students loaded")
         self.assignments = self.course.get_assignments()
-        logger.debug(f"Canvas assignments loaded.")
+        logger.debug(f"Canvas assignments loaded")
         self.groups = self.course.get_groups()
-        logger.debug(f"Canvas groups loaded.")
-        logger.debug("Canvas initialized.")
+        logger.debug(f"Canvas groups loaded")
+        logger.debug("Canvas initialized")
 
 
 if __name__ == "__main__":

joint_teapot/workers/git.py

@@ -22,7 +22,7 @@ class Git:
         if not os.path.isdir(repos_dir):
             raise Exception(f"{repos_dir} does not exist! Create it first.")
         self.repos_dir = repos_dir
-        logger.debug("Git initialized.")
+        logger.debug("Git initialized")
 
     def clone_repo(self, repo_name: str, branch: str = "master") -> Repo:
         repo_dir = os.path.join(self.repos_dir, repo_name)

joint_teapot/workers/gitea.py

@@ -58,20 +58,20 @@ class Gitea:
         self.repository_api = focs_gitea.RepositoryApi(self.api_client)
         self.settings_api = focs_gitea.SettingsApi(self.api_client)
         self.user_api = focs_gitea.UserApi(self.api_client)
-        logger.debug("Gitea initialized.")
+        logger.debug("Gitea initialized")
 
     @lru_cache()
     def _get_team_id_by_name(self, name: str) -> int:
         res = self.organization_api.team_search(self.org_name, q=str(name), limit=1)
         if len(res["data"]) == 0:
-            raise Exception(f"Team not found by name {name}")
+            raise Exception(f"{name} not found by name in Gitea")
         return res["data"][0]["id"]
 
     @lru_cache()
     def _get_username_by_canvas_student(self, student: User) -> str:
         res = self.user_api.user_search(q=student.sis_login_id, limit=1)
         if len(res["data"]) == 0:
-            raise Exception(f"User not found by canvas student {student}")
+            raise Exception(f"{student} not found in Gitea")
         return res["data"][0]["username"]
 
     def add_canvas_students_to_teams(
@@ -86,12 +86,14 @@ class Gitea:
                     team_member = first(team_members, lambda x: x.login == username)
                     if team_member is None:
                         self.organization_api.org_add_team_member(team_id, username)
+                        logger.info(f"{student} added to team {team_name}")
                     else:
                         team_members.remove(team_member)
+                        logger.warning(f"{student} already in team {team_name}")
                 except Exception as e:
                     logger.error(e)
             for team_member in team_members:
-                logger.warning(
+                logger.error(
                     f"{team_member.full_name} found in team {team_name} "
                     + "but not found in Canvas students"
                 )
@@ -122,9 +124,12 @@ class Gitea:
                     repo = self.organization_api.create_org_repo(
                         self.org_name, body=body
                     )
+                    logger.info(f"Personal repo {repo_name} for {student} created")
                 except ApiException as e:
                     if e.status == 409:
-                        logger.warning(f"Personal repo for {student} already exists.")
+                        logger.warning(
+                            f"Personal repo {repo_name} for {student} already exists"
+                        )
                     else:
                         raise (e)
                 username = self._get_username_by_canvas_student(student)