koji

Clone

#1606 pull owner from correct place
Merged 4 years ago by mikem. Opened 4 years ago by mikem.
mikem/koji fix-cg-owner  into  master

file modified
+1 -4 
@@ -5863,10 +5863,7 @@ 
 

              extra = json.dumps(buildinfo['extra'])
 

          else:
 

              extra = None
 

-         if buildinfo.get('owner'):
 

-             owner = get_user(self.buildinfo['owner'], strict=True)['id']
 

-         else:
 

-             owner = context.session.user_id
 

+         owner = self.buildinfo.get('owner', context.session.user_id)
 

          source = self.buildinfo.get('source')
 

          st_complete = koji.BUILD_STATES['COMPLETE']
 

          st_old = buildinfo['state']

Fixes https://pagure.io/koji/issue/1604

In this function, we have binfo, buildinfo, and self.buildinfo.
A little confusing ;)

Note that self.buildinfo['owner'] is validated in prep_build()

:thumbsup:

Metadata Update from @julian8628:
- Pull-request tagged with: testing-ready
4 years ago

:thumbsup:

Metadata Update from @mfilip:
- Pull-request tagged with: testing-done
4 years ago

Commit 6f4c7b9 fixes this pull-request

Pull-Request has been merged by mikem
4 years ago
Changes Summary 1
+1 -4
file changed
hub/kojihub.py
Powered by Pagure 5.13.3
Documentation • File an Issue • About • SSH Hostkey/Fingerprint
© Red Hat, Inc. and others.