Description of problem: Version-Release number of selected component (if applicable):5.9.0.15.20180103231404_ee61a82 How reproducible:100% Steps to Reproduce: 1. Deploy VM in SCVMM which has disk located on fileshare. 2. Perform SSA on it. 3. Actual results:Error in evm.log "Unable to mount filesystem. Reason:[No root filesystem found." Expected results: It should scan VM without any error. Additional info:
Jerry this error looks similar to the ones seen in some Azure environments. Sending your way in case it's the same error.
Totally unrelated. This is an SCVMM issue. A powershell command to get the length of the disk to be scanned failed because, at first glance at the log, the device file does not exist.
The actual issue here is that the proper credentials were not provided, and then the error that was reported when the Smartstate Scan failed was not meaningful. We will fix so that the issue with the improper credentials will be propagated to the UI.
https://github.com/ManageIQ/manageiq-smartstate/pull/53 has been added to fix this problem. It should also be fixed in Gaprindashvili in manageiq-smartstate, and in Fine in manageiq-gems-pending.
New commit detected on ManageIQ/manageiq-smartstate/master: https://github.com/ManageIQ/manageiq-smartstate/commit/8bf70f07fc1e769c79b7b6f53a51b2e5f24ce329 commit 8bf70f07fc1e769c79b7b6f53a51b2e5f24ce329 Author: Jerry Keselman <jkeselma> AuthorDate: Mon Feb 19 14:56:37 2018 -0500 Commit: Jerry Keselman <jkeselma> CommitDate: Mon Feb 19 14:58:53 2018 -0500 Show Improper Credentials Message in Smartstate Failure in UI When the incorrect domain credentials are provided and a Virtual Machine disk is located in a network accessible disk share, access to the virtual disk may be denied. Previously the error displayed in the UI for a failure of this type was not meaningful. This change will show the error as related to the credentials. Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1534352 lib/Scvmm/miq_hyperv_disk.rb | 3 ++- lib/disk/DiskProbe.rb | 4 ++++ 2 files changed, 6 insertions(+), 1 deletion(-)
Closing this as its already been verified in two z-streams and has test coverage around it.