sys.exit() should be use only in main script, otherwise it does not log exceptions properly and it is very hard to debug where an error occurred.
raise RuntimeError instead of sys.exit(1) is perfectly fine, admintools will catch it.
What is the proposed scope?
Acked for 4.4 with scope: do simple replace of sys.exit calls into exceptions only in installer modules.
4.4.0 was released, moving open tickets to 4.4.1
master:
Metadata Update from @mbasti: - Issue assigned to stlaz - Issue set to the milestone: FreeIPA 4.4.1
Login to comment on this ticket.