Cronjob? Ser ni något fel?

sudo väntar väl på att ett lösenord ska skrivas in?

Måste det göras som root?

Måste det göras som root?

Du har skrivit stjärna i alla fält, så det ska alltså köras varje minut? Låter som att det kommer skapa mycket backups?

Det som står i /etc/crontab körs väl förövrigt alltid som root, om inte annat specifikt anges? sudo-delen av kommandot är onödigt men tror inte det påverkar resultatet. Jag hade suddat den delen oavsett.

Hur lägger du till det i crontab?

Har du behörighet att skriva överallt då?

du borde nog ändra crontab medelst: sudo crontab -eu mc
och lägga in
* * * * * tar -cpzf /home/mc/backups/backup_$(date +%Y%m%d_%H%M%S).tar.gz /home/mc/mc/Minecraft
i den crontaben.

Från manualen (`man 5 crontab`):

The ``sixth'' field (the rest of the line) specifies the command to  be
run.   The  entire  command  portion  of the line, up to a newline or %
character, will be executed by /bin/sh or by the shell specified in the
SHELL  variable of the crontab file.  Percent-signs (%) in the command,
unless escaped with backslash (\), will be changed into newline charac‐
ters,  and  all  data  after the first % will be sent to the command as
standard input. There is no way to split a  single  command  line  onto
multiple lines, like the shell's trailing "\".

Se till att escape:a dina %-tecken.

Personligen brukar jag försöka hålla crontab-raderna så enkla som möjligt, och helst bara använda dem för att kalla på externa skript, där man kan lägga in så mycket logik man vill. Här skulle man exempelvis kunna skapa `backup_minecraft` som externt skript och kalla på det via cron.

Jo, fast måste du verkligen köra sudo? Borde du inte ha behörighet med det vanliga kontot?

Jo, fast måste du verkligen köra sudo? Borde du inte ha behörighet med det vanliga kontot?

Från manualen (`man 5 crontab`):

The ``sixth'' field (the rest of the line) specifies the command to  be
run.   The  entire  command  portion  of the line, up to a newline or %
character, will be executed by /bin/sh or by the shell specified in the
SHELL  variable of the crontab file.  Percent-signs (%) in the command,
unless escaped with backslash (\), will be changed into newline charac‐
ters,  and  all  data  after the first % will be sent to the command as
standard input. There is no way to split a  single  command  line  onto
multiple lines, like the shell's trailing "\".

Se till att escape:a dina %-tecken.

Personligen brukar jag försöka hålla crontab-raderna så enkla som möjligt, och helst bara använda dem för att kalla på externa skript, där man kan lägga in så mycket logik man vill. Här skulle man exempelvis kunna skapa `backup_minecraft` som externt skript och kalla på det via cron.

Precis, här undviker du problemet genom att ta bort alla procenttecken. Vill du ha dem kvar för att formatera utskriften av `date` så måste du "escape:a" dina procenttecken med backslash, dvs i stället för att skriva:

* * * * * echo "Ja så har det ändå äntligen hänt, ska det va ska det va 100%"

så får du skriva

* * * * * echo "Ja så har det ändå äntligen hänt, ska det va ska det va 100\%"

i crontab.

Alternativet är att skapa ett externt skript som genomför backupen, och hålla crontab enkel. I detta fallet är det ju inte så komplexa åtgärder det handlar om, men i praktiken skulle du kunna

1. skapa det externa skriptet `/home/mc/bin/backup_minecraft` med innehåll:

   #!/bin/sh
   tar -cpzf /home/mc/backups/backup_$(date +%Y%m%d_%H%M%S).tar.gz /home/mc/mc/Minecraft

2. göra detta körbart (`chmod +x /home/mc/bin/backup_minecraft`)

3. kalla på detta genom crontab med en rad likt

   0 0 * * * /home/mc/bin/backup_minecraft

Fördelen är att du är mindre begränsad i hur du utformar ditt skript. Här är det som sagt inte särdeles avancerat i vilket fall, men redan om man börjar använda subskal som du gör här så tycker jag det är smidigare att direkt lägga logik externt (det hade ju exempelvis undvikit problem för dig här). Att det går att använda generella skalkonstruktioner i crontab ser jag mer som en artefakt av dess konstruktion snarare än att det skulle vara ett fundamentalt designval att låta raderna vara Turingkompletta.

Att använda skript som kan köras på egen hand utanför cron gör det också enklare att felsöka, det är lätt att bygga in exempelvis en debugväxel, man kan lätt köra skriptet på godtycklig tid för att testa (i stället för att som du här ställa in cron på att köra raden varje minut för att kunna testa), du kan lättare återanvända logiken mellan flera olika crontabs, med mera. Kort sagt: se crontab mer som en enkel lista på saker att köra (som råkar ha lite extra "bells and whistles" tillgängliga) än en programmeringsmiljö.

Ta isf och kika lite närmare på "rsync", så hanterar du ev avbrott bättre också

