TPT-3909: Integration tests for IAM volumes io_ready

test/integration/iam_io_ready_test.go

@@ -0,0 +1,240 @@
+package integration
+
+import (
+	"context"
+	"testing"
+
+	"github.com/linode/linodego"
+	"github.com/stretchr/testify/assert"
+	"github.com/stretchr/testify/require"
+)
+
+func setupVolumeAttachedToLinode(
+	t *testing.T,
+	fixturesYaml string,
+	detachVolume bool,
+) (*linodego.Client, *linodego.Volume, *linodego.Instance, func()) {
+	t.Helper()
+	var fixtureTeardown func()
+
+	client, fixtureTeardown := createTestClient(t, fixturesYaml)
+
+	instance, err := createInstance(t, client, true, func(l *linodego.Client, options *linodego.InstanceCreateOptions) {
+		options.Booted = linodego.Pointer(true)
+	})
+	require.NoErrorf(t, err, "Error setting up Linode instance: %v", err)
+
+	teardownInstance := func() {
+		err = client.DeleteInstance(context.Background(), instance.ID)
+		require.NoErrorf(t, err, "Error deleting instance: %v", err)
+	}
+
+	instance, err = client.WaitForInstanceStatus(context.Background(), instance.ID, linodego.InstanceRunning, 180)
+	require.NoErrorf(t, err, "Error waiting for instance to be running: %v", err)
+
+	volume, teardownVolume, err := createVolume(t, client, func(l *linodego.Client, options *linodego.VolumeCreateOptions) {
+		options.Region = instance.Region
+	})
+	require.NoErrorf(t, err, "Error creating volume: %v", err)
+
+	volume, err = client.AttachVolume(context.Background(), volume.ID, &linodego.VolumeAttachOptions{LinodeID: instance.ID})
+	require.NoErrorf(t, err, "Error attaching volume to instance: %v", err)
+
+	volume, err = client.WaitForVolumeIOReadyStatus(context.Background(), volume.ID, true, 45)
+	require.NoErrorf(t, err, "Error waiting for IO Ready status of attached volume: %v", err)
+
+	teardown := func() {
+		if detachVolume {
+			err = client.DetachVolume(context.Background(), volume.ID)
+			require.NoErrorf(t, err, "Error detaching volume: %v", err)
+
+			_, err = client.WaitForVolumeIOReadyStatus(context.Background(), volume.ID, false, 45)
+			require.NoErrorf(t, err, "Error waiting for IO Ready status of detached volume: %v", err)
+		}
+		teardownVolume()
+		teardownInstance()
+		fixtureTeardown()
+	}
+
+	return client, volume, instance, teardown
+}
+
+func TestIAM_GetIOReadyForNotAttachedVolume(t *testing.T) {
+	client, recordStopper := createTestClient(t, "fixtures/TestIAM_GetIOReadyForNotAttachedVolume")
+	defer recordStopper()
+
+	volume, teardown, err := createVolume(t, client, func(l *linodego.Client, options *linodego.VolumeCreateOptions) {
+		options.Label = label
+	})
+	defer teardown()
+	require.NoErrorf(t, err, "Error creating not attached volume: %v", err)
+
+	volume, err = client.WaitForVolumeStatus(context.Background(), volume.ID, linodego.VolumeActive, 30)
+	require.NoErrorf(t, err, "Error waiting for volume to be active: %v", err)
+
+	volumeList, err := client.ListVolumes(context.Background(), nil)
+	require.NoErrorf(t, err, "Error listing volumes: %v", err)
+	for _, vol := range volumeList {
+		if vol.Label == label {
+			assert.Equal(t, linodego.VolumeActive, volumeList[0].Status)
+			assert.Empty(t, volumeList[0].LinodeID)
+			assert.Empty(t, volumeList[0].LinodeLabel)
+			assert.False(t, volumeList[0].IOReady)
+		}
+	}
+
+	volume, err = client.GetVolume(context.Background(), volume.ID)
+	require.NoErrorf(t, err, "Error getting not attached volume: %v", err)
+	assert.Equal(t, label, volume.Label)
+	assert.Equal(t, linodego.VolumeActive, volume.Status)
+	assert.Empty(t, volume.LinodeID)
+	assert.Empty(t, volume.LinodeLabel)
+	assert.False(t, volume.IOReady)
+}
+
+func TestIAM_GetIOReadyForAttachedDetachedVolume(t *testing.T) {
+	client, volume, instance, teardown := setupVolumeAttachedToLinode(t, "fixtures/TestIAM_GetIOReadyForAttachedDetachedVolume", false)
+	defer teardown()
+
+	instanceVolumes, err := client.ListInstanceVolumes(context.Background(), instance.ID, nil)
+	require.NoErrorf(t, err, "Error listing instance volumes: %v", err)
+	require.Len(t, instanceVolumes, 1, "Expected 1 volume attached to instance, got %d", len(instanceVolumes))
+	assert.Equal(t, instance.ID, *instanceVolumes[0].LinodeID)
+	assert.Equal(t, instance.Label, instanceVolumes[0].LinodeLabel)
+	assert.True(t, instanceVolumes[0].IOReady)
+
+	volume, err = client.GetVolume(context.Background(), volume.ID)
+	require.NoErrorf(t, err, "Error getting attached volume: %v", err)
+	assert.Equal(t, instance.ID, *volume.LinodeID)
+	assert.Equal(t, instance.Label, volume.LinodeLabel)
+	assert.True(t, volume.IOReady)
+
+	err = client.DetachVolume(context.Background(), volume.ID)
+	require.NoErrorf(t, err, "Error detaching volume: %v", err)
+
+	volume, err = client.WaitForVolumeIOReadyStatus(context.Background(), volume.ID, false, 45)
+	require.NoErrorf(t, err, "Error waiting for IO Ready status of detached volume: %v", err)
+
+	instanceVolumes, err = client.ListInstanceVolumes(context.Background(), instance.ID, nil)
+	require.NoErrorf(t, err, "Error listing instance volumes after detach: %v", err)
+	require.Len(t, instanceVolumes, 0, "Expected no volumes attached to instance, got %d", len(instanceVolumes))
+
+	volume, err = client.GetVolume(context.Background(), volume.ID)
+	require.NoErrorf(t, err, "Error getting volume after detach: %v", err)
+	assert.Empty(t, volume.LinodeID)
+	assert.Empty(t, volume.LinodeLabel)
+	assert.False(t, volume.IOReady)
+}
+
+func TestIAM_GetIOReadyForUpdatedVolume(t *testing.T) {
+	client, volume, instance, teardown := setupVolumeAttachedToLinode(t, "fixtures/TestIAM_GetIOReadyForUpdatedVolume", true)
+	defer teardown()
+	assert.Equal(t, instance.ID, *volume.LinodeID)
+	assert.Equal(t, instance.Label, volume.LinodeLabel)
+	assert.NotContains(t, "-updated", volume.Label)
+	assert.Empty(t, volume.Tags)
+	assert.True(t, volume.IOReady)
+
+	labelUpdated := volume.Label + "-updated"
+	tagsUpdated := []string{"updated"}
+
+	updateOpts := linodego.VolumeUpdateOptions{
+		Label: labelUpdated,
+		Tags:  &tagsUpdated,
+	}
+	volume, err := client.UpdateVolume(context.Background(), volume.ID, updateOpts)
+	require.NoErrorf(t, err, "Error updating volume: %v", err)
+
+	instanceVolumes, err := client.ListInstanceVolumes(context.Background(), instance.ID, nil)
+	require.NoErrorf(t, err, "Error listing instance volumes: %v", err)
+	require.Len(t, instanceVolumes, 1, "Expected 1 volume attached to instance, got %d", len(instanceVolumes))
+	assert.Equal(t, instance.ID, *instanceVolumes[0].LinodeID)
+	assert.Equal(t, instance.Label, instanceVolumes[0].LinodeLabel)
+	assert.Equal(t, labelUpdated, instanceVolumes[0].Label)
+	assert.Equal(t, tagsUpdated, instanceVolumes[0].Tags)
+	assert.True(t, instanceVolumes[0].IOReady)
+
+	volume, err = client.GetVolume(context.Background(), volume.ID)
+	require.NoErrorf(t, err, "Error getting updated volume: %v", err)
+	assert.Equal(t, instance.ID, *volume.LinodeID)
+	assert.Equal(t, instance.Label, volume.LinodeLabel)
+	assert.Equal(t, labelUpdated, volume.Label)
+	assert.Equal(t, tagsUpdated, volume.Tags)
+	assert.True(t, volume.IOReady)
+}
+
+func TestIAM_GetIOReadyForClonedVolume(t *testing.T) {
+	t.Skip("Skipping test due to possible API defect - JIRA ticket: STORENGSUP-855")
+
+	client, volume, instance, teardown := setupVolumeAttachedToLinode(t, "fixtures/TestIAM_GetIOReadyForClonedVolume", true)
+	defer teardown()
+	assert.Equal(t, instance.ID, *volume.LinodeID)
+	assert.Equal(t, instance.Label, volume.LinodeLabel)
+	assert.True(t, volume.IOReady)
+
+	labelCloned := volume.Label + "-cloned"
+
+	instanceVolumes, err := client.ListInstanceVolumes(context.Background(), instance.ID, nil)
+	require.NoErrorf(t, err, "Error listing instance volumes: %v", err)
+	require.Len(t, instanceVolumes, 1, "Expected 1 volume attached to instance, got %d", len(instanceVolumes))
+
+	volumeCloned, err := client.CloneVolume(context.Background(), volume.ID, labelCloned)
+	require.NoErrorf(t, err, "Error cloning volume: %v", err)
+
+	instanceVolumes, err = client.ListInstanceVolumes(context.Background(), instance.ID, nil)
+	require.NoErrorf(t, err, "Error listing instance volumes: %v", err)
+	require.Len(t, instanceVolumes, 2, "Expected 2 volumes attached to instance, got %d", len(instanceVolumes))
+	for _, vol := range instanceVolumes {
+		assert.Equal(t, instance.ID, *vol.LinodeID)
+		assert.Equal(t, instance.Label, vol.LinodeLabel)
+		assert.True(t, vol.IOReady)
+	}
+
+	volumeCloned, err = client.GetVolume(context.Background(), volumeCloned.ID)
+	require.NoErrorf(t, err, "Error getting cloned volume: %v", err)
+	assert.Equal(t, labelCloned, volumeCloned.Label)
+	assert.Equal(t, instance.ID, *volumeCloned.LinodeID)
+	assert.Equal(t, instance.Label, volumeCloned.LinodeLabel)
+	assert.True(t, volume.IOReady)
+
+	// Cleaning the cloned volume
+	err = client.DetachVolume(context.Background(), volumeCloned.ID)
+	require.NoErrorf(t, err, "Error detaching cloned volume: %v", err)
+
+	_, err = client.WaitForVolumeIOReadyStatus(context.Background(), volume.ID, false, 15)
+	require.NoErrorf(t, err, "Error waiting for IO Ready status of detached volume: %v", err)
+
+	err = client.DeleteVolume(context.Background(), volumeCloned.ID)
+	require.NoErrorf(t, err, "Error deleting cloned volume: %v", err)
+}
+
+func TestIAM_GetIOReadyForResizedVolume(t *testing.T) {
+	client, volume, instance, teardown := setupVolumeAttachedToLinode(t, "fixtures/TestIAM_GetIOReadyForResizedVolume", true)
+	defer teardown()
+	assert.Equal(t, instance.ID, *volume.LinodeID)
+	assert.Equal(t, instance.Label, volume.LinodeLabel)
+	assert.True(t, volume.IOReady)
+
+	newSize := volume.Size + 10
+
+	err := client.ResizeVolume(context.Background(), volume.ID, newSize)
+	require.NoErrorf(t, err, "Error resizing volume: %v", err)
+
+	_, err = client.WaitForVolumeStatus(context.Background(), volume.ID, linodego.VolumeActive, 30)
+	require.NoErrorf(t, err, "Error waiting for volume to be active: %v", err)
+
+	instanceVolumes, err := client.ListInstanceVolumes(context.Background(), instance.ID, nil)
+	require.NoErrorf(t, err, "Error listing instance volumes: %v", err)
+	require.Len(t, instanceVolumes, 1, "Expected 1 volume attached to instance, got %d", len(instanceVolumes))
+	assert.Equal(t, instance.ID, *instanceVolumes[0].LinodeID)
+	assert.Equal(t, instance.Label, instanceVolumes[0].LinodeLabel)
+	assert.Equal(t, newSize, instanceVolumes[0].Size)
+	assert.True(t, instanceVolumes[0].IOReady)
+
+	volume, err = client.GetVolume(context.Background(), volume.ID)
+	require.NoErrorf(t, err, "Error getting updated volume: %v", err)
+	assert.Equal(t, instance.ID, *volume.LinodeID)
+	assert.Equal(t, instance.Label, volume.LinodeLabel)
+	assert.Equal(t, newSize, volume.Size)
+	assert.True(t, volume.IOReady)
+}

