123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331 |
- /*
- Copyright 2016 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 photon_pd
- import (
- "context"
- "errors"
- "testing"
- "k8s.io/api/core/v1"
- "k8s.io/kubernetes/pkg/cloudprovider/providers/photon"
- "k8s.io/kubernetes/pkg/volume"
- volumetest "k8s.io/kubernetes/pkg/volume/testing"
- "k8s.io/apimachinery/pkg/types"
- "k8s.io/klog"
- )
- func TestGetDeviceName_Volume(t *testing.T) {
- plugin := newPlugin()
- name := "my-photon-volume"
- spec := createVolSpec(name, false)
- deviceName, err := plugin.GetVolumeName(spec)
- if err != nil {
- t.Errorf("GetDeviceName error: %v", err)
- }
- if deviceName != name {
- t.Errorf("GetDeviceName error: expected %s, got %s", name, deviceName)
- }
- }
- func TestGetDeviceName_PersistentVolume(t *testing.T) {
- plugin := newPlugin()
- name := "my-photon-pv"
- spec := createPVSpec(name, true)
- deviceName, err := plugin.GetVolumeName(spec)
- if err != nil {
- t.Errorf("GetDeviceName error: %v", err)
- }
- if deviceName != name {
- t.Errorf("GetDeviceName error: expected %s, got %s", name, deviceName)
- }
- }
- // One testcase for TestAttachDetach table test below
- type testcase struct {
- name string
- // For fake Photon cloud provider:
- attach attachCall
- detach detachCall
- diskIsAttached diskIsAttachedCall
- t *testing.T
- // Actual test to run
- test func(test *testcase) (string, error)
- // Expected return of the test
- expectedDevice string
- expectedError error
- }
- func TestAttachDetach(t *testing.T) {
- diskName := "000-000-000"
- nodeName := types.NodeName("instance")
- readOnly := false
- spec := createVolSpec(diskName, readOnly)
- detachError := errors.New("Fake detach error")
- diskCheckError := errors.New("Fake DiskIsAttached error")
- tests := []testcase{
- // Successful Attach call
- {
- name: "Attach_Positive",
- diskIsAttached: diskIsAttachedCall{diskName, nodeName, false, diskCheckError},
- attach: attachCall{diskName, nodeName, nil},
- test: func(testcase *testcase) (string, error) {
- attacher := newAttacher(testcase)
- return attacher.Attach(spec, nodeName)
- },
- expectedDevice: "/dev/disk/by-id/wwn-0x000000000",
- },
- // Disk is already attached
- {
- name: "Attach_Positive_AlreadyAttached",
- diskIsAttached: diskIsAttachedCall{diskName, nodeName, false, diskCheckError},
- attach: attachCall{diskName, nodeName, nil},
- test: func(testcase *testcase) (string, error) {
- attacher := newAttacher(testcase)
- return attacher.Attach(spec, nodeName)
- },
- expectedDevice: "/dev/disk/by-id/wwn-0x000000000",
- },
- // Detach succeeds
- {
- name: "Detach_Positive",
- diskIsAttached: diskIsAttachedCall{diskName, nodeName, true, nil},
- detach: detachCall{diskName, nodeName, nil},
- test: func(testcase *testcase) (string, error) {
- detacher := newDetacher(testcase)
- return "", detacher.Detach(diskName, nodeName)
- },
- },
- // Disk is already detached
- {
- name: "Detach_Positive_AlreadyDetached",
- diskIsAttached: diskIsAttachedCall{diskName, nodeName, false, nil},
- test: func(testcase *testcase) (string, error) {
- detacher := newDetacher(testcase)
- return "", detacher.Detach(diskName, nodeName)
- },
- },
- // Detach succeeds when DiskIsAttached fails
- {
- name: "Detach_Positive_CheckFails",
- diskIsAttached: diskIsAttachedCall{diskName, nodeName, false, diskCheckError},
- detach: detachCall{diskName, nodeName, nil},
- test: func(testcase *testcase) (string, error) {
- detacher := newDetacher(testcase)
- return "", detacher.Detach(diskName, nodeName)
- },
- },
- // Detach fails
- {
- name: "Detach_Negative",
- diskIsAttached: diskIsAttachedCall{diskName, nodeName, false, diskCheckError},
- detach: detachCall{diskName, nodeName, detachError},
- test: func(testcase *testcase) (string, error) {
- detacher := newDetacher(testcase)
- return "", detacher.Detach(diskName, nodeName)
- },
- expectedError: detachError,
- },
- }
- for _, testcase := range tests {
- testcase.t = t
- device, err := testcase.test(&testcase)
- if err != testcase.expectedError {
- t.Errorf("%s failed: expected err=%q, got %q", testcase.name, testcase.expectedError.Error(), err.Error())
- }
- if device != testcase.expectedDevice {
- t.Errorf("%s failed: expected device=%q, got %q", testcase.name, testcase.expectedDevice, device)
- }
- t.Logf("Test %q succeeded", testcase.name)
- }
- }
- // newPlugin creates a new gcePersistentDiskPlugin with fake cloud, NewAttacher
- // and NewDetacher won't work.
- func newPlugin() *photonPersistentDiskPlugin {
- host := volumetest.NewFakeVolumeHost("/tmp", nil, nil)
- plugins := ProbeVolumePlugins()
- plugin := plugins[0]
- plugin.Init(host)
- return plugin.(*photonPersistentDiskPlugin)
- }
- func newAttacher(testcase *testcase) *photonPersistentDiskAttacher {
- return &photonPersistentDiskAttacher{
- host: nil,
- photonDisks: testcase,
- }
- }
- func newDetacher(testcase *testcase) *photonPersistentDiskDetacher {
- return &photonPersistentDiskDetacher{
- photonDisks: testcase,
- }
- }
- func createVolSpec(name string, readOnly bool) *volume.Spec {
- return &volume.Spec{
- Volume: &v1.Volume{
- VolumeSource: v1.VolumeSource{
- PhotonPersistentDisk: &v1.PhotonPersistentDiskVolumeSource{
- PdID: name,
- },
- },
- },
- }
- }
- func createPVSpec(name string, readOnly bool) *volume.Spec {
- return &volume.Spec{
- PersistentVolume: &v1.PersistentVolume{
- Spec: v1.PersistentVolumeSpec{
- PersistentVolumeSource: v1.PersistentVolumeSource{
- PhotonPersistentDisk: &v1.PhotonPersistentDiskVolumeSource{
- PdID: name,
- },
- },
- },
- },
- }
- }
- // Fake PhotonPD implementation
- type attachCall struct {
- diskName string
- nodeName types.NodeName
- ret error
- }
- type detachCall struct {
- diskName string
- nodeName types.NodeName
- ret error
- }
- type diskIsAttachedCall struct {
- diskName string
- nodeName types.NodeName
- isAttached bool
- ret error
- }
- func (testcase *testcase) AttachDisk(ctx context.Context, diskName string, nodeName types.NodeName) error {
- expected := &testcase.attach
- if expected.diskName == "" && expected.nodeName == "" {
- // testcase.attach looks uninitialized, test did not expect to call
- // AttachDisk
- testcase.t.Errorf("Unexpected AttachDisk call!")
- return errors.New("Unexpected AttachDisk call!")
- }
- if expected.diskName != diskName {
- testcase.t.Errorf("Unexpected AttachDisk call: expected diskName %s, got %s", expected.diskName, diskName)
- return errors.New("Unexpected AttachDisk call: wrong diskName")
- }
- if expected.nodeName != nodeName {
- testcase.t.Errorf("Unexpected AttachDisk call: expected nodeName %s, got %s", expected.nodeName, nodeName)
- return errors.New("Unexpected AttachDisk call: wrong nodeName")
- }
- klog.V(4).Infof("AttachDisk call: %s, %s, returning %v", diskName, nodeName, expected.ret)
- return expected.ret
- }
- func (testcase *testcase) DetachDisk(ctx context.Context, diskName string, nodeName types.NodeName) error {
- expected := &testcase.detach
- if expected.diskName == "" && expected.nodeName == "" {
- // testcase.detach looks uninitialized, test did not expect to call
- // DetachDisk
- testcase.t.Errorf("Unexpected DetachDisk call!")
- return errors.New("Unexpected DetachDisk call!")
- }
- if expected.diskName != diskName {
- testcase.t.Errorf("Unexpected DetachDisk call: expected diskName %s, got %s", expected.diskName, diskName)
- return errors.New("Unexpected DetachDisk call: wrong diskName")
- }
- if expected.nodeName != nodeName {
- testcase.t.Errorf("Unexpected DetachDisk call: expected nodeName %s, got %s", expected.nodeName, nodeName)
- return errors.New("Unexpected DetachDisk call: wrong nodeName")
- }
- klog.V(4).Infof("DetachDisk call: %s, %s, returning %v", diskName, nodeName, expected.ret)
- return expected.ret
- }
- func (testcase *testcase) DiskIsAttached(ctx context.Context, diskName string, nodeName types.NodeName) (bool, error) {
- expected := &testcase.diskIsAttached
- if expected.diskName == "" && expected.nodeName == "" {
- // testcase.diskIsAttached looks uninitialized, test did not expect to
- // call DiskIsAttached
- testcase.t.Errorf("Unexpected DiskIsAttached call!")
- return false, errors.New("Unexpected DiskIsAttached call!")
- }
- if expected.diskName != diskName {
- testcase.t.Errorf("Unexpected DiskIsAttached call: expected diskName %s, got %s", expected.diskName, diskName)
- return false, errors.New("Unexpected DiskIsAttached call: wrong diskName")
- }
- if expected.nodeName != nodeName {
- testcase.t.Errorf("Unexpected DiskIsAttached call: expected nodeName %s, got %s", expected.nodeName, nodeName)
- return false, errors.New("Unexpected DiskIsAttached call: wrong nodeName")
- }
- klog.V(4).Infof("DiskIsAttached call: %s, %s, returning %v, %v", diskName, nodeName, expected.isAttached, expected.ret)
- return expected.isAttached, expected.ret
- }
- func (testcase *testcase) DisksAreAttached(ctx context.Context, diskNames []string, nodeName types.NodeName) (map[string]bool, error) {
- return nil, errors.New("Not implemented")
- }
- func (testcase *testcase) CreateDisk(volumeOptions *photon.VolumeOptions) (volumeName string, err error) {
- return "", errors.New("Not implemented")
- }
- func (testcase *testcase) DeleteDisk(volumeName string) error {
- return errors.New("Not implemented")
- }
- func (testcase *testcase) GetVolumeLabels(volumeName string) (map[string]string, error) {
- return map[string]string{}, errors.New("Not implemented")
- }
- func (testcase *testcase) GetDiskPath(volumeName string) (string, error) {
- return "", errors.New("Not implemented")
- }
|