<div dir="ltr">Should uncap patches be -W until next bandit release?<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Em ter, 14 de mai de 2019 às 17:26, Doug Hellmann <<a href="mailto:doug@doughellmann.com">doug@doughellmann.com</a>> escreveu:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Zane Bitter <<a href="mailto:zbitter@redhat.com" target="_blank">zbitter@redhat.com</a>> writes:<br>
<br>
> On 13/05/19 1:40 PM, Ben Nemec wrote:<br>
>> <br>
>> <br>
>> On 5/13/19 12:23 PM, Ben Nemec wrote:<br>
>>> Nefarious cap bandits are running amok in the OpenStack community! <br>
>>> Won't someone take a stand against these villainous headwear thieves?!<br>
>>><br>
>>> Oh, sorry, just pasted the elevator pitch for my new novel. ;-)<br>
>>><br>
>>> Actually, this email is to summarize the plan we came up with in the <br>
>>> Oslo meeting this morning. Since we have a bunch of projects affected <br>
>>> by the Bandit breakage I wanted to make sure we had a common fix so we <br>
>>> don't have a bunch of slightly different approaches in each project. <br>
>>> The plan we agreed on in the meeting was to push a two patch series to <br>
>>> each repo - one to cap bandit <1.6.0 and one to uncap it with a <br>
>>> !=1.6.0 exclusion. The first should be merged immediately to unblock <br>
>>> ci, and the latter can be rechecked once bandit 1.6.1 releases to <br>
>>> verify that it fixes the problem for us.<br>
><br>
> I take it that just blocking 1.6.0 in global-requirements isn't an <br>
> option? (Would it not work, or just break every project's requirements <br>
> job? I could live with the latter since they're broken anyway because of <br>
> the sphinx issue below...)<br>
<br>
Because bandit is a "linter" it is in the blacklist in the requirements<br>
repo, which means it is not constrained there. Projects are expected to<br>
manage the versions of linters they use, and roll forward when they are<br>
ready to deal with any new rules introduced by the linters (either by<br>
following or disabling them).<br>
<br>
So, no, unfortunately we can't do this globally through the requirements<br>
repo right now.<br>
<br>
-- <br>
Doug<br>
<br>
</blockquote></div><br clear="all"><br>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>
        <p style="font-weight:bold;margin:0px;padding:0px;font-size:14px;text-transform:capitalize;font-family:"RedHatText",sans-serif">
          <span>Moisés</span> <span>Guimarães</span><span style="color:rgb(170,170,170);margin:0px"></span>
        </p>
        
        <p style="font-weight:normal;font-size:12px;margin:0px;text-transform:capitalize;font-family:"RedHatText",sans-serif">
          <span>Software Engineer</span>
        </p>
        <p style="font-weight:normal;margin:0px 0px 4px;font-size:12px;font-family:"RedHatText",sans-serif">
          <a style="color:rgb(0,136,206);font-size:12px;margin:0px;text-decoration:none;font-family:"RedHatText",sans-serif" href="https://www.redhat.com" target="_blank">Red Hat <span></span></a>
        </p>
    <div style="margin-bottom:4px">
      
      
    </div>
    <p style="font-weight:normal;margin:0px;font-size:12px;font-family:"RedHatText",sans-serif">
      
      
      
    </p>
    
    

    <div style="margin-top:12px">
      <table border="0">
        <tbody><tr>
          <td width="100px"><a href="https://red.ht/sig" target="_blank"> <img src="https://static.redhat.com/libs/redhat/brand-assets/latest/corp/logo.png" width="90" height="auto"></a> </td>
          
        </tr>
      </tbody></table>
    </div>

  </div></div></div></div></div>