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

maintain: update readme and install on k8s docs #3107

Merged
merged 5 commits into from Sep 2, 2022

Conversation

technovangelist
Copy link
Contributor

Summary

This updates the readme and the install on kubernetes docs based on the changes made in the quickstart

Checklist

  • Wrote appropriate unit tests
  • Considered security implications of the change
  • Updated associated docs where necessary
  • Updated associated configuration where necessary
  • Change is backwards compatible if it needs to be (user can upgrade without manual steps?)
  • Nothing sensitive logged
  • Considered data migrations for smooth upgrades

Related Issues

Resolves #

Signed-off-by: Matthew Williams <m@technovangelist.com>
Signed-off-by: Matthew Williams <m@technovangelist.com>
Signed-off-by: Matthew Williams <m@technovangelist.com>
Signed-off-by: Matthew Williams <m@technovangelist.com>
Signed-off-by: Matthew Williams <m@technovangelist.com>
Copy link
Collaborator

@BruceMacD BruceMacD left a comment

Choose a reason for hiding this comment

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

password: SetThisPassword! #note this password is now set as plaintext in this file

# Create a "admin@example.com" user and set a password passed in as a file. The file will need
# to be mounted into the pod using `volumes` and `volumeMounts`.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is there a command that allows me to do this easily? How about a link?

Install Infra via `helm`:

```
helm repo add infrahq https://helm.infrahq.com
helm repo update
helm install infra infrahq/infra
helm update --install infra infrahq/infra
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
helm update --install infra infrahq/infra
helm upgrade --install infra infrahq/infra -f values.yaml

Install Infra via `helm`:

```
helm repo add infrahq https://helm.infrahq.com
helm repo update
helm install infra infrahq/infra
helm update --install infra infrahq/infra
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
helm update --install infra infrahq/infra
helm upgrade --install infra infrahq/infra -f values.yaml

@technovangelist technovangelist merged commit 9b2293d into main Sep 2, 2022
@technovangelist technovangelist deleted the matt/docsupdate branch September 2, 2022 15:24
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.

None yet

3 participants