sanity: capture newly created volume IDs

avalluri · avalluri · commit e34aae8e8300 · 2019-04-26T16:17:23.000+03:00
In one of ListVolumes tests, newly created volume ids were not captured. Though
the test has needed cleaning up code, it was affected because of the empty
newVolIDs array. This leads to leaking of volumes.

Signed-off-by: Amarnath Valluri &lt;amarnath.valluri@intel.com&gt;
diff --git a/pkg/sanity/controller.go b/pkg/sanity/controller.go
@@ -319,6 +319,7 @@ var _ = DescribeSanity("Controller Service", func(sc *SanityContext) {
 					vol, err := c.CreateVolume(context.Background(), req)
 					Expect(err).NotTo(HaveOccurred())
 					Expect(vol).NotTo(BeNil())
+					newVolIDs = append(newVolIDs, vol.Volume.VolumeId)
 				}
 
 				// Update the current total vols count.

Original file line number	Diff line number	Diff line change
`@@ -319,6 +319,7 @@ var _ = DescribeSanity("Controller Service", func(sc *SanityContext) {`
`319`	`319`	`vol, err := c.CreateVolume(context.Background(), req)`
`320`	`320`	`Expect(err).NotTo(HaveOccurred())`
`321`	`321`	`Expect(vol).NotTo(BeNil())`
	`322`	`+ newVolIDs = append(newVolIDs, vol.Volume.VolumeId)`
`322`	`323`	`}`
`323`	`324`
`324`	`325`	`// Update the current total vols count.`