Skip to content

Commit 0e2486c

Browse files
committed
fix: use full paths
- remove doi_mint_pending_releases from backup - doi_mint_pending_releases and doi_sync_all_metadata could be moved to /etc/cron.weekly or removed entirely since they are simply a fallback if the scheduled async huey doi sync fails
1 parent b1b8c51 commit 0e2486c

File tree

3 files changed

+2
-11
lines changed

3 files changed

+2
-11
lines changed

django/deploy/cron.daily/backup

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -4,12 +4,3 @@
44

55
export DJANGO_SETTINGS_MODULE="core.settings.production"
66
inv -r /code db.backup borg.backup >> /shared/logs/comses-backup.log 2>&1
7-
8-
# FIXME: should we synchronize DOI metadata after backups or before backups?
9-
# Pros of after backups: DOI metadata syncing alters the DB and taking a snapshot backup before running this may protect against
10-
# errors or bugs that happened as a result of the sync
11-
#
12-
# Pros of before backups: backups will be more up-to-date
13-
#
14-
# currently syncing after backups
15-
./manage.py doi_mint_pending_releases --no-interactive --no-dry-run

django/deploy/cron.daily/doi_mint_pending_releases

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,4 +7,4 @@
77
# Pros of before backups: backups will be more up-to-date
88
#
99
# currently syncing after backups
10-
./manage.py doi_mint_pending_releases --no-interactive --no-dry-run
10+
/code/manage.py doi_mint_pending_releases --no-interactive --no-dry-run
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
11
#!/bin/sh
22

3-
./manage.py doi_sync_metadata --no-interactive --no-dry-run
3+
/code/manage.py doi_sync_metadata --no-interactive --no-dry-run

0 commit comments

Comments
 (0)