This is not clear to me - kojid part still use that option, so why it is remove here?

This is not clear to me - kojid part still use that option, so why it is remove here?

The reason is that the previous code unconditionally set opts['repo_id'], even if a value was already present. So previously, a 'repo_id' field passed into the call was ignored. It's a bit pedantic, but this preserves the api more accurately.

You're right that the repo_id option is still accepted by the task itself. Kojid does pass it when it makes subtasks, but that does not use this call.

I did debate this a bit. We can always drop the restriction later to add an "allow user to specify repo for wrapper rpm tasks" feature.

Commit 2634e190 fixes this pull-request