Fix broken link in deployment prereq

[rafvasq]

Updates Openshift link in Kubernetes cluster deployment prerequisites.

Resolves #293

[ckadner]

Thanks @rafvasq for finding and fixing the broken link and curl command.

I just have a small nit-pick :-)

[ckadner]

Thanks Rafael. Sorry, I did not look at your second change carefully earlier.

[ckadner]

On second look, I don't think this change is right, even though curl seems to work with either command variation.

https://stackoverflow.com/questions/4181703/how-to-concatenate-string-variables-in-bash/4181721#4181721

This form _{$OS}_ prints the URL with braces "_{Darwin}_":

$ echo "https://github.com/kubernetes-sigs/kustomize/releases/download/v3.2.0/kustomize_3.2.0_{$OS}_amd64"

https://github.com/kubernetes-sigs/kustomize/releases/download/v3.2.0/kustomize_3.2.0_{Darwin}_amd64

The original format (_${OS}_) prints the URL correctly ( "_Darwin_" without braces):

$ echo "https://github.com/kubernetes-sigs/kustomize/releases/download/v3.2.0/kustomize_3.2.0_${OS}_amd64"

https://github.com/kubernetes-sigs/kustomize/releases/download/v3.2.0/kustomize_3.2.0_Darwin_amd64

[rafvasq]

Thanks for looking into it. I was having issues with the original curl command and now realize that it wasn't working for me because I'm using oh-my-zsh, which escapes characters with backslash when I paste a URL.

[ckadner]

/lgtm

Thanks Rafael

[mlx-bot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ckadner, rafvasq

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [ckadner]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[ckadner]

/assign @rafvasq