Commit 0e28a016 authored by k8s-merge-robot's avatar k8s-merge-robot Committed by GitHub

Merge pull request #26439 from aanm/adding-g-curl-option

Automatic merge from submit-queue Add -g curl option to hack/lib/util.sh wait_for_url Add `-g` curl option ``` This option switches off the "URL globbing parser". When you set this option, you can specify URLs that contain the letters {}[] without having them being interpreted by curl itself. Note that these letters are not normal legal URL contents but they should be encoded according to the URI standard. ``` > Why? So that IPv6 addresses with `[` and `]`, like `[::1]:8080`, work with that script. Signed-off-by: 's avatarAndré Martins <aanm90@gmail.com>
parents b51e5c01 8497255b
...@@ -32,7 +32,7 @@ kube::util::wait_for_url() { ...@@ -32,7 +32,7 @@ kube::util::wait_for_url() {
local i local i
for i in $(seq 1 $times); do for i in $(seq 1 $times); do
local out local out
if out=$(curl -fs $url 2>/dev/null); then if out=$(curl -gfs $url 2>/dev/null); then
kube::log::status "On try ${i}, ${prefix}: ${out}" kube::log::status "On try ${i}, ${prefix}: ${out}"
return 0 return 0
fi fi
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment