<p dir="ltr"><br>
On Jun 4, 2015 5:35 AM, &quot;Mathieu Bridon&quot; &lt;<a href="mailto:bochecha@fedoraproject.org">bochecha@fedoraproject.org</a>&gt; wrote:<br>
&gt;<br>
&gt; On Fri, 2015-05-29 at 11:56 -0700, Toshio Kuratomi wrote:<br>
&gt; &gt; On Fri, May 29, 2015 at 12:43:32PM +0200, Mathieu Bridon wrote:<br>
&gt; &gt; &gt; If everybody agrees this is the desired behaviour, can we deploy it<br>
&gt; &gt; &gt; in production?<br>
&gt;<br>
&gt; This had been deployed in production, which took me a bit by surprise<br>
&gt; as this discussion didn&#39;t seem to have been finished.<br>
&gt;<br>
&gt; It caused a bit of a stir due to SELinux (it was disabled in staging<br>
&gt; when I tested those changes), but we fixed it with Pierre-Yves:<br>
&gt;<br>
&gt;     <a href="https://fedorahosted.org/rel-eng/ticket/6191">https://fedorahosted.org/rel-eng/ticket/6191</a><br>
&gt;<br>
&gt; &gt; Can we avoid the reupload?  Hopefully it won&#39;t hit too many people<br>
&gt; &gt; (because they&#39;ll remember when they have a tarball already uploaded)<br>
&gt; &gt; but it can be painful to wait for something to upload again if it&#39;s<br>
&gt; &gt; not needed.<br>
&gt;<br>
&gt; The same script handles both:<br>
&gt;<br>
&gt; * checking if a file exists<br>
&gt; * uploading a file<br>
&gt;<br>
&gt; When it&#39;s called with the upload, we can&#39;t do much in the script to<br>
&gt; avoid a reupload, it&#39;s just going to be reuploaded, and as such the<br>
&gt; behaviour I described in my previous email will ensure we end up with<br>
&gt; the two copies linked to each other.<br>
&gt;<br>
&gt; However, we could avoid the reupload in normal cases (i.e &quot;fedpkg<br>
&gt; upload&quot; and &quot;fedpkg new-sources&quot;) because pyrpkg always checks if the<br>
&gt; file exists (calling the CGI script) before uploading it.<br>
&gt;<br>
&gt; So we could make the &quot;check&quot; portion of the code verify if the file is<br>
&gt; present either in the old or new location, and if it&#39;s found only in<br>
&gt; the old one, symlink it to the new one.<br>
&gt;<br>
&gt; If that makes sense, I&#39;ll cook up a patch (and test it with SELinux<br>
&gt; enforcing in staging, this time ;) )<br>
&gt;<br>
Yep, that sounds like a good plan!</p>
<p dir="ltr">-Toshio</p>