<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
  <head>
    <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#ffffff" text="#000000">
    On 07/27/2011 12:00 PM, Martin Kosek wrote:
    <blockquote
      cite="mid:1311782411.12277.24.camel@dhcp-25-52.brq.redhat.com"
      type="cite">
      <pre wrap="">On Wed, 2011-07-27 at 10:41 -0400, Rob Crittenden wrote:
</pre>
      <blockquote type="cite">
        <pre wrap="">Martin Kosek wrote:
</pre>
        <blockquote type="cite">
          <pre wrap="">Fix automountkey-mod so that automountkey attribute is correctly
updated. Add this test case to the unit tests.

<a class="moz-txt-link-freetext" href="https://fedorahosted.org/freeipa/ticket/1528">https://fedorahosted.org/freeipa/ticket/1528</a>
</pre>
        </blockquote>
        <pre wrap="">
It fixes the problem but I've found another: --key isn't required so if 
you don't pass it in then a backtrace will occur:

Traceback (most recent call last):
   File "/home/rcrit/redhat/freeipa-master/ipaserver/rpcserver.py", line 
220, in wsgi_execute
     result = self.Command[name](*args, **options)
   File "/home/rcrit/redhat/freeipa-master/ipalib/frontend.py", line 
425, in __call__
     ret = self.run(*args, **options)
   File "/home/rcrit/redhat/freeipa-master/ipalib/frontend.py", line 
731, in run
     return self.execute(*args, **options)
   File "/home/rcrit/redhat/freeipa-master/ipalib/plugins/automount.py", 
line 873, in execute
     keys += (self.obj.get_pk(options['automountkey'],
KeyError: 'automountkey'

Also, automountinformation is already required. This may be a leftover 
from when we used it in description, this can probably be lifted too.

rob
</pre>
      </blockquote>
      <pre wrap="">
Good catch. I fixed this bug too and I also made --newinfo optional so
that automountkey may be just renamed without changing its info
attribute.

I didn't bump up API VERSION as these are either compatible changes or
they caused server internal error.

Martin
</pre>
    </blockquote>
    <br>
    Should the ticket be moved into 2.1 July sprint then?<br>
    <br>
    <blockquote
      cite="mid:1311782411.12277.24.camel@dhcp-25-52.brq.redhat.com"
      type="cite">
      <pre wrap="">
<fieldset class="mimeAttachmentHeader"></fieldset>
_______________________________________________
Freeipa-devel mailing list
<a class="moz-txt-link-abbreviated" href="mailto:Freeipa-devel@redhat.com">Freeipa-devel@redhat.com</a>
<a class="moz-txt-link-freetext" href="https://www.redhat.com/mailman/listinfo/freeipa-devel">https://www.redhat.com/mailman/listinfo/freeipa-devel</a></pre>
    </blockquote>
    <br>
    <br>
    <pre class="moz-signature" cols="72">-- 
Thank you,
Dmitri Pal

Sr. Engineering Manager IPA project,
Red Hat Inc.


-------------------------------
Looking to carve out IT costs?
<a class="moz-txt-link-abbreviated" href="http://www.redhat.com/carveoutcosts/">www.redhat.com/carveoutcosts/</a>


</pre>
  </body>
</html>