Project

General

Custom queries

Profile

Actions

Feature #3034

open

TFTP file download should be synchronous and handle errors

Added by Dominic Cleal over 11 years ago. Updated over 6 years ago.

Status:
New
Priority:
Normal
Assignee:
-
Category:
TFTP
Target version:
-
Difficulty:
Triaged:
Yes
Fixed in Releases:
Found in Releases:

Description

The TFTP file download feature currently spawns wget in the background, so offers no tracking of progress or download errors.

It should perform downloads as a synchronous operation, which will allow us to report success or failure back to Foreman when provisioning hosts. Async downloads also mean that Foreman often boots VMs up too quickly, before the boot files have downloaded.

wget has another issue under failure, as it leaves a zero byte file in place which means we think downloads were a success.

Given the proxy's usually deployed on WEBrick, a synchronous download could tie up its resources, making it inaccessible to others though.


Related issues 7 (2 open5 closed)

Related to Foreman - Refactor #5121: Download kernel and image during OS creation rather than prior buildNew04/09/2014Actions
Related to Smart Proxy - Bug #6289: vmlinuz/initrd files corrupted during when multiple hosts created simultaneouslyClosed06/19/2014Actions
Related to Smart Proxy - Bug #16674: when downloading boot images, a failure does not fail the operation in the UIDuplicate09/23/2016Actions
Related to Smart Proxy - Bug #5069: TFTP automatic boot file download does not track updatesNew04/03/2014Actions
Related to Foreman - Feature #19389: Change TFTP filename pattern to include unique installation media IDClosedShimon ShteinActions
Has duplicate Smart Proxy - Bug #11067: Failing TFTP fetch filed doesn't trigger error in UIDuplicate07/09/2015Actions
Has duplicate Foreman - Bug #16609: race condition when using pxe icw virtual HW and a new OSDuplicate09/20/2016Actions
Actions #2

Updated by Dominic Cleal almost 11 years ago

  • Bugzilla link set to https://bugzilla.redhat.com/show_bug.cgi?id=1110378

edit: moved to #6289

Actions #5

Updated by Lukas Zapletal over 10 years ago

For the record, RHOSP team hit this. We might want to speak about bumping up the priority of this.

Actions #11

Updated by Lukas Zapletal over 8 years ago

When touching this code, consider downloading the files under different (temporary) names so we shorten the window when different host (not the one being blocked by the download) sees incorrect file (e.g. during upgrading to newer version).

Also I would like to keep the current (asynchronous) behavior in some way so it can be used when we implement #5121

I am currently working on an issue when file was downloaded incorrectly (error during transmission) and wget is not able to fix this anymore since it only compares timestamps. If you can take this into consideration when designing the new way of downloading.

Actions

Also available in: Atom PDF