pagure

#1911 Introduce the pagure-loadjson service
Merged by pingou. Opened by pingou.
load_service  into  master

Download 1911.patch

This service takes the task of updating the database from the information
present in the git repo off the git hook and into its own process.
It will make migrating large repo to pagure an easier process. It also
offers an easier monitoring of the progress.

rebased

While I'm in favor of this splitting, I do think we need to provide the feedback from the import to the user.
For example by emailing them the result of their own push.

How much details do we want to put in the email? All the issue uid loaded? More ?

1 new commit added

  • Email the user about the output of the upload to the DB

1 new commit added

  • Indicate also the skipped files

1 new commit added

  • Ship pagure-loadjson in the tarball and install it in the spec file

This seems like it'll only import tickets?

Currently because this is what is most needed atm, but my idea is to adjust the pagure pr hook to behave in the same way

I would suggest to then update the docs et al to remove mentioning PRs until that gets in then.

(on that note: I'm missing docs :D)

1 new commit added

  • Be honest about which data type loadjson supports atm

looks good to me.
It is missing tests, but that's true for all of the extra services, and should probably be taken up separately sometime.

1 new commit added

  • Document how to install the loadjson service

rebased

+1

Thanks!

I opened a ticket for the unit-tests

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.