Overall notion of enhancing the saved data for WrapperRPM builds is find and the code looks good overall.

This PR does more than it says. The title only mentions saving source, but it also:

• saves logs
• saves custom_user_metadata

Saving the logs isn't mentioned here or in the original issue, though it is certainly a valid feature to add. We should make sure the title/description of the PR is accurate.

The bit with custom_user_metadata probably needs some adjustment. This change propagates the field from the task opts to extra, as the build task does. However, we don't provide a way to set that opt, short of calling the api directly. We should probably add the --custom-user-metadata option to the wrapper-rpm command.

It's also worth noting that many wrapperRPM tasks are triggered as subtasks of other tasks, and this makes me wonder if we need to propagate that option (granted, none of those tasks have a --custom-user-metadata option either. This sort of propagation might be out of scope for this issue.

I'm a little uncertain how we want custom-user-metadata to work outside the fairly narrow scope where it was first introduced. It might be best to leave this out and revisit it later.

I've removed custom_user_metadata part. Logs are not part of the PR (it was there, just added comma in the end).

OK, looks fine, though I note that #3409 has now been edited to also mention custom_user_metadata. I still think that part is worth a more consideration before we change it.

Commit 81231068 fixes this pull-request