Skip to content

Conversation

@enerfip-dev
Copy link

No description provided.

@enerfip-dev enerfip-dev self-assigned this Jul 16, 2019
@demental demental force-pushed the encrypt-with-gpg branch 2 times, most recently from 1dc718a to f622161 Compare July 16, 2019 17:11
@enerfip-dev enerfip-dev changed the title add optional GPG symmetric encryption add optional GPG symmetric encryption and MongoDB support Jul 16, 2019
@demental demental force-pushed the encrypt-with-gpg branch 2 times, most recently from a7bfe7f to 4fa2907 Compare July 17, 2019 06:11

```
heroku config:add HEROKU_API_KEY=`heroku authorizations:create -S -d my-database-backups` -a my-database-backups
heroku config:add HEROKU_API_KEY=`heroku authorizations:create -S -s read -d my-database-backups` -a my-database-backups
Copy link
Author

Choose a reason for hiding this comment

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

This does not work : 403 error

@@ -0,0 +1,9 @@
#!/bin/bash
Copy link
Author

Choose a reason for hiding this comment

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

TODO : check if there is a buildpack to add aws cli tools

Copy link
Collaborator

@gdufloux gdufloux left a comment

Choose a reason for hiding this comment

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

Mise à part l'install local de awscli qu'il me semble qu'on perd au passage, RAS.
y'a plus qu'à test que ça fonctionne :)

bin/backup.sh Outdated
Comment on lines 21 to 25
#install aws-cli
curl https://s3.amazonaws.com/aws-cli/awscli-bundle.zip -o awscli-bundle.zip
unzip awscli-bundle.zip
chmod +x ./awscli-bundle/install
./awscli-bundle/install -i /tmp/aws
Copy link
Collaborator

Choose a reason for hiding this comment

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

toujours utile pour du test local tout de meme non ?

Copy link
Collaborator

@gdufloux gdufloux left a comment

Choose a reason for hiding this comment

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

typo sur KEEP_ONCE_BACKUP_PER_DAY


BACKUP_FILE_NAME="$(date +"%Y-%m-%d-%H-%M")-$APP-$DATABASE.dump"
if [ $KEEP_ONE_BACKUP_PER_WEEK != false ]; then
BACKUP_FILE_NAME="$(date +"%Y-%U")-$APP-$DATABASE.dump"
Copy link
Collaborator

Choose a reason for hiding this comment

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

ou %W si tu veux Week number of the year. Week starts with Monday

bin/backup.sh Outdated

if [ $KEEP_ONE_BACKUP_PER_WEEK != false ]; then
BACKUP_FILE_NAME="$(date +"%Y-%U")-$APP-$DATABASE.dump"
elif [ $KEEP_ONCE_BACKUP_PER_DAY != false ]; then
Copy link
Collaborator

Choose a reason for hiding this comment

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

KEEP_ONCE_BACKUP_PER_DAY -> KEEP_ONE_BACKUP_PER_DAY

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.

4 participants