[Freeipa-devel] [freeipa PR#860][comment] adtrustinstance: fix ID range comparison

2017-06-15 Thread MartinBasti via FreeIPA-devel
  URL: https://github.com/freeipa/freeipa/pull/860
Title: #860: adtrustinstance: fix ID range comparison

MartinBasti commented:
"""
master:

* 440c61dc40353833cad3a5fc509821ce1f23757f adtrustinstance: fix ID range 
comparison


"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/860#issuecomment-308708256
___
FreeIPA-devel mailing list -- freeipa-devel@lists.fedorahosted.org
To unsubscribe send an email to freeipa-devel-le...@lists.fedorahosted.org


[Freeipa-devel] [freeipa PR#860][comment] adtrustinstance: fix ID range comparison

2017-06-07 Thread abbra via FreeIPA-devel
  URL: https://github.com/freeipa/freeipa/pull/860
Title: #860: adtrustinstance: fix ID range comparison

abbra commented:
"""
ipaIDRangeSize is used by `sidgen` and `range-check` plugins in 389-ds and also 
by SSSD. It cannot be None. In fact, it has to be more than 0 (and less than 
UINT32_MAX) or otherwise it is considered wrong.
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/860#issuecomment-306795021
___
FreeIPA-devel mailing list -- freeipa-devel@lists.fedorahosted.org
To unsubscribe send an email to freeipa-devel-le...@lists.fedorahosted.org


[Freeipa-devel] [freeipa PR#860][comment] adtrustinstance: fix ID range comparison

2017-06-07 Thread stlaz via FreeIPA-devel
  URL: https://github.com/freeipa/freeipa/pull/860
Title: #860: adtrustinstance: fix ID range comparison

stlaz commented:
"""
No, because I have no idea what that does. @abbra might know more.
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/860#issuecomment-306792693
___
FreeIPA-devel mailing list -- freeipa-devel@lists.fedorahosted.org
To unsubscribe send an email to freeipa-devel-le...@lists.fedorahosted.org


[Freeipa-devel] [freeipa PR#860][comment] adtrustinstance: fix ID range comparison

2017-06-07 Thread MartinBasti via FreeIPA-devel
  URL: https://github.com/freeipa/freeipa/pull/860
Title: #860: adtrustinstance: fix ID range comparison

MartinBasti commented:
"""
@stlaz could you provide more evidence? :-)
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/860#issuecomment-306780029
___
FreeIPA-devel mailing list -- freeipa-devel@lists.fedorahosted.org
To unsubscribe send an email to freeipa-devel-le...@lists.fedorahosted.org


[Freeipa-devel] [freeipa PR#860][comment] adtrustinstance: fix ID range comparison

2017-06-07 Thread stlaz via FreeIPA-devel
  URL: https://github.com/freeipa/freeipa/pull/860
Title: #860: adtrustinstance: fix ID range comparison

stlaz commented:
"""
I don't think it is.
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/860#issuecomment-306779785
___
FreeIPA-devel mailing list -- freeipa-devel@lists.fedorahosted.org
To unsubscribe send an email to freeipa-devel-le...@lists.fedorahosted.org


[Freeipa-devel] [freeipa PR#860][comment] adtrustinstance: fix ID range comparison

2017-06-07 Thread MartinBasti via FreeIPA-devel
  URL: https://github.com/freeipa/freeipa/pull/860
Title: #860: adtrustinstance: fix ID range comparison

MartinBasti commented:
"""
Is 'None' a valid value for `ipaIDRangeSize` ?
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/860#issuecomment-306779616
___
FreeIPA-devel mailing list -- freeipa-devel@lists.fedorahosted.org
To unsubscribe send an email to freeipa-devel-le...@lists.fedorahosted.org