#1606 pull owner from correct place
Merged 2 years ago by mikem. Opened 2 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()

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

2 years ago

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

2 years ago

Commit 6f4c7b9 fixes this pull-request

Pull-Request has been merged by mikem

2 years ago