Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Doco updates and questions #17

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

lazzarello
Copy link

Getting set up with your awesome project. Shuffles some things around in the docs, asked some questions. Open to answers as review.

Copy link
Member

@stevesloka stevesloka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR! Just a couple things to clean up in the docs.

* To push your image to docker hub, set $PREFIX to your account name, then `make push`

## Get Started
1. Create auth tokens file (how?)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tokens can be whatever length or format you would like, I typically just generate a random string .

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

excellent, thanks for confirming. I'll update with a "generate a SHA-1" one-liner.

* docker-registry: Set to url of private docker registry
* template-namespace: Namespace to 'clone from when creating new deployments'
* path-to-tokens: Full path including file name to tokens file for authorization, setting to empty string will disable.

## Building

* Install Go
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe link to golang.org?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

linked to the getting started/install stuff

@lazzarello
Copy link
Author

I added an example token generation step, then deployed the secret and the rc spec to my k8s cluster. looking good so far. moving onto the services.

@stevesloka
Copy link
Member

Awesome! Thanks so much for this! Would you mind rebasing + squashing down before merging?

@lazzarello
Copy link
Author

I did rebase. I think you can alter the commits if you like. I commit a lot. It's a habit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants