URL: https://github.com/SSSD/sssd/pull/5600 Author: DeepakDas7 Title: #5600: SSSD Log: log_bad_address_msg_modification Action: opened
PR body: """ Improve Log Containing Bad Address string
Resolves: https://github.com/SSSD/sssd/issues/5577 """
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5600/head:pr5600 git checkout pr5600
URL: https://github.com/SSSD/sssd/pull/5600 Author: DeepakDas7 Title: #5600: SSSD Log: log_bad_address_msg_modification Action: synchronized
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5600/head:pr5600 git checkout pr5600
URL: https://github.com/SSSD/sssd/pull/5600 Author: DeepakDas7 Title: #5600: SSSD Log: log_bad_address_msg_modification Action: synchronized
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5600/head:pr5600 git checkout pr5600
URL: https://github.com/SSSD/sssd/pull/5600 Title: #5600: SSSD Log: log_bad_address_msg_modification
Label: +Waiting for review
URL: https://github.com/SSSD/sssd/pull/5600 Title: #5600: SSSD Log: log_bad_address_msg_modification
Label: +Bugzilla
URL: https://github.com/SSSD/sssd/pull/5600 Title: #5600: SSSD Log: log_bad_address_msg_modification
ikerexxe commented: """ @elkoniu can you take a look at it? """
See the full comment at https://github.com/SSSD/sssd/pull/5600#issuecomment-828330624
URL: https://github.com/SSSD/sssd/pull/5600 Author: DeepakDas7 Title: #5600: SSSD Log: log_bad_address_msg_modification Action: synchronized
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5600/head:pr5600 git checkout pr5600
URL: https://github.com/SSSD/sssd/pull/5600 Title: #5600: SSSD Log: log_bad_address_msg_modification
elkoniu commented: """ @ikerexxe Sorry for the delay, have this PR on my review list for this week. """
See the full comment at https://github.com/SSSD/sssd/pull/5600#issuecomment-831876660
URL: https://github.com/SSSD/sssd/pull/5600 Title: #5600: SSSD Log: log_bad_address_msg_modification
Label: -Waiting for review
URL: https://github.com/SSSD/sssd/pull/5600 Title: #5600: SSSD Log: log_bad_address_msg_modification
Label: +Changes requested
URL: https://github.com/SSSD/sssd/pull/5600 Title: #5600: SSSD Log: log_bad_address_msg_modification
elkoniu commented: """ I think that when error occurs we should check if we have access to server name and print it always if available rather than making this decision on top of `ret` variable value (`result == EFAULT || result == EIO || result == EPERM)`). """
See the full comment at https://github.com/SSSD/sssd/pull/5600#issuecomment-831987182
URL: https://github.com/SSSD/sssd/pull/5600 Author: DeepakDas7 Title: #5600: SSSD Log: log_bad_address_msg_modification Action: synchronized
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5600/head:pr5600 git checkout pr5600
URL: https://github.com/SSSD/sssd/pull/5600 Title: #5600: SSSD Log: log_bad_address_msg_modification
alexey-tikhonov commented: """ It seems those patches break at least one of existing "multihost" tests: https://s3.eu-central-1.amazonaws.com/sssd-ci/PR-5600/6/fedora33/multihost-p...
I don't say patches are wrong, but at least tests should be updated as well. """
See the full comment at https://github.com/SSSD/sssd/pull/5600#issuecomment-836555516
URL: https://github.com/SSSD/sssd/pull/5600 Author: DeepakDas7 Title: #5600: [WIP] SSSD Log: log_bad_address_msg_modification Action: edited
Changed field: title Original value: """ SSSD Log: log_bad_address_msg_modification """
URL: https://github.com/SSSD/sssd/pull/5600 Title: #5600: [WIP] SSSD Log: log_bad_address_msg_modification
elkoniu commented: """ @DeepakDas7 as we discussed on IRC ping me any time this PR will be ready for review :) """
See the full comment at https://github.com/SSSD/sssd/pull/5600#issuecomment-842707488
URL: https://github.com/SSSD/sssd/pull/5600 Author: DeepakDas7 Title: #5600: [WIP] SSSD Log: log_bad_address_msg_modification Action: synchronized
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5600/head:pr5600 git checkout pr5600
URL: https://github.com/SSSD/sssd/pull/5600 Author: DeepakDas7 Title: #5600: [WIP] SSSD Log: log_bad_address_msg_modification Action: synchronized
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5600/head:pr5600 git checkout pr5600
URL: https://github.com/SSSD/sssd/pull/5600 Author: DeepakDas7 Title: #5600: [WIP] SSSD Log: log_bad_address_msg_modification Action: synchronized
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5600/head:pr5600 git checkout pr5600
URL: https://github.com/SSSD/sssd/pull/5600 Author: DeepakDas7 Title: #5600: [WIP] SSSD Log: log_bad_address_msg_modification Action: synchronized
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5600/head:pr5600 git checkout pr5600
URL: https://github.com/SSSD/sssd/pull/5600 Title: #5600: [WIP] SSSD Log: log_bad_address_msg_modification
Label: +Waiting for review
URL: https://github.com/SSSD/sssd/pull/5600 Title: #5600: [WIP] SSSD Log: log_bad_address_msg_modification
Label: -Changes requested
URL: https://github.com/SSSD/sssd/pull/5600 Title: #5600: [WIP] SSSD Log: log_bad_address_msg_modification
alexey-tikhonov commented: """ Hi @DeepakDas7, why is this still [WIP]? """
See the full comment at https://github.com/SSSD/sssd/pull/5600#issuecomment-861612924
URL: https://github.com/SSSD/sssd/pull/5600 Author: DeepakDas7 Title: #5600: SSSD Log: log_bad_address_msg_modification Action: edited
Changed field: title Original value: """ [WIP] SSSD Log: log_bad_address_msg_modification """
URL: https://github.com/SSSD/sssd/pull/5600 Author: DeepakDas7 Title: #5600: SSSD Log: log_bad_address_msg_modification Action: synchronized
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5600/head:pr5600 git checkout pr5600
URL: https://github.com/SSSD/sssd/pull/5600 Author: DeepakDas7 Title: #5600: SSSD Log: log_bad_address_msg_modification Action: synchronized
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5600/head:pr5600 git checkout pr5600
URL: https://github.com/SSSD/sssd/pull/5600 Title: #5600: SSSD Log: log_bad_address_msg_modification
Label: -Waiting for review
URL: https://github.com/SSSD/sssd/pull/5600 Title: #5600: SSSD Log: log_bad_address_msg_modification
Label: +Accepted
URL: https://github.com/SSSD/sssd/pull/5600 Title: #5600: SSSD Log: log_bad_address_msg_modification
ikerexxe commented: """ I've reviewed the tests that are failing and there seems to be a timeout error that seems to be related with the infrastructure. @elkoniu do you think we should relaunch the tests? Or are you comfortable with the actual results? """
See the full comment at https://github.com/SSSD/sssd/pull/5600#issuecomment-865648943
URL: https://github.com/SSSD/sssd/pull/5600 Title: #5600: SSSD Log: log_bad_address_msg_modification
elkoniu commented: """ @ikerexxe I just triggered test run again, lets wait for the results and decide then. """
See the full comment at https://github.com/SSSD/sssd/pull/5600#issuecomment-865934826
URL: https://github.com/SSSD/sssd/pull/5600 Title: #5600: SSSD Log: log_bad_address_msg_modification
elkoniu commented: """ @DeepakDas7 can you rebase this to latest master? """
See the full comment at https://github.com/SSSD/sssd/pull/5600#issuecomment-866836691
URL: https://github.com/SSSD/sssd/pull/5600 Author: DeepakDas7 Title: #5600: SSSD Log: log_bad_address_msg_modification Action: synchronized
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5600/head:pr5600 git checkout pr5600
URL: https://github.com/SSSD/sssd/pull/5600 Title: #5600: SSSD Log: log_bad_address_msg_modification
elkoniu commented: """ @pbrezina It looks like this is the merge candidate. """
See the full comment at https://github.com/SSSD/sssd/pull/5600#issuecomment-868017752
URL: https://github.com/SSSD/sssd/pull/5600 Title: #5600: SSSD Log: log_bad_address_msg_modification
Label: +Ready to push
URL: https://github.com/SSSD/sssd/pull/5600 Title: #5600: SSSD Log: log_bad_address_msg_modification
pbrezina commented: """ Pushed PR: https://github.com/SSSD/sssd/pull/5600
* `master` * 7646ac958da5d0f3997c7655e39f7a2f43d6ec1f - SSSD Log: log_bad_address_msg_mod
"""
See the full comment at https://github.com/SSSD/sssd/pull/5600#issuecomment-876286351
URL: https://github.com/SSSD/sssd/pull/5600 Title: #5600: SSSD Log: log_bad_address_msg_modification
Label: +Pushed
URL: https://github.com/SSSD/sssd/pull/5600 Title: #5600: SSSD Log: log_bad_address_msg_modification
Label: -Accepted
URL: https://github.com/SSSD/sssd/pull/5600 Title: #5600: SSSD Log: log_bad_address_msg_modification
Label: -Ready to push
URL: https://github.com/SSSD/sssd/pull/5600 Author: DeepakDas7 Title: #5600: SSSD Log: log_bad_address_msg_modification Action: closed
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5600/head:pr5600 git checkout pr5600
sssd-devel@lists.fedorahosted.org