#2081 new policy for dist-repo
Merged by tkopecek. Opened by tkopecek.
tkopecek/koji issue1660  into  master

Download 2081.patch

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

rebased onto 57336632009784857882e8a7bf45b68c60d987de

these 2 asserts failed,'dist-repo' <-> 'dist_repo'

otherwise, :thumbsup:

1 new commit added

  • fix typo

Ouch, I've renamed it in the end to be consistent with existing policy names. Fixed in test.

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

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

I think we should assert login before checking policy.

We're providing the keys value to the policy, but there is no policy handler than can interpret it.

  • there are no specific handlers for this value
  • none of the generic handlers understand lists

Otoh, we're omitting the opts parameter, which could be relevant in principle.

I suppose all we really need to solve #1660 is user data, which is implicit. Perhaps the easiest thing would be to only explicitly provide the tag value.

rebased onto ef60c2e8dab9ad4b9e9a45e0368656b074c128fb

Metadata Update from @tkopecek:
- Pull-request untagged with: testing-done

:thumbsup:

Commit 164e4bfb fixes this pull-request

Pull-Request has been merged by tkopecek

Metadata