diff --git a/roles/check-release-approval/files/check_approval.py b/roles/check-release-approval/files/check_approval.py index f38b7a07d3..fdb0ae093e 100755 --- a/roles/check-release-approval/files/check_approval.py +++ b/roles/check-release-approval/files/check_approval.py @@ -127,7 +127,8 @@ class GerritChange(object): # Fetch PTL and release liaisons liaisons = get_liaisons(self.workspace, team) - if 'email' in govteam['ptl']: + # Some teams may be PTL-less, so don't assume we have PTL info + if 'email' in govteam.get('ptl', {}): liaisons.append(govteam['ptl']['email']) LOG.debug('%s needs %s' % (deliv_file, liaisons))