New commit detected on ManageIQ/manageiq/euwe: https://github.com/ManageIQ/manageiq/commit/7898c8e3d270dd503e95fe3614128695bfc40800 commit 7898c8e3d270dd503e95fe3614128695bfc40800 Author: Martin Povolny <mpovolny> AuthorDate: Tue Dec 6 10:05:20 2016 +0100 Commit: Satoe Imaishi <simaishi> CommitDate: Mon Jan 9 12:46:43 2017 -0500 Merge pull request #12768 from borod108/bugs/1396068revert_active No revert for "active" snapshot (cherry picked from commit eb0315daf8d6d10817247ae29bfc4efb08668aba) https://bugzilla.redhat.com/show_bug.cgi?id=1402992 app/helpers/application_helper/toolbar_builder.rb | 1 + 1 file changed, 1 insertion(+)
Verified on CFME-5.7.1.0 & RHV-4.0.5. For a down VM, the revert icon is not active, when the "Active VM" is marked.
Hi Dave, As per the comment in https://bugzilla.redhat.com/show_bug.cgi?id=1396068#c9, I agree with Dan, this fix is valid for rhev vm and not or vmware vm as VMware allows positioning on the active snapshot. For vmware vm in 5.7.1, the revert option is showing as disabled for active vm and this has to be fix. Thanks, Aziza
Should you open another bug for the vmware provider?
My take on it is that your change introduced this regression through this proposed fix and it should stay all within this BZ. Ultimately it needs to be fixed either way, the questions are when and by who? We can certainly open separate BZ if the decision is to defer which I lean towards nacking but would like john to weigh in.
https://github.com/ManageIQ/manageiq/pull/13953
New commit detected on ManageIQ/manageiq/euwe: https://github.com/ManageIQ/manageiq/commit/a2c7e9a60ecf6b85c06a5cf77d5b5628accea830 commit a2c7e9a60ecf6b85c06a5cf77d5b5628accea830 Author: Adam Grare <agrare> AuthorDate: Thu Feb 16 14:51:25 2017 -0500 Commit: Adam Grare <agrare> CommitDate: Thu Feb 16 14:58:57 2017 -0500 Revert "Merge pull request #12768 from borod108/bugs/1396068revert_active" This reverts commit 7898c8e3d270dd503e95fe3614128695bfc40800. https://bugzilla.redhat.com/show_bug.cgi?id=1402992 app/helpers/application_helper/toolbar_builder.rb | 1 - 1 file changed, 1 deletion(-)
https://github.com/ManageIQ/manageiq/pull/14421
PR: https://github.com/ManageIQ/manageiq/pull/14326
For VMware vm,the revert option is showing as enabled for active vm. Verified:5.7.3.1.20170608124102_4b35a9e Ilanit, Would you please verify this on RHV provider?
For RHV vm, the revert option is showing as disabled for active vm. Verified: CFME-5.7.3.1.20170608124102_4b35a9e/RHV-4.1.3
Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://access.redhat.com/errata/RHSA-2017:1601