#962 put source target scratch into policy_data in make_task
Merged by mikem. Opened by julian8628.
julian8628/koji issue/150  into  master

Download 962.patch

fixes: #150

  • source: check if src, spec_url, url are in arguments, if yes fill the value in. skipping newRepo task
  • target: check if target, build_target, target_info are in arguments, if yes, then try to get target name, and make sure if it exists. If so, put target name into policy_data
  • scratch: check if scratch is in opts, if true, putting the value, otherwise, putting False

This swallows Exceptions and skipping putting extra data into channel policy unless the target doesn't exist.

A few tweaks here:
https://github.com/mikem23/koji-playground/commits/pagure/pr/962

Seems to work fine here

There are a few more changes I'd like to make here, but they can wait until later. This PR is sufficient for now.

2 new commits added

  • minor adjustments for readability
  • drop redundant warning

A few tweaks here:
https://github.com/mikem23/koji-playground/commits/pagure/pr/962

Thanks! I've applied your commits here

Commit e67bd0f3 fixes this pull-request

Pull-Request has been merged by mikem

Metadata