[Freeipa-devel] [freeipa PR#122][comment] Acceptance tests

2016-11-22 Thread martbab
  URL: https://github.com/freeipa/freeipa/pull/122
Title: #122: Acceptance tests

martbab commented:
"""
Fixed upstream
master:
https://fedorahosted.org/freeipa/changeset/3e53bbcc34bd256da36209fd8cf8ac5d33ec8093
https://fedorahosted.org/freeipa/changeset/4225484356426a73cc11211bceda7f06ee23d093
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/122#issuecomment-26350
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#122][comment] Acceptance tests

2016-10-04 Thread mbasti-rh
  URL: https://github.com/freeipa/freeipa/pull/122
Title: #122: Acceptance tests

mbasti-rh commented:
"""
Please see my inline comments, I would not bother with removing DNS tests from 
suite because they are fast and should not affect CA/KRA
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/122#issuecomment-251360383
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#122][comment] Acceptance tests

2016-09-27 Thread mbasti-rh
  URL: https://github.com/freeipa/freeipa/pull/122
Title: #122: Acceptance tests

mbasti-rh commented:
"""
I quite disagree with marks, please read my inline comments
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/122#issuecomment-249858212
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code