Jag misstänker att det är Minecraft-världsdata som du vill backa upp? Då borde du även köra Minecraft i en screen på någon användare (minecraft)?
Du bör skicka kommandot "save-off" och "save-all" i screenen innan backup så servern sparar ned allt som körs i minnet till filer först - för att få en så komplett backup som möjligt. När backupen är klar skickar du "save-on" till Minecraft så den kan återuppta world auto-save.

http://minecraft.gamepedia.com/Tutorials/Server_startup_script - Denna länk behandlar ganska väl hur du sätter upp Minecraft som en tjänst på servern, och även hur du kan ta backup & skicka kommandon genom samma tjänst.

Du kan använda variabler för att "minnas" namn och liknande enklare. Även funktioner kan generellt vara smidiga. Rigoröst kommenterad prototyp följer; se till att läsa och förstå innan den används (inte minst `find`-raden som raderar filer), och notera också att det är helt otestat från mitt håll:

#!/bin/sh
BACKUP_PATH="/home/mc/backups"
MC_PATH="/home/mc/mc/Minecraft"
TGZ_FILE="backup_$(date +%Y%m%d_%H%M%S).tar.gz"
SCREEN_NAME="MC"

# När man börjar upprepa sig så är det ofta bra att skapa en funktion så att
# man undviker skrivfel och samtidigt håller skriptet mer lättläst.
mc_cmd() {
	# Man får hålla tungan rätt i mun för att hänga med i hur
	# citationstecknen fungerar här, men det ska vara all right för
	# närvarande. $1 är helt enkelt det första argumentet när funktionen
	# kallas.
	screen -S "$SCREEN_NAME" -X eval 'stuff "/'"$1"'\015"'
}

mc_cmd save-off
mc_cmd save-all
tar cpzf "$BACKUP_PATH/$TGZ_FILE" "$MC_PATH"
mc_cmd save-on

# Denna bit ser inte speciellt robust ut i mina ögon. Man ska alltid vara
# defensiv när det gäller att skicka filnamn genom pipes, då filnamn _kan_
# innehålla saker som kan överraska (exempelvis radbrytningar och annat). Även
# om man "vet" att de inte gör det så är det inte bra praxis att lära sig
# olater, när det finns mer robusta sätt tillgängliga. Ersätt hellre med
# exempelvis `find` som rensar allt äldre än 24 dagar (här baserat på dess
# datumstämpel snarare än dess filnamn), om det är det du vill.
#cd /home/mc/backups
#ls -t | sed -e '1,24d' | xargs -d '\n' rm
find "$BACKUP_PATH" -mindepth 1 -mtime +24 -delete
# Läs `man find` för att se vad ovanstående kommando gör!

# Filuppladdning löses troligen enklare om du har ett shellkonto någonstans och
# kan köra `rsync` som nämndes tidigare i tråden, men det är också lösbart
# genom FTP.
#
# Så som du har skrivit nu är inte vettigt dock. Varje rad i ett shellskript
# exekveras av tolken, så det som du tror hanteras av FTP-programmet
# försöker skalet i stället köra som kommandon, vilket lär ge en del
# felmeddelanden i bästa fall, och göra något oväntat i sämsta fall.
#
# Det är dock nästan möjligt att göra som du skriver, men kommandona behöver
# skickas in som en sträng. Vi kan göra det med "heredoc"-syntax:
#ftp <<EOF
#open minip
#user mittanv mittlösenord
#put "$BACKUP_PATH/$TGZ_FILE"
#EOF
# Alternativt kan man här använda Curl:
curl --upload-file "$BACKUP_PATH/$TGZ_FILE" ftp://minip --user mittanv:mittlösenord
# Om detta körs på en "delad dator" så finns det en säkerhetsbrist i att ange
# inloggningsuppgifter direkt på kommandoraden. Bättre är att använda en fil
# med dessa uppgifter lagrade; se växeln `-n` till Curl för att använda
# `~/.netrc` till detta (denna fil går även att använda för `ftp`).
#
# Man kan även titta på `lftp` och `ncftp` om man vill göra mer avancerade
# saker över FTP.
#
# Notera att det allmänt är lite vanskligt att behöva ange lösenord i klartext,
# och FTP är på egen hand väldigt osäkert. Vanligare är att använda
# SSH-inloggning med nycklar, om nu fjärrservern man vill köra backup till är
# en "vettig dator" som förstår SSH :-) .

Om du tar bort `-delete` från `find`-kommandot så bara skrivs de filer som skulle ha tagits bort ut. Använd detta för att säkerställa att raden gör vad du vill. Notera att det blir fullständigt kaos om du av någon anledning skulle lämna variabeln `BACKUP_PATH` tom, så gör inte det. Kanske du till och med borde ha en extrakontroll innan `find`-raden som säkerställer att du angett en giltig katalog där, typ:

if ! [ -d "$BACKUP_PATH" ]; then
	printf '%s: fel: "%s" är inte en giltig katalog! Avbryter.\n' "${0##*/}" "$BACKUP_PATH" 1>&2
	exit 2
fi
find "$BACKUP_PATH" -mindepth 1 -mtime +24 -delete

Det skadar inte att vara defensiv när det handlar om kommandon som exempelvis raderar filer eller på andra sätt kan förändra saker till det sämre.