Bug 28028 - mageiasync should close rsync upon exit
Summary: mageiasync should close rsync upon exit
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: 7
Hardware: All Linux
Priority: Normal major
Target Milestone: ---
Assignee: QA Team
QA Contact:
URL:
Whiteboard: MGA7-64-OK
Keywords: Junior_job, advisory, validated_update
Depends on:
Blocks:
 
Reported: 2021-01-06 17:38 CET by Morgan Leijström
Modified: 2021-01-14 16:14 CET (History)
5 users (show)

See Also:
Source RPM: mageiasync-0.4.0-1.mga7.src.rpm
CVE:
Status comment:

Attachments
Add an attachment (proposed patch, testcase, etc.)

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
Note You need to log in before you can comment on or make changes to this bug.