[SSSD] [sssd PR#32][comment] Requesting a pull to SSSD:master from fidencio:wip/#3138

2017-07-27 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/32 Title: #32: Requesting a pull to SSSD:master from fidencio:wip/#3138 fidencio commented: """ This PR doesn't make sense anymore in the state that this branch is. I'm closing the PR and a new one will be re-open by whoever ends up working on this in the

[SSSD] [sssd PR#32][comment] Requesting a pull to SSSD:master from fidencio:wip/#3138

2016-10-03 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/32 Title: #32: Requesting a pull to SSSD:master from fidencio:wip/#3138 jhrozek commented: """ Yes, as discussed, moving this functionality to the monitor is something we'll go with. Therefore I'm adding the Changes requested label. """ See the full

[SSSD] [sssd PR#32][comment] Requesting a pull to SSSD:master from fidencio:wip/#3138

2016-09-28 Thread simo5
URL: https://github.com/SSSD/sssd/pull/32 Title: #32: Requesting a pull to SSSD:master from fidencio:wip/#3138 simo5 commented: """ The problem of using genconf is race conditions and the fact it won't be possible to call it again after the service is started. I am for adding socket activation

[SSSD] [sssd PR#32][comment] Requesting a pull to SSSD:master from fidencio:wip/#3138

2016-09-28 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/32 Title: #32: Requesting a pull to SSSD:master from fidencio:wip/#3138 fidencio commented: """ @jhrozek Answer for 1) I do believe it may work, but I would have to adapt `sssd --genconf` logic, in order to avoid race-conditions. Shall I go for it? I

[SSSD] [sssd PR#32][comment] Requesting a pull to SSSD:master from fidencio:wip/#3138

2016-09-28 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/32 Title: #32: Requesting a pull to SSSD:master from fidencio:wip/#3138 jhrozek commented: """ Hi, thank you for working on this. I think there are two "architectural" questions we should answer. 1) Lukas raised the suggestion of using PreExec with `sssd

[SSSD] [sssd PR#32][comment] Requesting a pull to SSSD:master from fidencio:wip/#3138

2016-09-23 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/32 Title: #32: Requesting a pull to SSSD:master from fidencio:wip/#3138 fidencio commented: """ Okay, with the updated version of the patches both issues that I had are fixed. Here is the link for the CI (that has passed):