Skip to content

Commit

Permalink
infra/image/build.sh: Use consistent options for hostname
Browse files Browse the repository at this point in the history
This patch makes the CLI argument for setting the image hostname
consistent with 'start.sh', settling on '-n HOSTNAME' for both scripts.
  • Loading branch information
rjeffman committed Nov 12, 2024
1 parent 24569b8 commit a96611f
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions infra/image/build.sh
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ valid_distro() {
usage() {
local prog="${0##*/}"
cat << EOF
usage: ${prog} [-h] [-p] [-c HOSTNAME] [-s] distro
usage: ${prog} [-h] [-p] [-n HOSTNAME] [-s] distro
${prog} build a container image to test ansible-freeipa.
EOF
}
Expand All @@ -29,7 +29,7 @@ positional arguments:
optional arguments:
-c HOSTNAME Container hostname
-n HOSTNAME Container hostname
-p Give extended privileges to the container
-s Deploy IPA server
EOF
Expand All @@ -43,11 +43,11 @@ quayname="quay.io/ansible-freeipa/upstream-tests"
deploy_server="N"
privileged=""

while getopts ":hc:ps" option
while getopts ":hn:ps" option
do
case "${option}" in
h) help && exit 0 ;;
c) hostname="${OPTARG}" ;;
n) hostname="${OPTARG}" ;;
p) privileged="privileged" ;;
s) deploy_server="Y" ;;
*) die -u "Invalid option: ${option}" ;;
Expand Down

0 comments on commit a96611f

Please sign in to comment.