rmvm unable to purge raw type VM images

[whowutwut]

[root@c910f02c05p03 ~]# chdef fs2vm113 vmstorageformat=
1 object definitions have been created or modified.
[root@c910f02c05p03 ~]# mkvm fs2vm113 -s 10g
[root@c910f02c05p03 ~]# xdsh fs2 ls /var/lib/libvirt/images/fs2vm113*
fs2: /var/lib/libvirt/images/fs2vm113.sda.qcow2
[root@c910f02c05p03 ~]# rmvm -p -f fs2vm113
[root@c910f02c05p03 ~]# xdsh fs2 ls /var/lib/libvirt/images/fs2vm113*
fs2: ls: cannot access /var/lib/libvirt/images/fs2vm113*: No such file or directory
[root@c910f02c05p03 ~]#

Changing to not be .raw ... and default to .qcow2, it works...

[root@c910f02c05p03 ~]# chdef fs2vm113 vmstorageformat=
1 object definitions have been created or modified.
[root@c910f02c05p03 ~]# mkvm fs2vm113 -s 10g
[root@c910f02c05p03 ~]# xdsh fs2 ls /var/lib/libvirt/images/fs2vm113*
fs2: /var/lib/libvirt/images/fs2vm113.sda.qcow2
[root@c910f02c05p03 ~]# rmvm -p -f fs2vm113
[root@c910f02c05p03 ~]# xdsh fs2 ls /var/lib/libvirt/images/fs2vm113*
fs2: ls: cannot access /var/lib/libvirt/images/fs2vm113*: No such file or directory
[root@c910f02c05p03 ~]#

[immarvin]

hi @whowutwut , what is the original definition of fs2vm113? seems the commands in the success case and fail case are the same

[gurevichmark]

@immarvin and @whowutwut
I think we are not purging .raw on purpose.
Take a look at #2631
Seems like we put a message in the log file, but not on the screen.

[immarvin]

hi @gurevichmark , thanks for the information, is it possible to print a warning message on the screen when user tries to purge a vm with raw devices as its disk? use
xCAT::MsgUtils->message('WS',"xxxxxx") instead of xCAT::MsgUtils->trace

[gurevichmark]

It is certainly possible to do that. The reason why I decided to put the message into a log file, is because I felt it might be confusing to the user to see that message on the screen. The user might think: "If the disk for this vm was created outside of xCAT, why is xCAT notifying me about not removing it ?"

[whowutwut]

@immarvin I think I copied and pasted incorrectly...

I will double check this tomorrow.

[zet809]

Hi, @whowutwut , have this issue been verified? Shall we close this issue?

[immarvin]

hi @gurevichmark , any update on this?

[gurevichmark]

After talking with @whowutwut we decided to make the message display on the console, instead of "hiding" it in the log file.