Page MenuHomePhabricator

Use logging package instead of print statements in transferpy package
Closed, ResolvedPublic

Description

The transferpy is a package used for database backup and recovery. Currently, it uses print statements for logging purposes. It will be useful if we use the logging python package instead.

Event Timeline

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

Change 607028 had a related patch set uploaded (by Privacybatm; owner: Privacybatm):
[operations/software/transferpy@master] transferpy: Use logging package instead of print statements

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

I don't think we will be able to incorporate data transfer progress information. The RemoteExecution works in such a way that it will send the full data as a whole without any communication to the main program. Since the netcat command runs in the remote machine, the framework running machine has no information about it! What do you think?

Change 607028 merged by jenkins-bot:
[operations/software/transferpy@master] transferpy: Use logging package instead of print statements

https://gerrit.wikimedia.org/r/c/operations/software/transferpy/ /607028

Most of this is done, but let's keep this open, even with lower priority, to see if we can add some extra logging at a later time from new features.