On Apr 7, 2016, at 3:27 AM, Lukas Slebodnik
<lslebodn(a)redhat.com> wrote:
> On (06/04/16 15:38), Jakub Hrozek wrote:
>> On Wed, Apr 06, 2016 at 03:16:20PM +0200, Jakub Hrozek wrote:
>>> On Wed, Apr 06, 2016 at 08:39:39AM -0400, Stephen Gallagher wrote:
>>>
>>>
>>>>> On Apr 6, 2016, at 8:37 AM, Jakub Hrozek <jhrozek(a)redhat.com>
wrote:
>>>>>
>>>>> On Tue, Apr 05, 2016 at 02:34:33PM -0400, Stephen Gallagher wrote:
>>>>> We only need to go online if we receive a netlink signal that might
>>>>> indicate that the external connection might have become available.
This
>>>>> will never be true for link-local addresses.
>>>>>
>>>>
>>>> The indentation of DEBUG messages is a bit off on two of places. I fixed
>>>> them locally, if you agree, I would like to push:
>>>>
https://github.com/jhrozek/sssd/commit/818eada2a68b6c9cff9eb3285ff6126ba4...
>>>>
>>>> (Just the intentation changed, nothing else..)
>>>
>>>
>>> Fine by me.
>>
>> OK, ACK
>>
>> Thank you for the patch and the investigation!
>
> CI:
http://sssd-ci.duckdns.org/logs/job/40/75/summary.html
>
> btw I tested by bringing up and down the loopback interface and making
> sure that addresses like ::1 are filtered out.
>
> * master: a9d1b4b61b614a954c784f224b8fe7a47b6dd206
Stephen,
I think you would like to have this patch in fedora 24 :-)
I can backport this patch just to fedora if you file a BZ
or we can backport this patch to 1.13 branch
and it will be in fedora with next release (1.13.4)
which should be next week.
What do you prefer?
LS
Next week is fine. I'm carrying the patch locally on my machine and I haven't
heard any other complaints.