#5750 Stop using sys.exit() from modules
Closed: Fixed None Opened 7 years ago by mbasti.

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:

  • 5776f1e Remove sys.exit from install modules and scripts

Metadata Update from @mbasti:
- Issue assigned to stlaz
- Issue set to the milestone: FreeIPA 4.4.1

6 years ago

Login to comment on this ticket.

Metadata