Skip to content
This repository has been archived by the owner on Nov 11, 2021. It is now read-only.

Need an empty prefix for the sed command to work on Mac and Helm update before install #108

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

Conversation

@gravitonian gravitonian changed the title Need an empty prefix for the sed command to work on Mac Need an empty prefix for the sed command to work on Mac and Helm update before install May 13, 2019
README.md Outdated
```

### 11. Deploy the DBP

```bash
# Make sure you have the latest Helm charts locally
helm update
Copy link
Contributor

Choose a reason for hiding this comment

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

See #108 (comment). The command for updating the repos has changed. The command 'helm update' is deprecated in favor of 'helm repo update'

@gravitonian
Copy link
Author

Fixed

@jottley jottley self-requested a review May 28, 2019 19:03
@rgauss
Copy link
Contributor

rgauss commented Aug 13, 2019

The broken sed command was actually fixed recently, note there was also an issue with LOCAL_IP not being replaced.

I suppose it wouldn't hurt to add the help repo update, but if so it should be consistent across all sections / environments.

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

Successfully merging this pull request may close these issues.

None yet

4 participants