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

Update postgres install centos #351

Merged
merged 6 commits into from Jan 8, 2018

Conversation

Projects
None yet
3 participants
@sandoche2k
Contributor

sandoche2k commented Jan 5, 2018

No description provided.

@sandoche2k sandoche2k added this to the 2017.4 milestone Jan 5, 2018

@matsduf

We have tried to make the installation document look the same for all OSs and for all databases. Is it possible to structure the same way as for MySQL?

Show outdated Hide outdated docs/Installation.md Outdated
Show outdated Hide outdated docs/Installation.md Outdated
Show outdated Hide outdated docs/Installation.md Outdated
@sandoche2k

This comment has been minimized.

Show comment
Hide comment
@sandoche2k

sandoche2k Jan 8, 2018

Contributor

@matsduf updated the install instructions

Contributor

sandoche2k commented Jan 8, 2018

@matsduf updated the install instructions

@mattias-p

Looks good in general. Requesting a few tweaks.

Show outdated Hide outdated docs/Installation.md Outdated
Show outdated Hide outdated docs/Installation.md Outdated
Show outdated Hide outdated docs/Installation.md Outdated
@mattias-p

Github collapsed my previous review comments, so I'm adding them again.

Show outdated Hide outdated docs/Installation.md Outdated
Show outdated Hide outdated docs/Installation.md Outdated
Show outdated Hide outdated docs/Installation.md Outdated
Show outdated Hide outdated docs/Installation.md Outdated
Show outdated Hide outdated docs/Installation.md Outdated
Show outdated Hide outdated docs/Installation.md Outdated
Show outdated Hide outdated docs/Installation.md Outdated
Show outdated Hide outdated docs/Installation.md Outdated
Show outdated Hide outdated docs/Installation.md Outdated
Show outdated Hide outdated docs/Installation.md Outdated
Show outdated Hide outdated docs/Installation.md Outdated
Show outdated Hide outdated docs/Installation.md Outdated
Show outdated Hide outdated docs/Installation.md Outdated
Add Zonemaster user:
```sh
sudo useradd -r -c "Zonemaster daemon user" zonemaster
```

This comment has been minimized.

@matsduf

matsduf Jan 8, 2018

Contributor

Move these four lines after line 66.

@matsduf

matsduf Jan 8, 2018

Contributor

Move these four lines after line 66.

Show outdated Hide outdated docs/Installation.md Outdated
@matsduf

matsduf approved these changes Jan 8, 2018

@sandoche2k sandoche2k merged commit 5377875 into zonemaster:develop Jan 8, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment