Bug 28028

Summary:	mageiasync should close rsync upon exit
Product:	Mageia	Reporter:	Morgan Leijström <fri>
Component:	RPM Packages	Assignee:	QA Team <qa-bugs>
Status:	RESOLVED FIXED	QA Contact:
Severity:	major
Priority:	Normal	CC:	geiger.david68210, mageia, ouaurelien, sysadmin-bugs, yvesbrungard
Version:	7	Keywords:	Junior_job, advisory, validated_update
Target Milestone:	---
Hardware:	All
OS:	Linux
Whiteboard:	MGA7-64-OK
Source RPM:	mageiasync-0.4.0-1.mga7.src.rpm	CVE:
Status comment:

Description Morgan Leijström 2021-01-06 17:38:03 CET

Description of problem:
If you press Quit when it started syncing, mageiasync exits but leave rsync process working downloading gigabytes...

Steps to Reproduce:
1. Start syncing
2. When it is syncing the .iso file press Quit
3. See a hidden file in target directory growing, prest Alt+esc to see two rsync processes still running

Comment 1 Aurelien Oudelet 2021-01-07 23:22:19 CET

Hi, thanks for reporting this bug.
Assigned to the package maintainer.

(Please set the status to 'assigned' if you are working on it)

Severity: normal => major
Assignee: bugsquad => mageiatools

Nicolas Lécureuil 2021-01-10 12:24:42 CET

CC: (none) => mageia
Keywords: (none) => Junior_job

Comment 2 Nicolas Lécureuil 2021-01-10 14:51:23 CET

fixed by papoteur in commit: 
http://gitweb.mageia.org/qa/MageiaSync/commit/?id=5530fc19b0c10d01ebdf5f4fceef0e1ca33c16fa

Comment 3 papoteur 2021-01-10 18:54:00 CET

Release 0.4.1 is now published.

CC: (none) => yves.brungard_mageia

Comment 4 Morgan Leijström 2021-01-10 22:29:40 CET

OK thanks

I actually set this bug against mga7 but it got pushed on Cauldron...
I think most of us testers dont store the ISOs on the mga8 testing machines.

Comment 5 David GEIGER 2021-01-12 07:11:32 CET

An update is now done also for mga7 in Core/Updates_testing repo!

CC: (none) => geiger.david68210

Comment 6 papoteur 2021-01-12 07:48:23 CET

Packages:
mageiasync-0.4.1-1.mga7.rpm
mageiasync-0.4.1-1.mga7.src.rpm

Advisory
=======================
mageiasync-0.4.1
Now stop rsync when quitting.
Calculate and check only SHA3-512 sum.
Add check for destination directory.
========================

Comment 7 David GEIGER 2021-01-12 08:03:12 CET

Assigning to QA.

Assignee: mageiatools => qa-bugs

Comment 8 Morgan Leijström 2021-01-12 11:00:35 CET

Test OK mga7-64

Whiteboard: (none) => MGA7-64-OK

Comment 9 Aurelien Oudelet 2021-01-12 14:18:26 CET

Validating,
Advisory pushed to SVN.

Keywords: (none) => advisory, validated_update
CC: (none) => ouaurelien, sysadmin-bugs

Comment 10 Mageia Robot 2021-01-14 16:14:46 CET

An update for this issue has been pushed to the Mageia Updates repository.

https://advisories.mageia.org/MGAA-2021-0008.html

Resolution: (none) => FIXED
Status: NEW => RESOLVED