Skip to content

Commit

Permalink
sanity: use correct path during cleanup
Browse files Browse the repository at this point in the history
kubernetes-csi#159 introduced the
ability to use different paths for each tests via external commands or
callbacks, but it missed two lines in the cleanup code which also have
to use the actual paths instead of the configured paths.
  • Loading branch information
pohly committed Apr 4, 2019
1 parent 7097f9f commit 27a79d5
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions pkg/sanity/cleanup.go
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,7 @@ func (cl *Cleanup) DeleteVolumes() {
ctx,
&csi.NodeUnpublishVolumeRequest{
VolumeId: info.VolumeID,
TargetPath: cl.Context.Config.TargetPath,
TargetPath: cl.Context.targetPath,
},
); err != nil {
logger.Printf("warning: NodeUnpublishVolume: %s", err)
Expand All @@ -99,7 +99,7 @@ func (cl *Cleanup) DeleteVolumes() {
ctx,
&csi.NodeUnstageVolumeRequest{
VolumeId: info.VolumeID,
StagingTargetPath: cl.Context.Config.StagingPath,
StagingTargetPath: cl.Context.stagingPath,
},
); err != nil {
logger.Printf("warning: NodeUnstageVolume: %s", err)
Expand Down

0 comments on commit 27a79d5

Please sign in to comment.