Page MenuHomePhabricator

Package transferpy framework
Closed, ResolvedPublic

Description

The transferpy in wmfmariadbpy is a package used for database backup and recovery. Currently, it is distributed as a python script. it will be helpful for the user if we distribute it as a Debian package.

Event Timeline

Privacybatm created this task.
Privacybatm moved this task from Triage to GSOC2020 on the DBA board.

Change 598984 had a related patch set uploaded (by Privacybatm; owner: Privacybatm):
[operations/software/wmfmariadbpy@master] [WIP] wmfmariadbpy: Package wmfmariadbpy

https://gerrit.wikimedia.org/r/598984

I have uploaded a new patch set with a working deb file inside dist folder (https://gerrit.wikimedia.org/r/c/operations/software/wmfmariadbpy/+/598984/2/dist/transferpy_1.0-1_amd64.deb)

I have some doubts now:
In order to package transferpy, shall I make a new setup.py inside transferpy folder? If yes, should I move existing files inside the transferpy to new folder inside transferpy?

I ask the above question because altering setup.py would cause errors in CI because I have removed (prune in MANIFEST) the wmfmariadb files from setup.py in order to package transferpy alone.

Let me think about it. Things are getting more and more complex, maintaining a lot of (mostly unrelated) stuff in the same repo. How would you see about splitting out transferpy to its own separate repo. Would that make CI (testing and doc generation) as well as packaging easier for you? We can talk about this on today's meeting, but please think if that would simplify development for you- we can totally ask for a repo if that helps you.

Yeah, I will think about it, Thank you.

Change 602754 had a related patch set uploaded (by Privacybatm; owner: Privacybatm):
[operations/software/transferpy@master] transferpy: Package transferpy

https://gerrit.wikimedia.org/r/602754

Change 598984 abandoned by Privacybatm:
transferpy: Package transferpy

Reason:
transferpy has moved to its own repository. So patch for packaging also need to be moved. This patch will continue here at https://gerrit.wikimedia.org/r/c/operations/software/transferpy/ /602754

Thank you!

https://gerrit.wikimedia.org/r/598984

Cumin 4 for buster is packaged in an rc status: https://phabricator.wikimedia.org/T245114#6207237

https://apt.wikimedia.org/wikimedia/pool/main/c/cumin/

This may help with dependencies?

I was told api should be compatible with cumin 3, so no changes should be needed on our side.

Thank you for the information, and yes, it was helpful :-)

Change 602754 merged by Jcrespo:
[operations/software/transferpy@master] transferpy: Package transferpy

https://gerrit.wikimedia.org/r/602754

Privacybatm renamed this task from Package transferpy framework under wmfmariadbpy to Package transferpy framework.Jun 24 2020, 9:12 AM

Change 607456 had a related patch set uploaded (by Privacybatm; owner: Privacybatm):
[operations/software/transferpy@master] transferpy: Add reference to deb package in the documentation

https://gerrit.wikimedia.org/r/607456

Change 607456 merged by Jcrespo:
[operations/software/transferpy@master] transferpy: Add reference to deb package in the documentation

https://gerrit.wikimedia.org/r/607456

Merged the packaging patch!

Congrats on a good job!

Let's add to the TODO list to add a proper man (1) page to the package (we could generate it -maybe- from the help).

Change 598984 restored by Privacybatm:
[operations/software/wmfmariadbpy@master] transferpy: Package transferpy

https://gerrit.wikimedia.org/r/598984

Change 598984 abandoned by Privacybatm:
[operations/software/wmfmariadbpy@master] transferpy: Package transferpy

Reason:
This has been already merged using another patch.

https://gerrit.wikimedia.org/r/598984

Change 618756 had a related patch set uploaded (by Hashar; owner: Hashar):
[integration/config@master] Add debian-glue to operations/software/transferpy

https://gerrit.wikimedia.org/r/618756

Change 618756 merged by jenkins-bot:
[integration/config@master] Add debian-glue to operations/software/transferpy

https://gerrit.wikimedia.org/r/618756

Change 618959 had a related patch set uploaded (by Hashar; owner: Hashar):
[operations/software/transferpy@master] Dummy change to test CI

https://gerrit.wikimedia.org/r/618959

Change 618959 abandoned by Hashar:
[operations/software/transferpy@master] Dummy change to test CI

Reason:
Do not merge. That is merely to trigger CI.

It fails cause the distribution "wikimedia" does not exist. Should be stretch / buster / unstable ;)

https://gerrit.wikimedia.org/r/618959