Merge pull request #1205 from mungomat/backup_time

Backup time

internal/archiver/archive_reader.go

@@ -28,7 +28,7 @@ func (r *Reader) Archive(ctx context.Context, name string, rd io.Reader, p *rest
 	}
 
 	debug.Log("start archiving %s", name)
-	sn, err := restic.NewSnapshot([]string{name}, r.Tags, r.Hostname)
+	sn, err := restic.NewSnapshot([]string{name}, r.Tags, r.Hostname, time.Now())
 	if err != nil {
 		return nil, restic.ID{}, err
 	}

internal/archiver/archiver.go

@@ -651,7 +651,7 @@ func (p baseNameSlice) Swap(i, j int)      { p[i], p[j] = p[j], p[i] }
 // Snapshot creates a snapshot of the given paths. If parentrestic.ID is set, this is
 // used to compare the files to the ones archived at the time this snapshot was
 // taken.
-func (arch *Archiver) Snapshot(ctx context.Context, p *restic.Progress, paths, tags []string, hostname string, parentID *restic.ID) (*restic.Snapshot, restic.ID, error) {
+func (arch *Archiver) Snapshot(ctx context.Context, p *restic.Progress, paths, tags []string, hostname string, parentID *restic.ID, time time.Time) (*restic.Snapshot, restic.ID, error) {
 	paths = unique(paths)
 	sort.Sort(baseNameSlice(paths))
 
@@ -666,7 +666,7 @@ func (arch *Archiver) Snapshot(ctx context.Context, p *restic.Progress, paths, t
 	defer p.Done()
 
 	// create new snapshot
-	sn, err := restic.NewSnapshot(paths, tags, hostname)
+	sn, err := restic.NewSnapshot(paths, tags, hostname, time)
 	if err != nil {
 		return nil, restic.ID{}, err
 	}

internal/archiver/archiver_test.go

@@ -108,7 +108,7 @@ func archiveDirectory(b testing.TB) {
 
 	arch := archiver.New(repo)
 
-	_, id, err := arch.Snapshot(context.TODO(), nil, []string{BenchArchiveDirectory}, nil, "localhost", nil)
+	_, id, err := arch.Snapshot(context.TODO(), nil, []string{BenchArchiveDirectory}, nil, "localhost", nil, time.Now())
 	OK(b, err)
 
 	b.Logf("snapshot archived as %v", id)
@@ -302,7 +302,7 @@ func TestArchiveEmptySnapshot(t *testing.T) {
 
 	arch := archiver.New(repo)
 
-	sn, id, err := arch.Snapshot(context.TODO(), nil, []string{"file-does-not-exist-123123213123", "file2-does-not-exist-too-123123123"}, nil, "localhost", nil)
+	sn, id, err := arch.Snapshot(context.TODO(), nil, []string{"file-does-not-exist-123123213123", "file2-does-not-exist-too-123123123"}, nil, "localhost", nil, time.Now())
 	if err == nil {
 		t.Errorf("expected error for empty snapshot, got nil")
 	}
@@ -354,7 +354,7 @@ func TestArchiveNameCollision(t *testing.T) {
 
 	arch := archiver.New(repo)
 
-	sn, id, err := arch.Snapshot(context.TODO(), nil, []string{"testfile", filepath.Join("..", "testfile")}, nil, "localhost", nil)
+	sn, id, err := arch.Snapshot(context.TODO(), nil, []string{"testfile", filepath.Join("..", "testfile")}, nil, "localhost", nil, time.Now())
 	OK(t, err)
 
 	t.Logf("snapshot archived as %v", id)

internal/archiver/testing.go

@@ -3,6 +3,7 @@ package archiver
 import (
 	"context"
 	"testing"
+	"time"
 
 	"github.com/restic/restic/internal/restic"
 )
@@ -10,7 +11,7 @@ import (
 // TestSnapshot creates a new snapshot of path.
 func TestSnapshot(t testing.TB, repo restic.Repository, path string, parent *restic.ID) *restic.Snapshot {
 	arch := New(repo)
-	sn, _, err := arch.Snapshot(context.TODO(), nil, []string{path}, []string{"test"}, "localhost", parent)
+	sn, _, err := arch.Snapshot(context.TODO(), nil, []string{path}, []string{"test"}, "localhost", parent, time.Now())
 	if err != nil {
 		t.Fatal(err)
 	}

internal/checker/checker_test.go

@@ -7,6 +7,7 @@ import (
 	"path/filepath"
 	"sort"
 	"testing"
+	"time"
 
 	"github.com/restic/restic/internal/archiver"
 	"github.com/restic/restic/internal/checker"
@@ -305,7 +306,7 @@ func TestCheckerModifiedData(t *testing.T) {
 	defer cleanup()
 
 	arch := archiver.New(repo)
-	_, id, err := arch.Snapshot(context.TODO(), nil, []string{"."}, nil, "localhost", nil)
+	_, id, err := arch.Snapshot(context.TODO(), nil, []string{"."}, nil, "localhost", nil, time.Now())
 	test.OK(t, err)
 	t.Logf("archived as %v", id.Str())
 

internal/restic/snapshot.go

@@ -29,7 +29,7 @@ type Snapshot struct {
 
 // NewSnapshot returns an initialized snapshot struct for the current user and
 // time.
-func NewSnapshot(paths []string, tags []string, hostname string) (*Snapshot, error) {
+func NewSnapshot(paths []string, tags []string, hostname string, time time.Time) (*Snapshot, error) {
 	for i, path := range paths {
 		if p, err := filepath.Abs(path); err != nil {
 			paths[i] = p
@@ -38,7 +38,7 @@ func NewSnapshot(paths []string, tags []string, hostname string) (*Snapshot, err
 
 	sn := &Snapshot{
 		Paths:    paths,
-		Time:     time.Now(),
+		Time:     time,
 		Tags:     tags,
 		Hostname: hostname,
 	}

internal/restic/snapshot_test.go

@@ -2,6 +2,7 @@ package restic_test
 
 import (
 	"testing"
+	"time"
 
 	"github.com/restic/restic/internal/restic"
 	. "github.com/restic/restic/internal/test"
@@ -10,6 +11,6 @@ import (
 func TestNewSnapshot(t *testing.T) {
 	paths := []string{"/home/foobar"}
 
-	_, err := restic.NewSnapshot(paths, nil, "foo")
+	_, err := restic.NewSnapshot(paths, nil, "foo", time.Now())
 	OK(t, err)
 }

internal/restic/testing.go

@@ -164,7 +164,7 @@ func TestCreateSnapshot(t testing.TB, repo Repository, at time.Time, depth int,
 	t.Logf("create fake snapshot at %s with seed %d", at, seed)
 
 	fakedir := fmt.Sprintf("fakedir-at-%v", at.Format("2006-01-02 15:04:05"))
-	snapshot, err := NewSnapshot([]string{fakedir}, []string{"test"}, "foo")
+	snapshot, err := NewSnapshot([]string{fakedir}, []string{"test"}, "foo", time.Now())
 	if err != nil {
 		t.Fatal(err)
 	}

internal/walk/walk_test.go

@@ -25,7 +25,7 @@ func TestWalkTree(t *testing.T) {
 
 	// archive a few files
 	arch := archiver.New(repo)
-	sn, _, err := arch.Snapshot(context.TODO(), nil, dirs, nil, "localhost", nil)
+	sn, _, err := arch.Snapshot(context.TODO(), nil, dirs, nil, "localhost", nil, time.Now())
 	OK(t, err)
 
 	// flush repo, write all packs