openshift/cloud-credential-operator

Documentation: Improve documentation by adding more details

vlours opened this issue · 5 comments

I was following the GCP documentation trying to Install an OpenShift Cluster with GCP Workload Identity.

I noticed that some of the steps are not well described and need to be reviewed:

  • RELEASE_IMAGE => this variable is noted, but not described. Sharing an example, and suggesting checking the release tags from quay.io should help the user to understand it.

  • ccoctl the command used to describe the syntax is almost correct but can be even better if you use some example to describe it. Furthermore, I've raised Issue #451 to address the confusing sentence to confirm the JSON .
    You should be able to explain it like this (placing this step before the openshift-installer create manifests:

INSTALL_CONFIG_FILE=install-config.yaml
CCO_NAME=$(yq -r '.metadata.name' ${INSTALL_CONFIG_FILE}); CCO_REGION=$(yq -r '.platform.gcp.region' ${INSTALL_CONFIG_FILE}); CCO_PROJECT=$(yq -r '.platform.gcp.projectID' ${INSTALL_CONFIG_FILE})
  • Also, the ccoctl is not macOS friendly, where the openshift-installer is, and the documentation should provide a way to do it.
    I managed to run it into a docker container on my laptop. Here is my command:
# Variables need to be set from  the install_config file as described previously
# Copy the osServiceAccount.json into your Install_dir
cp ~/Download/osServiceAccount.json /full/path/to/your/install_dir/
# Use docker to run ccoctl into an ubi8 image
$ docker run --rm -ti -v /full/path/to/your/install_dir:/mnt redhat/ubi8-minimal /mnt/ccoctl gcp create-all --name=${CCO_NAME} --region=${CCO_REGION} --project=${CCO_PROJECT} --credentials-requests-dir=./mnt/credreqs --output-dir=/mnt/cco_output

And to delete the cco:

docker run --rm -ti -v /full/path/to/your/install_dir:/mnt redhat/ubi8-minimal /mnt/ccoctl gcp delete --name=${CCO_NAME} --project=${CCO_PROJECT}

Otherwise, the rest of the documentation is OK.

Issues go stale after 90d of inactivity.

Mark the issue as fresh by commenting /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
Exclude this issue from closing by commenting /lifecycle frozen.

If this issue is safe to close now please do so with /close.

/lifecycle stale

Stale issues rot after 30d of inactivity.

Mark the issue as fresh by commenting /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.
Exclude this issue from closing by commenting /lifecycle frozen.

If this issue is safe to close now please do so with /close.

/lifecycle rotten
/remove-lifecycle stale

PR #465 waiting for the master branch to be opened.
/remove-lifecycle rotten

Issues go stale after 90d of inactivity.

Mark the issue as fresh by commenting /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
Exclude this issue from closing by commenting /lifecycle frozen.

If this issue is safe to close now please do so with /close.

/lifecycle stale

PR #465 was successfully merged.
Closing this Issue.