pagure

#2208 Make add_file_to_git use a lock as we do for our other git repositories
Merged by pingou. Opened by pingou.
fix_upload_attachments  into  master

Download 2208.patch

This is in order to prevent breaking the git history when two requests
are touching the same git repo at the same time.
We had done most of these git calls but looks like we missed this one.

Fixes https://pagure.io/pagure/issue/2200

Looks reasonable to me. I'm not sure how easy it would be to test this. I suppose you could start a couple of subprocesses or something, but I think it's OK to consider that at your option.

LGTM.

This is going to be most tricky to test so I think I'll pass for now.

Thanks for your review!

rebased

rebased

Pull-Request has been merged by pingou
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.