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

Add backup info to ILPKit docs #423

Closed
wants to merge 1 commit into from
Closed

Add backup info to ILPKit docs #423

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Jun 15, 2017

No description provided.

@ghost
Copy link
Author

ghost commented Jun 15, 2017

Fixes #422

```
To restore your ilpkit backup use
```
postgres$ psql ilpkit < ilpkitdb.backup
Copy link

Choose a reason for hiding this comment

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

should be postgres$ psql ilpkit < ~/yourcloudmountedfolder/ilpkitdb.backup

Copy link
Contributor

Choose a reason for hiding this comment

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

also depends on whether the cloud mounted folder is in postgres's home directory or your regular user's

Copy link

Choose a reason for hiding this comment

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

Good point. Also, the postgres user does not have sudo privileges, so the creation of the folder would fail in the first place: postgres$ sudo mkdir ~/yourcloudmountedfolder.

How about we run this not as postgres, but as sharafian?

@@ -110,6 +110,22 @@ Finally, create the `ilpkit` database while logged in as the `postgres` user.

```sh
postgres$ createdb ilpkit
```

For redundancy of data you may want to run this on a crontab. Outputing the file to
Copy link

Choose a reason for hiding this comment

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

Should the backup instructions go into a new section, separate from the setup DB instructions?

@sharafian
Copy link
Contributor

Closing due to outdated-ness

@sharafian sharafian closed this Jun 21, 2018
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.

3 participants