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

Document how to configure external data stores. #851

Merged
merged 3 commits into from
Jul 16, 2019
Merged

Document how to configure external data stores. #851

merged 3 commits into from
Jul 16, 2019

Conversation

yzl
Copy link
Contributor

@yzl yzl commented Jul 10, 2019

πŸ”© Description

Document the current tested state of configuring external data stores with Automate.

Fixes #846

πŸ‘ Definition of Done

πŸ‘Ÿ Demo Script / Repro Steps

⛓️ Related Resources

βœ… Checklist

  • Necessary tests added/updated?
  • Necessary docs added/updated?
  • Code actually executed?
  • Vetting performed (unit tests, lint, etc.)?

@yzl yzl requested a review from a team as a code owner July 10, 2019 20:52
@yzl yzl closed this Jul 10, 2019
@yzl yzl reopened this Jul 10, 2019
@yzl yzl changed the title Document how to configure external data stores. [WIP] Document how to configure external data stores. Jul 10, 2019
@susanev susanev added the documentation Anything related to the Automate docs. label Jul 10, 2019
@@ -119,3 +126,126 @@ sudo chef-automate config patch config.toml
```

and retry opening Chef Automate in your browser.

### Configuring External Data Stores
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I thought it was worth spelling this out here (instead of pointing to the configuration page) because if someone knows upfront they need an external data store, we want them to do their initial deploy that way. I'm not sure this is the best way to encourage that, so I'm willing to hear other ideas.

Copy link
Contributor

Choose a reason for hiding this comment

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

I'm wondering if we want the the initial install instructions to be the only instructions for now?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@stevendanna That makes sense to me, so I removed the ones on the configuration page.

Data Stores]({{< relref "#configuring-external-data-stores" >}}) for information on
configuring Chef Automate to use an externally-deployed PostgreSQL database cluster
and/or Elasticsearch cluster. If you have requirements around a highly-available
deployment of Chef Automate, please reach out to a Customer Success or Professional
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Are these the right folks to contact?

@yzl yzl changed the title [WIP] Document how to configure external data stores. Document how to configure external data stores. Jul 11, 2019
@@ -698,6 +710,37 @@ Then run `chef-automate config patch </path/to/your-file.toml>` to deploy your c
# name = "automate"
```

#### Configuring An External PostgreSQL Database
To use Chef Automate with an external PostgreSQL cluster, create a `.toml` file containing the following partial configuration.
Copy link
Contributor

Choose a reason for hiding this comment

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

πŸ€”I'm wondering if we prefer "TOML file" to ".toml file"?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We're not consistent, so I'm going to make us consistent by making it "TOML file".

enable = true

```

Copy link
Contributor

Choose a reason for hiding this comment

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

I think we are OK not restarting the world in these directions because the restart-workaround code will restart all pgsql clients. However, I think in general we might still want to do a full service restart so that database initialization code in the various services run again.

@@ -119,3 +126,126 @@ sudo chef-automate config patch config.toml
```

and retry opening Chef Automate in your browser.

### Configuring External Data Stores
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm wondering if we want the the initial install instructions to be the only instructions for now?

Yvonne Lam added 3 commits July 12, 2019 11:35
Signed-off-by: Yvonne Lam <yvonne@chef.io>
install only.

Signed-off-by: Yvonne Lam <yvonne@chef.io>
Signed-off-by: Yvonne Lam <yvonne@chef.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deployment-team documentation Anything related to the Automate docs.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Finish documenting external datastores.
4 participants