From d22cdf2dd9f09a80804c24fb0efa80572ca37b86 Mon Sep 17 00:00:00 2001
From: Samuel Ortiz <sameo@linux.intel.com>
Date: Thu, 20 Dec 2018 13:17:41 +0100
Subject: [PATCH] virtcontainers: store: Add an internal backend interface

All Store backends will have to implement that simple interface.

Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>
---
 virtcontainers/store/backend.go    | 50 +++++++++++++++++++++
 virtcontainers/store/filesystem.go | 70 ++++++++++++++++++++++++++++++
 virtcontainers/store/manager.go    | 18 +++++++-
 3 files changed, 136 insertions(+), 2 deletions(-)
 create mode 100644 virtcontainers/store/backend.go
 create mode 100644 virtcontainers/store/filesystem.go

diff --git a/virtcontainers/store/backend.go b/virtcontainers/store/backend.go
new file mode 100644
index 0000000000..f61d2df59a
--- /dev/null
+++ b/virtcontainers/store/backend.go
@@ -0,0 +1,50 @@
+// Copyright (c) 2019 Intel Corporation
+//
+// SPDX-License-Identifier: Apache-2.0
+//
+
+package store
+
+import (
+	"context"
+	"fmt"
+)
+
+type backendType string
+
+const (
+	filesystemBackend backendType = "filesystem"
+)
+
+const (
+	filesystemScheme string = "file"
+)
+
+func schemeToBackendType(scheme string) (backendType, error) {
+	switch scheme {
+	case filesystemScheme:
+		return filesystemBackend, nil
+	}
+
+	return "", fmt.Errorf("Unsupported scheme %s", scheme)
+}
+
+func newBackend(scheme string) (backend, error) {
+	t, err := schemeToBackendType(scheme)
+	if err != nil {
+		return nil, err
+	}
+
+	switch t {
+	case filesystemBackend:
+		return &filesystem{}, nil
+	}
+
+	return nil, fmt.Errorf("Unsupported scheme %s", scheme)
+}
+
+type backend interface {
+	new(ctx context.Context, path string, host string) error
+	load(item Item, data interface{}) error
+	store(item Item, data interface{}) error
+}
diff --git a/virtcontainers/store/filesystem.go b/virtcontainers/store/filesystem.go
new file mode 100644
index 0000000000..56e9aa2fa1
--- /dev/null
+++ b/virtcontainers/store/filesystem.go
@@ -0,0 +1,70 @@
+// Copyright (c) 2019 Intel Corporation
+//
+// SPDX-License-Identifier: Apache-2.0
+//
+
+package store
+
+import (
+	"context"
+
+	opentracing "github.com/opentracing/opentracing-go"
+	"github.com/sirupsen/logrus"
+)
+
+type filesystem struct {
+	ctx context.Context
+
+	path string
+}
+
+// Logger returns a logrus logger appropriate for logging Store filesystem messages
+func (f *filesystem) logger() *logrus.Entry {
+	return storeLog.WithFields(logrus.Fields{
+		"subsystem": "store",
+		"backend":   "filesystem",
+		"path":      f.path,
+	})
+}
+
+func (f *filesystem) trace(name string) (opentracing.Span, context.Context) {
+	if f.ctx == nil {
+		f.logger().WithField("type", "bug").Error("trace called before context set")
+		f.ctx = context.Background()
+	}
+
+	span, ctx := opentracing.StartSpanFromContext(f.ctx, name)
+
+	span.SetTag("subsystem", "store")
+	span.SetTag("type", "filesystem")
+	span.SetTag("path", f.path)
+
+	return span, ctx
+}
+
+func (f *filesystem) new(ctx context.Context, path string, host string) error {
+	f.ctx = ctx
+	f.path = path
+
+	f.logger().Infof("New filesystem store backend for %s", path)
+
+	return nil
+}
+
+func (f *filesystem) load(item Item, data interface{}) error {
+	span, _ := f.trace("load")
+	defer span.Finish()
+
+	span.SetTag("item", item)
+
+	return nil
+}
+
+func (f *filesystem) store(item Item, data interface{}) error {
+	span, _ := f.trace("store")
+	defer span.Finish()
+
+	span.SetTag("item", item)
+
+	return nil
+}
diff --git a/virtcontainers/store/manager.go b/virtcontainers/store/manager.go
index de64b1aea9..cfae50bb49 100644
--- a/virtcontainers/store/manager.go
+++ b/virtcontainers/store/manager.go
@@ -86,6 +86,8 @@ type Store struct {
 	scheme string
 	path   string
 	host   string
+
+	backend backend
 }
 
 type manager struct {
@@ -152,6 +154,18 @@ func New(ctx context.Context, storeURL string) (*Store, error) {
 		host:   u.Host,
 	}
 
+	backend, err := newBackend(s.scheme)
+	if err != nil {
+		return nil, err
+	}
+
+	s.backend = backend
+
+	// Create new backend
+	if err := s.backend.new(ctx, s.path, s.host); err != nil {
+		return nil, err
+	}
+
 	if err := stores.addStore(s); err != nil {
 		return nil, err
 	}
@@ -193,7 +207,7 @@ func (s *Store) Load(item Item, data interface{}) error {
 	s.RLock()
 	defer s.RUnlock()
 
-	return nil
+	return s.backend.load(item, data)
 }
 
 // Store stores a virtcontainers item into a Store.
@@ -206,5 +220,5 @@ func (s *Store) Store(item Item, data interface{}) error {
 	s.Lock()
 	defer s.Unlock()
 
-	return nil
+	return s.backend.store(item, data)
 }