Fix unmount existing EBS

When unmount existing EBS, `blkid -c /dev/null #{device}` return PTUUID instead of UUID. UUID is required for chef resource to unmount and disable volume. Siwtch to use unmount command and remove line from fstab instead of using chef mount resource to avoid using UUID during unmount.

Signed-off-by: chenwany <[email protected]>

Author: chenwany

cookbooks/aws-parallelcluster-config/resources/manage_ebs.rb

@@ -135,7 +135,7 @@
 
   vol_array.each_with_index do |volumeid, index|
     dev_path[index] = "/dev/disk/by-ebs-volumeid/#{volumeid}"
-    dev_uuids[index] = get_uuid(dev_path[index])
+    dev_uuids[index] = get_uuid_for_unmount(shared_dir_array[index])
 
     # Unmount and remove volume from /etc/fstab
     mount shared_dir_array[index] do
@@ -170,11 +170,18 @@
     # unexport the volume
     delete_lines "remove volume from /etc/exports" do
       path "/etc/exports"
-      pattern "#{shared_dir_array[index]} *"
+      pattern "#{shared_dir_array[index]} "
     end
   end
 
   execute "unexport volume" do
     command "exportfs -ra"
   end
 end
+
+action_class do
+  def get_uuid_for_unmount(mount_dir)
+    cmd = Mixlib::ShellOut.new("lsblk -f | grep #{mount_dir} | awk '{{print $3}}'")
+    cmd.run_command.stdout.strip
+  end
+end