[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #4763: Rewrote postinstall to Python (3.6+)

[GitBox <gi...@apache.org>]

ocket8888 commented on a change in pull request #4763:
URL: https://github.com/apache/trafficcontrol/pull/4763#discussion_r574944086



##########
File path: traffic_ops/install/bin/postinstall.py
##########
@@ -0,0 +1,1247 @@
+#!/usr/bin/python3
+"""
+This script is meant as a drop-in replacement for the old _postinstall Perl script.
+
+It does, however, offer several more command-line flags not present in the original, to aid in
+testing.
+
+-a, --automatic               If there are questions in the config file which do not have answers,
+                              the script will look to the defaults for the answer. If the answer is
+                              not in the defaults the script will exit.
+--cfile [FILE]                An input config file used to ask and answer questions.
+--debug                       Enables verbose logging output.
+--defaults [FILE]             Writes out a configuration file with defaults which can be used as

Review comment:
       [Looks like argparse does, in fact, support this](https://stackoverflow.com/questions/31127366/single-dash-for-argparse-long-options)




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org