Michael Calmer wrote:
% Hi,
% 
% we had here some trouble with the proxy and installing hughe RPMs. In the end 
% we found the OOM killer working because the proxy load the complete RPM into 
% memory during delivery.
% 
% So I created two patches for the proxy:
% 
% 0001: Is when sending content to the server via the proxy. Typically this way 
% not much data are transfered, but it can happen. So I removed the "body" 
% parameter from several functions and use "self.req" instead.
% 
% 0002: This is when receiving data from the Server and deliver it to the 
% client. I used the SmartIO class to create a temp file when copying the data 
% from the response into the request. A direct usage of the FD was not 
possible. 
% It seems, that the response FD is already removed when the request function 
% try to read from it. So we need a temp file.
% Note: SmartIO uses /tmp by default to create the tempfile. New systems 
% sometimes uses tmpfs on /tmp (Fedora?). So it might be a good idea to change 
% the default to a place without tmpfs.

Hi Michael,

Thanks for the fix. I reviewed it and commit.

Regards,

--
Michael Mráka
Satellite Engineering, Red Hat

_______________________________________________
Spacewalk-devel mailing list
Spacewalk-devel@redhat.com
https://www.redhat.com/mailman/listinfo/spacewalk-devel

Reply via email to