centralcloud/oncall-engine
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Add more logs for updating slack user group members (#4146)

Browse source
This commit is contained in:
Innokentii Konstantinov 2024-04-03 16:28:22 +08:00 • committed by GitHub
parent 33364b63c6
commit 8294ab5639
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 6 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
engine/apps/slack/models/slack_usergroup.py
View file

@ -110,6 +110,8 @@ class SlackUserGroup(models.Model):
logger.info(f"Skipping usergroup {self.slack_id}, already populated correctly")
return
logger.info(f"Slack user group {self.slack_id} memberlist in not up-to-date, updating, members {slack_ids}")
try:
self.update_members(slack_ids)
except SlackAPIPermissionDeniedError:
@ -121,12 +123,14 @@ class SlackUserGroup(models.Model):
try:
sc.usergroups_users_update(usergroup=self.slack_id, users=slack_ids)
except (SlackAPITokenError, SlackAPIUsergroupNotFoundError, SlackAPIInvalidUsersError) as err:
logger.warning(f"Slack usergroup update failed: {err}")
except SlackAPIError:
logger.warning(f"Slack usergroup {self.slack_id} update failed: {err}")
except SlackAPIError as slack_api_error:
logger.warning(f"Slack usergroup {self.slack_id} update failed: {slack_api_error}")
raise
else:
self.members = slack_ids
self.save(update_fields=("members",))
logger.info(f"Saved cached memberlist for slack user group {self.slack_id}, members {slack_ids}")
def get_users_from_members_for_organization(self, organization):
return organization.users.filter(