<div dir="ltr">Updated patch attached.<br><div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 13, 2015 at 10:59 AM, Martin Basti <span dir="ltr"><<a href="mailto:mbasti@redhat.com" target="_blank">mbasti@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  
    
  
  <div text="#000000" bgcolor="#FFFFFF"><span class="">
    <br>
    <br>
    <div>On 13.10.2015 18:53, Gabe Alford wrote:<br>
    </div>
    <blockquote type="cite">
      <div dir="ltr">
        <div>
          <div>Thanks Martin,<br>
            <br>
          </div>
          What about adding no_create and no_update flags? <br>
          <br>
        </div>
        <div>Gabe<br>
        </div>
      </div>
    </blockquote></span>
    Yes, that may work, also please increment minor version of API and
    add ticket into commit message (<a href="https://fedorahosted.org/freeipa/ticket/5366" target="_blank">https://fedorahosted.org/freeipa/ticket/5366)</a><br>
    <br>
    Thanks.<span class="HOEnZb"><font color="#888888"><br>
    Martin</font></span><span class=""><br>
    <blockquote type="cite">
      <div class="gmail_extra"><br>
        <div class="gmail_quote">On Tue, Oct 13, 2015 at 9:54 AM, Martin
          Basti <span dir="ltr"><<a href="mailto:mbasti@redhat.com" target="_blank">mbasti@redhat.com</a>></span>
          wrote:<br>
          <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
            <div text="#000000" bgcolor="#FFFFFF">
              <div>
                <div> <br>
                  <br>
                  <div>On 09.10.2015 19:17, Gabe Alford wrote:<br>
                  </div>
                  <blockquote type="cite">
                    <div dir="ltr">
                      <div>
                        <div>
                          <div>Hello,<br>
                            <br>
                          </div>
                          This patch enables nsaccountlock in user.py
                          cli. It is very handy to be able to search and
                          find users with disabled/enabled accounts,
                          etc. That said, I couldn't find why it was
                          no_option in the first place, so I am not 100%
                          sure if it breaks something or the reasoning
                          behind no_option.<br>
                        </div>
                        <div><br>
                        </div>
                        Thanks,<br>
                        <br>
                      </div>
                      Gabe<br>
                    </div>
                    <br>
                    <fieldset></fieldset>
                    <br>
                  </blockquote>
                </div>
              </div>
              Hello,<br>
              <br>
              <a href="https://fedorahosted.org/freeipa/ticket/5366" target="_blank">https://fedorahosted.org/freeipa/ticket/5366</a><br>
              <br>
              This patch allows to enable/disable user via user-mod, and
              we do not want to do this, so NACK for this patch.<br>
              I'm not sure yet how to write it in elegant way.<span><font color="#888888"><br>
                  <br>
                  Martin.<br>
                </font></span></div>
          </blockquote>
        </div>
        <br>
      </div>
    </blockquote>
    <br>
  </span></div>

</blockquote></div><br></div></div></div>