This PR adds some additional useful data to readTaggedBuilds. If a build is explicitly tagged into more than one tag it is now easy to determine it from the results of readTaggedBuilds.
readTaggedBuilds
Isn't it too expensive for typical usecase which doesn't need these data? Doesn't it make sense to add some option for this?
I'm not sure... I know when I'm looking for the information about a build I generally want all the tags the build is in, but that is the bits I'm working on rather than the bits in koji that already work.
koji
I would use listTags(build_id) in such case. If performance should be the problem, multicall should solve it. Could it work for you?
listTags(build_id)
pretty please pagure-ci rebuild
I can look into multicall.
@jcpunk is it ok to drop this PR or would you like to continue with this?
We can drop this PR. I've not gotten my multicall setup into a pretty condition, but a workable one so I'm satisfied.
Pull-Request has been closed by tkopecek
This PR adds some additional useful data to
readTaggedBuilds. If a build is explicitly tagged into more than one tag it is now easy to determine it from the results ofreadTaggedBuilds.