Skip to content

Add additional logging for sync_repository task #7254

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 2, 2020
Merged

Add additional logging for sync_repository task #7254

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions readthedocs/api/v2/views/integrations.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -407,6 +407,7 @@ def handle_webhook(self):

# Sync versions when a branch/tag was created/deleted
if event in (GITHUB_CREATE, GITHUB_DELETE):
log.info('Triggered sync_versions: project=%s event=%s', self.project, event)
return self.sync_versions_response(self.project)

# Handle pull request events
Expand Down Expand Up @@ -447,6 +448,7 @@ def handle_webhook(self):
# already have the CREATE/DELETE events. So we don't trigger the sync twice.
return self.sync_versions_response(self.project, sync=False)

log.info('Triggered sync_versions: project=%s events=%s', self.project, events)
return self.sync_versions_response(self.project)

# Trigger a build for all branches in the push
Expand Down Expand Up @@ -559,6 +561,8 @@ def handle_webhook(self):
after = data['after']
# Tag/branch created/deleted
if GITLAB_NULL_HASH in (before, after):
log.info('Triggered sync_versions: project=%s before=%s after=%s',
self.project, before, after)
return self.sync_versions_response(self.project)
# Normal push to master
try:
Expand Down Expand Up @@ -659,6 +663,8 @@ def handle_webhook(self):
# will be triggered with the normal push.
if branches:
return self.get_response_push(self.project, branches)
log.info('Triggered sync_versions: project=%s event=%s',
self.project, event)
return self.sync_versions_response(self.project)
except KeyError:
raise ParseError('Invalid request')
Expand Down