Skip to content

Commit c2b1f0b

Browse files
mkolesniktpantelis
authored andcommitted
Remove deploytool_.*_args
These aren't really used and are just dead code now. Signed-off-by: Mike Kolesnik <[email protected]>
1 parent 87515d0 commit c2b1f0b

File tree

3 files changed

+4
-12
lines changed

3 files changed

+4
-12
lines changed

scripts/shared/deploy.sh

+1-5
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,6 @@
55
source "${SCRIPTS_DIR}/lib/shflags"
66
DEFINE_string 'settings' '' "Settings YAML file to customize cluster deployments"
77
DEFINE_string 'deploytool' 'operator' 'Tool to use for deploying (operator/helm/bundle/ocm)'
8-
DEFINE_string 'deploytool_broker_args' '' 'Any extra arguments to pass to the deploytool when deploying the broker'
9-
DEFINE_string 'deploytool_submariner_args' '' 'Any extra arguments to pass to the deploytool when deploying submariner'
108
DEFINE_boolean 'globalnet' false "Deploy with operlapping CIDRs (set to 'true' to enable)"
119
DEFINE_boolean 'service_discovery' false "Enable multicluster service discovery (set to 'true' to enable)"
1210
DEFINE_string 'timeout' '5m' "Timeout flag to pass to kubectl when waiting (e.g. 30s)"
@@ -20,14 +18,12 @@ eval set -- "${FLAGS_ARGV}"
2018
[[ "${FLAGS_globalnet}" = "${FLAGS_TRUE}" ]] && globalnet=true || globalnet=false
2119
[[ "${FLAGS_service_discovery}" = "${FLAGS_TRUE}" ]] && service_discovery=true || service_discovery=false
2220
deploytool="${FLAGS_deploytool}"
23-
deploytool_broker_args="${FLAGS_deploytool_broker_args}"
24-
deploytool_submariner_args="${FLAGS_deploytool_submariner_args}"
2521
settings="${FLAGS_settings}"
2622
timeout="${FLAGS_timeout}"
2723
image_tag="${FLAGS_image_tag}"
2824
cable_driver="${FLAGS_cable_driver}"
2925

30-
echo "Running with: globalnet=${globalnet@Q}, deploytool=${deploytool@Q}, deploytool_broker_args=${deploytool_broker_args@Q}, deploytool_submariner_args=${deploytool_submariner_args@Q}, settings=${settings@Q}, timeout=${timeout}, image_tag=${image_tag}, cable_driver=${cable_driver}, service_discovery=${service_discovery}"
26+
echo "Running with: globalnet=${globalnet@Q}, deploytool=${deploytool@Q}, settings=${settings@Q}, timeout=${timeout}, image_tag=${image_tag}, cable_driver=${cable_driver}, service_discovery=${service_discovery}"
3127

3228
set -em
3329

scripts/shared/lib/deploy_helm

+2-4
Original file line numberDiff line numberDiff line change
@@ -19,8 +19,7 @@ function setup_broker() {
1919
submariner-latest/submariner-k8s-broker \
2020
--create-namespace \
2121
--kube-context "${cluster}" \
22-
--namespace "${BROKER_NAMESPACE}" \
23-
${deploytool_broker_args}
22+
--namespace "${BROKER_NAMESPACE}"
2423
fi
2524

2625
submariner_broker_url=$(kubectl -n default get endpoints kubernetes -o jsonpath="{.subsets[0].addresses[0].ip}:{.subsets[0].ports[?(@.name=='https')].port}")
@@ -64,8 +63,7 @@ function helm_install_subm() {
6463
--set operator.image.pullPolicy="IfNotPresent" \
6564
--set submariner.images.repository="localhost:5000" \
6665
--set submariner.images.tag="local" \
67-
--set brokercrds.create="${crd_create}" \
68-
${deploytool_submariner_args}
66+
--set brokercrds.create="${crd_create}"
6967
}
7068

7169
function install_subm_all_clusters() {

scripts/shared/lib/deploy_operator

+1-3
Original file line numberDiff line numberDiff line change
@@ -40,8 +40,7 @@ function setup_broker() {
4040
--kubeconfig "${KUBECONFIGS_DIR}/kind-config-$cluster" \
4141
${subctlrepver} \
4242
${gn} \
43-
${sd} \
44-
${deploytool_broker_args}
43+
${sd}
4544
)
4645
}
4746

@@ -72,7 +71,6 @@ function subctl_install_subm() {
7271
--globalnet-cidr "${global_CIDRs[$cluster]}" \
7372
--natt=false \
7473
--cable-driver "${cable_driver}" \
75-
${deploytool_submariner_args} \
7674
${cidr_args} \
7775
"${OUTPUT_DIR}"/broker-info.subm
7876

0 commit comments

Comments
 (0)