test/integration/volumes_test.go

@@ -244,6 +244,9 @@ func TestVolume_Update(t *testing.T) {
 	if volume.Label != updatedLabel {
 		t.Errorf("Expected volume label to be equal to updated volume label")
 	}
+	if volume.IOReady {
+		t.Errorf("Expected volume IOReady to be False, got True")
+	}
 	assertDateSet(t, volume.Created)
 	assertDateSet(t, volume.Updated)
 }

waitfor.go

@@ -820,3 +820,29 @@ func (client Client) WaitForAlertDefinitionStatus(
 		}
 	}
 }
+
+func (client Client) WaitForVolumeIOReadyStatus(ctx context.Context, volumeID int, status bool, timeoutSeconds int) (*Volume, error) {
+	ctx, cancel := context.WithTimeout(ctx, time.Duration(timeoutSeconds)*time.Second)
+	defer cancel()
+
+	ticker := time.NewTicker(client.pollInterval)
+	defer ticker.Stop()
+
+	for {
+		select {
+		case <-ticker.C:
+			volume, err := client.GetVolume(ctx, volumeID)
+			if err != nil {
+				return volume, err
+			}
+
+			complete := (volume.IOReady == status)
+
+			if complete {
+				return volume, nil
+			}
+		case <-ctx.Done():
+			return nil, fmt.Errorf("failed to wait for Volume %d IO Ready status %t: %w", volumeID, status, ctx.Err())
+		}
+	}
+}