#7 Include the load_from_disk utility script
Closed by asaleh. Opened by pingou.
load_from_disk  into  master

Download 7.patch

This is the script used to populate pagure's database based on what is
on disk and in pkgdb.
It is very Fedora-specific and thus belongs more here than in pagure
itself.

Signed-off-by: Pierre-Yves Chibon pingou@pingoured.fr

rebased

This project doesn't appear to have a copyright claim that I see in the repo. I recommend adding a copyright header onto this file.

For PEP-8, requests should be grouped with pygit2 and sqlalchemy since it is a third party library.

For PEP-8, this if statement should be placed after the import statements below.

For PEP-8, the pagure imports should be grouped with the other third party imports, since pagure is technically a different project than pagure-dist-git.

Please document the arguments, their types, as well as the return value and its type.

Nah because then pagure won't import the proper config, I need the environment variable before the import

rebased

rebased

Ok I think this can be reviewed.

Don't be too formal, these are two utility scripts that are meant to be used when we migrate data from pkgdb and into pagure so not something we would maintain in the long run.

Thanks :)

py3-incompatible

rebased

2 new commits added

  • Include the sync_fas_group utility script
  • Include the load_from_disk utility script

rebased

2 new commits added

  • Include the sync_fas_group utility script
  • Include the load_from_disk utility script

I think we can drop this PR at this point unless you think we'll need these scripts in the future.

1 new commit added

  • Add utility script to migrate a single project from pkgdb to pagure

I think I would still like to keep these scripts around, they provide value as they allow to see what has been done and how

rebased onto a3ed1bdaeb2861a350f2a2711939f1626691fc7b

1 new commit added

  • Include pkgdb2pagure_acls.py in the releases

print()

print()

shouldn't we just take all *-sig groups dynamically?

I'll need the from __future__... then :)

I don't know that we have a way to search group for a pattern (+ that group should be a pkgdb group, which not all -sig groups are)

1 new commit added

  • Multiple fixes in the pkgdb2pagure_acls.py script

rebased onto 727d6f77c91287b42eda88af0a156d442936e731

Merged in different PR.

Pull-Request has been closed by asaleh

Metadata