Skip to content

Commit

Permalink
Merge pull request #5 from jsafrane/add-csi
Browse files Browse the repository at this point in the history
Add simple CSI attach handler
  • Loading branch information
jsafrane authored Nov 3, 2017
2 parents 572c5c3 + 5db9f7a commit d785fb5
Show file tree
Hide file tree
Showing 8 changed files with 982 additions and 69 deletions.
9 changes: 8 additions & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,12 @@

.PHONY: all csi-attacher clean test

ifdef V
TESTARGS = -v -args -alsologtostderr -v 5
else
TESTARGS =
endif

all: csi-attacher

csi-attacher:
Expand All @@ -23,4 +29,5 @@ clean:
-rm -rf csi-attacher

test:
go test `go list ./... | grep -v 'vendor'`
go test `go list ./... | grep -v 'vendor'` $(TESTARGS)
go vet `go list ./... | grep -v vendor`
17 changes: 7 additions & 10 deletions pkg/controller/controller.go
Original file line number Diff line number Diff line change
Expand Up @@ -51,9 +51,8 @@ type CSIAttachController struct {
// Handler is responsible for handling VolumeAttachment events from informer.
type Handler interface {
// SyncNewOrUpdatedVolumeAttachment processes one Add/Updated event from
// VolumeAttachment informers. It runs in a workqueue and should be
// reasonably fast (i.e. talking to API server is OK, talking to CSI is
// not).
// VolumeAttachment informers. It runs in a workqueue, guaranting that only
// one SyncNewOrUpdatedVolumeAttachment runs for given VA.
// SyncNewOrUpdatedVolumeAttachment is responsible for marking the
// VolumeAttachment either as forgotten (resets exponential backoff) or
// re-queue it into the provided queue to process it after exponential
Expand Down Expand Up @@ -130,21 +129,19 @@ func (ctrl *CSIAttachController) processNextWorkItem() bool {
defer ctrl.queue.Done(key)

vaName := key.(string)
glog.V(4).Infof("work for VolumeAttachment %s started", vaName)
glog.V(4).Infof("Started processing %q", vaName)

// get VolumeAttachment to process
va, err := ctrl.vaLister.Get(vaName)
if err != nil {
if apierrs.IsNotFound(err) {
// VolumeAttachment was deleted in the meantime, ignore.
// This will remove the VolumeAttachment from queue.
glog.V(4).Infof("%s deleted, ignoring", vaName)
glog.V(3).Infof("%q deleted, ignoring", vaName)
return true
}
if err != nil {
glog.Errorf("Error getting VolumeAttachment %s: %v", vaName, err)
ctrl.queue.AddRateLimited(vaName)
}
glog.Errorf("Error getting VolumeAttachment %q: %v", vaName, err)
ctrl.queue.AddRateLimited(vaName)
return true
}
if va.Spec.Attacher != ctrl.attacherName {
glog.V(4).Infof("Skipping VolumeAttachment %s for attacher %s", va.Name, va.Spec.Attacher)
Expand Down
254 changes: 254 additions & 0 deletions pkg/controller/csi_handler.go
Original file line number Diff line number Diff line change
@@ -0,0 +1,254 @@
/*
Copyright 2017 The Kubernetes Authors.
Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
You may obtain a copy of the License at
http://www.apache.org/licenses/LICENSE-2.0
Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
See the License for the specific language governing permissions and
limitations under the License.
*/

package controller

import (
"context"
"fmt"

"github.com/golang/glog"

storagev1 "k8s.io/api/storage/v1"
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
"k8s.io/client-go/kubernetes"
corelister "k8s.io/client-go/listers/core/v1"
"k8s.io/client-go/util/workqueue"

"github.com/kubernetes-csi/external-attacher-csi/pkg/connection"
)

// csiHandler is a handler that calls CSI to attach/detach volume.
// It adds finalizer to VolumeAttachment instance to make sure they're detached
// before deletion.
type csiHandler struct {
client kubernetes.Interface
attacherName string
csiConnection connection.CSIConnection
pvLister corelister.PersistentVolumeLister
nodeLister corelister.NodeLister
}

var _ Handler = &csiHandler{}

func NewCSIHandler(
client kubernetes.Interface,
attacherName string,
csiConnection connection.CSIConnection,
pvLister corelister.PersistentVolumeLister,
nodeLister corelister.NodeLister) Handler {

return &csiHandler{
client,
attacherName,
csiConnection,
pvLister,
nodeLister,
}
}

func (h *csiHandler) SyncNewOrUpdatedVolumeAttachment(va *storagev1.VolumeAttachment, queue workqueue.RateLimitingInterface) {
glog.V(4).Infof("CSIHandler: processing %q", va.Name)

var err error
if va.DeletionTimestamp == nil {
err = h.syncAttach(va)
} else {
err = h.syncDetach(va)
}
if err != nil {
// Re-queue with exponential backoff
glog.V(2).Infof("Error processing %q: %s", va.Name, err)
queue.AddRateLimited(va.Name)
return
}
// The operation has finished successfully, reset exponential backoff
queue.Forget(va.Name)
glog.V(4).Infof("CSIHandler: finished processing %q", va.Name)
}

func (h *csiHandler) syncAttach(va *storagev1.VolumeAttachment) error {
glog.V(4).Infof("Starting attach operation for %q", va.Name)
va, err := h.addVAFinalizer(va)
if err != nil {
return fmt.Errorf("could not add finalizer: %s", err)
}

if va.Status.Attached {
// Volume is attached, there is nothing to be done.
glog.V(4).Infof("%q is already attached", va.Name)
return nil
}

// Attach
glog.V(2).Infof("Attaching %q", va.Name)
metadata, err := h.csiAttach(va)
if err != nil {
var saveErr error
va, saveErr = h.saveAttachError(va, err)
if saveErr != nil {
// Just log it, propagate the attach error.
glog.V(2).Infof("Failed to save attach error to %q: %s", va.Name, saveErr.Error())
}
// Add context to the error for logging
err := fmt.Errorf("failed to attach: %s", err)
return err
}
glog.V(2).Infof("Attached %q", va.Name)

// Mark as attached
if _, err := markAsAttached(h.client, va, metadata); err != nil {
return fmt.Errorf("failed to mark as attached: %s", err)
}
glog.V(4).Infof("Fully attached %q", va.Name)
return nil
}

func (h *csiHandler) syncDetach(va *storagev1.VolumeAttachment) error {
glog.V(4).Infof("Starting detach operation for %q", va.Name)
if !h.hasVAFinalizer(va) {
glog.V(4).Infof("%q is already detached", va.Name)
return nil
}

glog.V(2).Infof("Detaching %q", va.Name)
if err := h.csiDetach(va); err != nil {
var saveErr error
va, saveErr = h.saveDetachError(va, err)
if saveErr != nil {
// Just log it, propagate the detach error.
glog.V(2).Infof("Failed to save detach error to %q: %s", va.Name, saveErr.Error())
}
// Add context to the error for logging
err := fmt.Errorf("failed to detach: %s", err)
return err
}
glog.V(2).Infof("Detached %q", va.Name)

if _, err := markAsDetached(h.client, va); err != nil {
return fmt.Errorf("could not mark as detached: %s", err)
}
glog.V(4).Infof("Fully detached %q", va.Name)
return nil
}

func (h *csiHandler) addVAFinalizer(va *storagev1.VolumeAttachment) (*storagev1.VolumeAttachment, error) {
finalizerName := getFinalizerName(h.attacherName)
for _, f := range va.Finalizers {
if f == finalizerName {
// Finalizer is already present
glog.V(4).Infof("Finalizer is already set on %q", va.Name)
return va, nil
}
}

// Finalizer is not present, add it
glog.V(4).Infof("Adding finalizer to %q", va.Name)
clone := va.DeepCopy()
clone.Finalizers = append(clone.Finalizers, finalizerName)
// TODO: use patch to save us from VersionError
newVA, err := h.client.StorageV1().VolumeAttachments().Update(clone)
if err != nil {
return va, err
}
glog.V(4).Infof("Finalizer added to %q", va.Name)
return newVA, nil
}

func (h *csiHandler) hasVAFinalizer(va *storagev1.VolumeAttachment) bool {
finalizerName := getFinalizerName(h.attacherName)
for _, f := range va.Finalizers {
if f == finalizerName {
return true
}
}
return false
}

func (h *csiHandler) csiAttach(va *storagev1.VolumeAttachment) (map[string]string, error) {
if va.Spec.PersistentVolumeName == nil {
return nil, fmt.Errorf("VolumeAttachment.spec.persistentVolumeName is empty")
}

pv, err := h.pvLister.Get(*va.Spec.PersistentVolumeName)
if err != nil {
return nil, err
}
node, err := h.nodeLister.Get(va.Spec.NodeName)
if err != nil {
return nil, err
}

ctx := context.TODO()
publishInfo, err := h.csiConnection.Attach(ctx, pv, node)
if err != nil {
return nil, err
}

return publishInfo, nil
}

func (h *csiHandler) csiDetach(va *storagev1.VolumeAttachment) error {
if va.Spec.PersistentVolumeName == nil {
return fmt.Errorf("VolumeAttachment.spec.persistentVolumeName is empty")
}

pv, err := h.pvLister.Get(*va.Spec.PersistentVolumeName)
if err != nil {
return err
}
node, err := h.nodeLister.Get(va.Spec.NodeName)
if err != nil {
return err
}

ctx := context.TODO()
if err := h.csiConnection.Detach(ctx, pv, node); err != nil {
return err
}

return nil
}

func (h *csiHandler) saveAttachError(va *storagev1.VolumeAttachment, err error) (*storagev1.VolumeAttachment, error) {
glog.V(4).Infof("Saving attach error to %q", va.Name)
clone := va.DeepCopy()
clone.Status.AttachError = &storagev1.VolumeError{
Message: err.Error(),
Time: metav1.Now(),
}
newVa, err := h.client.StorageV1().VolumeAttachments().Update(clone)
if err != nil {
return va, err
}
glog.V(4).Infof("Saved attach error to %q", va.Name)
return newVa, nil
}

func (h *csiHandler) saveDetachError(va *storagev1.VolumeAttachment, err error) (*storagev1.VolumeAttachment, error) {
glog.V(4).Infof("Saving detach error to %q", va.Name)
clone := va.DeepCopy()
clone.Status.DetachError = &storagev1.VolumeError{
Message: err.Error(),
Time: metav1.Now(),
}
newVa, err := h.client.StorageV1().VolumeAttachments().Update(clone)
if err != nil {
return va, err
}
glog.V(4).Infof("Saved detach error to %q", va.Name)
return newVa, nil
}
Loading

0 comments on commit d785fb5

Please sign in to comment.