[Freeipa-devel] [freeipa PR#840][comment] Add Role 'Enrollment Administrator'

2017-06-09 Thread martbab via FreeIPA-devel
  URL: https://github.com/freeipa/freeipa/pull/840
Title: #840: Add Role 'Enrollment Administrator'

martbab commented:
"""
master:

* 468eb3c712140399ed2ec346ff4356bffd590e09 Add Role 'Enrollment Administrator'


"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/840#issuecomment-307407213
___
FreeIPA-devel mailing list -- freeipa-devel@lists.fedorahosted.org
To unsubscribe send an email to freeipa-devel-le...@lists.fedorahosted.org


[Freeipa-devel] [freeipa PR#840][comment] Add Role 'Enrollment Administrator'

2017-06-09 Thread abbra via FreeIPA-devel
  URL: https://github.com/freeipa/freeipa/pull/840
Title: #840: Add Role 'Enrollment Administrator'

abbra commented:
"""
Got it. Thanks.
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/840#issuecomment-307369453
___
FreeIPA-devel mailing list -- freeipa-devel@lists.fedorahosted.org
To unsubscribe send an email to freeipa-devel-le...@lists.fedorahosted.org


[Freeipa-devel] [freeipa PR#840][comment] Add Role 'Enrollment Administrator'

2017-06-09 Thread Tiboris via FreeIPA-devel
  URL: https://github.com/freeipa/freeipa/pull/840
Title: #840: Add Role 'Enrollment Administrator'

Tiboris commented:
"""
I think adding 'Host Enrollment' privilege to 'IT Specialists' is not necessary 
because the  'IT Specialists' role has already a 'Host Administrators' 
privilege assigned an this privilege contain all of the 'Host Enrollment' 
privilege permissions.
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/840#issuecomment-307360260
___
FreeIPA-devel mailing list -- freeipa-devel@lists.fedorahosted.org
To unsubscribe send an email to freeipa-devel-le...@lists.fedorahosted.org


[Freeipa-devel] [freeipa PR#840][comment] Add Role 'Enrollment Administrator'

2017-06-09 Thread Tiboris via FreeIPA-devel
  URL: https://github.com/freeipa/freeipa/pull/840
Title: #840: Add Role 'Enrollment Administrator'

Tiboris commented:
"""
I think adding 'Host Enrollment' privilege to 'IT Specialists' is not necessary 
because the  'IT Specialists' role has already a 'Host Administrators' 
privilege assigned an this privilege contain all of the 'Host Enrollment' 
privilege permissions.
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/840#issuecomment-307360260
___
FreeIPA-devel mailing list -- freeipa-devel@lists.fedorahosted.org
To unsubscribe send an email to freeipa-devel-le...@lists.fedorahosted.org


[Freeipa-devel] [freeipa PR#840][comment] Add Role 'Enrollment Administrator'

2017-06-09 Thread Tiboris via FreeIPA-devel
  URL: https://github.com/freeipa/freeipa/pull/840
Title: #840: Add Role 'Enrollment Administrator'

Tiboris commented:
"""
On it.
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/840#issuecomment-307358845
___
FreeIPA-devel mailing list -- freeipa-devel@lists.fedorahosted.org
To unsubscribe send an email to freeipa-devel-le...@lists.fedorahosted.org


[Freeipa-devel] [freeipa PR#840][comment] Add Role 'Enrollment Administrator'

2017-06-09 Thread abbra via FreeIPA-devel
  URL: https://github.com/freeipa/freeipa/pull/840
Title: #840: Add Role 'Enrollment Administrator'

abbra commented:
"""
The patch works but we also agreed to add 'Host Enrollment' privilege to 'IT 
Specialists'. Please add this as a separate commit mentioning the same ticket.
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/840#issuecomment-307356354
___
FreeIPA-devel mailing list -- freeipa-devel@lists.fedorahosted.org
To unsubscribe send an email to freeipa-devel-le...@lists.fedorahosted.org


[Freeipa-devel] [freeipa PR#840][comment] Add Role 'Enrollment Administrator'

2017-06-09 Thread abbra via FreeIPA-devel
  URL: https://github.com/freeipa/freeipa/pull/840
Title: #840: Add Role 'Enrollment Administrator'

abbra commented:
"""
LGTM.
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/840#issuecomment-307343158
___
FreeIPA-devel mailing list -- freeipa-devel@lists.fedorahosted.org
To unsubscribe send an email to freeipa-devel-le...@lists.fedorahosted.org


[Freeipa-devel] [freeipa PR#840][comment] Add Role 'Enrollment Administrator'

2017-06-09 Thread Tiboris via FreeIPA-devel
  URL: https://github.com/freeipa/freeipa/pull/840
Title: #840: Add Role 'Enrollment Administrator'

Tiboris commented:
"""
Thanks, changed.
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/840#issuecomment-307324655
___
FreeIPA-devel mailing list -- freeipa-devel@lists.fedorahosted.org
To unsubscribe send an email to freeipa-devel-le...@lists.fedorahosted.org