Skip to content
This repository has been archived by the owner on May 6, 2022. It is now read-only.

The Helm Install docs in the API Aggregation Setup don't work on a Mac #1004

Closed
arschles opened this issue Jul 3, 2017 · 5 comments
Closed
Labels
kind/bug Categorizes issue or PR as related to a bug. needs-docs
Milestone

Comments

@arschles
Copy link
Contributor

arschles commented Jul 3, 2017

The helm install command in this section don't work on a mac. They produce the following error output:

admins-MacBook-Pro:service-catalog aaschles$ helm install charts/catalog     --name ${HELM_NAME} --namespace ${SVCCAT_NAMESPACE}     --set apiserver.auth.enabled=true         --set useAggregator=true         --set apiserver.tls.ca=$(base64 --wrap 0 ${SC_SERVING_CA})         --set apiserver.tls.cert=$(base64 --wrap 0 ${SC_SERVING_CERT})         --set apiserver.tls.key=$(base64 --wrap 0 ${SC_SERVING_KEY})
base64: unrecognized option `--wrap'
Usage:	base64 [-hvD] [-b num] [-i in_file] [-o out_file]
  -h, --help     display this message
  -D, --decode   decodes input
  -b, --break    break encoded string into num character lines
  -i, --input    input file (default: "-" for stdin)
  -o, --output   output file (default: "-" for stdout)
base64: unrecognized option `--wrap'
Usage:	base64 [-hvD] [-b num] [-i in_file] [-o out_file]
  -h, --help     display this message
  -D, --decode   decodes input
  -b, --break    break encoded string into num character lines
  -i, --input    input file (default: "-" for stdin)
  -o, --output   output file (default: "-" for stdout)
base64: unrecognized option `--wrap'
Usage:	base64 [-hvD] [-b num] [-i in_file] [-o out_file]
  -h, --help     display this message
  -D, --decode   decodes input
  -b, --break    break encoded string into num character lines
  -i, --input    input file (default: "-" for stdin)
  -o, --output   output file (default: "-" for stdout)
Error: a release named "catalog" already exists.
Please run: helm ls --all "catalog"; helm del --help
@arschles arschles added kind/bug Categorizes issue or PR as related to a bug. needs-docs labels Jul 3, 2017
arschles added a commit to arschles/kubernetes-service-catalog that referenced this issue Jul 3, 2017
And removing the base64 commands from the helm install command.

This will address
kubernetes-retired#1004
@MHBauer
Copy link
Contributor

MHBauer commented Jul 3, 2017

gnu base64 (linux) vs bsd base64 (mac).

We can say, run it on linux, or run it on gnu base64, but since it's a user run thing, we probably have to figure out how to use bsd base64 and tell the user what they might need to run.

@arschles
Copy link
Contributor Author

arschles commented Jul 3, 2017

@MHBauer see how I fixed this in #1005

arschles added a commit to arschles/kubernetes-service-catalog that referenced this issue Jul 3, 2017
* Bump OSB client (kubernetes-retired#1000)

(bc5d4ac17be7f9af33683fe4ee3cb44ee89ac667)

This update includes new wrapper functions to generate fake errors and a
fix for printing errors.

* add parameterization for user-broker image to e2e tests (kubernetes-retired#995)

* Add logging for normal Bind errors (kubernetes-retired#993)

This simply differentiates a broker failure response as outlined in the
spec versus some other unknown error.

* Adding required values to the APIRegistration object for v1beta1

* Accessing TLS information by file

And removing the base64 commands from the helm install command.

This will address
kubernetes-retired#1004

* Adding deletion instructions

* Dropping CA, certs and keys into the charts/catalog directory

so that the helm chart can read them

* Moving the TLS setup script into the chart

* Ignoring the TLS chain created by the TLS created by the script

* Adding a walkthrough for clusters running v1.7.0 and above

* Removing errant tickmark

* Adding flag to skip client CA lookups

This enables the service-catalog API server to start on systems where
requestheader-client-ca-file was not given

* Adding empty requestHeaderCA flag to the values.yaml file

This indicates that the flag exists & is used in the chart, and has
comments explaining what it’s for

* Removing context and using fully qualified binding name
@duglin duglin added this to the 0.2.0 milestone Jul 10, 2017
@arschles arschles modified the milestones: 0.1.0, 0.2.0 Aug 28, 2017
@arschles
Copy link
Contributor Author

arschles commented Aug 28, 2017

@MHBauer update: I closed #1005 without merge. We can't say that this must be run only on linux.

I can think of a few options:

  1. Enable people to run the helm command in a container
  2. Enable people to run the base64 command in a container
  3. Do the base64 encoding inside the helm chart (I did this in Cleaning up the setup instructions for Kubernetes 1.7 #1005)
  4. Add the bsd base64 instructions, as you suggested

I'm partial to (4) since it seems to be the easiest.

@pmorie
Copy link
Contributor

pmorie commented Aug 28, 2017 via email

arschles added a commit to arschles/kubernetes-service-catalog that referenced this issue Aug 28, 2017
@arschles
Copy link
Contributor Author

@pmorie the files have newlines in them - do you know another good way to pass them in on the helm CLI?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
kind/bug Categorizes issue or PR as related to a bug. needs-docs
Projects
None yet
Development

No branches or pull requests

4 participants