URL: https://github.com/freeipa/freeipa/pull/210
Title: #210: Tests: Stage User Tracker implementation
MartinBasti commented:
"""
Fixed upstream
master:
https://fedorahosted.org/freeipa/changeset/a336de630e9d1ef95a507cc3ee9200c001ab9193
URL: https://github.com/freeipa/freeipa/pull/210
Title: #210: Tests: Stage User Tracker implementation
martbab commented:
"""
@gkaihorodova the PR cannot be pushed in current form because the first commit
298e1a136c6a430e8deaa558a946ba51874ffd95 is already pushed to master.
So to rebase it
URL: https://github.com/freeipa/freeipa/pull/210
Title: #210: Tests: Stage User Tracker implementation
mbasti-rh commented:
"""
Needs rebase
```
Applying: Unaccessible variable self.attrs in Tracker
Patch failed at 0001 Unaccessible variable self.attrs in Tracker
The copy of the patch that
URL: https://github.com/freeipa/freeipa/pull/210
Title: #210: Tests: Stage User Tracker implementation
stlaz commented:
"""
I just wanted to remove the ACK till @martbab's comment is worked in so nobody
pushes it but I found some minor issues that I would like to see fixed in the
rebase as
URL: https://github.com/freeipa/freeipa/pull/210
Title: #210: Tests: Stage User Tracker implementation
martbab commented:
"""
Can we get this PR rebased against current master to see whether Travis CI is
okay? The current checks are polluted by regression that should be fixed.
"""
See the
URL: https://github.com/freeipa/freeipa/pull/210
Title: #210: Tests: Stage User Tracker implementation
martbab commented:
"""
Can we get this PR rebased against current master to see whether Travis CI is
okay. The current checks are polluted by regression that should be fixed.
"""
See the
URL: https://github.com/freeipa/freeipa/pull/210
Title: #210: Tests: Stage User Tracker implementation
martbab commented:
"""
Bump for review.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/210#issuecomment-266418295
--
Manage your subscription for the Freeipa-devel
URL: https://github.com/freeipa/freeipa/pull/210
Title: #210: Tests: Stage User Tracker implementation
mirielka commented:
"""
Review notes: same as in https://github.com/freeipa/freeipa/pull/181
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/210#issuecomment-260255679
--
