#3660 Add custom_user_metadata to build info for wrapperRPM build type
Merged a year ago by tkopecek. Opened a year ago by jcupova.
jcupova/koji issue-3655  into  master

file modified
+11
@@ -2050,6 +2050,13 @@ 

                                 'scratch': opts.get('scratch')

                             })

  

+         if opts.get('create_build') and opts.get('custom_user_metadata'):

+             try:

+                 json.dumps(opts['custom_user_metadata'])

+             except TypeError:

+                 error_msg = 'custom_user_metadata is not JSON serializable'

+                 raise koji.BuildError(error_msg)

+ 

          repo_id = opts.get('repo_id')

          if not repo_id:

              raise koji.BuildError("A repo id must be provided")
@@ -2151,6 +2158,8 @@ 

              data['task_id'] = self.id

              data['source'] = source['source']

              data['extra'] = {'source': {'original_url': source['url']}}

+             if opts.get('custom_user_metadata'):

+                 data['extra']['custom_user_metadata'] = opts['custom_user_metadata']

              self.logger.info("Reading package config for %(name)s" % data)

              pkg_cfg = self.session.getPackageConfig(build_target['dest_tag'], data['name'])

              if not opts.get('skip_tag'):
@@ -2222,6 +2231,8 @@ 

                     'rpms': rpms,

                     'logs': logs,

                     'source': source}

+         if opts.get('create_build') and opts.get('custom_user_metadata'):

+             results['custom_user_metadata'] = opts['custom_user_metadata']

  

          if not task:

              # Called as a standalone top-level task, so handle the rpms now.

rebased onto b303d1f3e8ef0d8f73e0a39f531646e61baa1015

a year ago

rebased onto 584fcdcd88442c07f612d0c3566612abe201d839

a year ago

try block below does similar thing, but will catch more errors. I think we can drop this 'if'.

It will get logged automatically on next line.

Looking to code - better than testing parentship, option create_build is more important here.

rebased onto e8a32f638b68a8a16a8d30dc11235fa1ba629a8f

a year ago

rebased onto 5ea6986504436f557a447f0ac05d12d692dae6ea

a year ago

rebased onto 4012b3e

a year ago

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

a year ago

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

a year ago

Commit bc528c8 fixes this pull-request

Pull-Request has been merged by tkopecek

a year ago
Metadata