Skip to content

Properly fix the submitter_id problem

Veronika Kabátová requested to merge veruu/pipeline-trigger:fix-pw1 into master

submitter_id used to be automatically included in the response, no matter what we set up as expected fields. This can be verified by manually querying patches from previous pipelines -- the trigger didn't change in the meanwhile but the field is suddenly not included.

patches = rpc.patch_list({}, True, RPC_FIELDS)
for patch in patches:
    print(patch['submitter_id'])
> KeyError

patches = rpc.patch_list({}, True, RPC_FIELDS + ['submitter_id'])
for patch in patches:
    print(patch['submitter_id'])
> <prints fine>

Revert the submitter mockup and instead add the field to the request.

Signed-off-by: Veronika Kabatova vkabatov@redhat.com

Merge request reports