Bug 1221532
| Summary: | SCVMM: "[RuntimeError]: Host not specified, unable to migrate VM Method:[rescue in execute]" on VM migrate | |||
|---|---|---|---|---|
| Product: | Red Hat CloudForms Management Engine | Reporter: | Aziza Karol <akarol> | |
| Component: | UI - OPS | Assignee: | Harpreet Kataria <hkataria> | |
| Status: | CLOSED ERRATA | QA Contact: | Aziza Karol <akarol> | |
| Severity: | medium | Docs Contact: | ||
| Priority: | medium | |||
| Version: | 5.4.0 | CC: | cpelland, hkataria, jhardy, mpovolny, obarenbo | |
| Target Milestone: | GA | Keywords: | ZStream | |
| Target Release: | 5.5.0 | |||
| Hardware: | Unspecified | |||
| OS: | Unspecified | |||
| Whiteboard: | ||||
| Fixed In Version: | 5.5.0.1 | Doc Type: | Bug Fix | |
| Doc Text: | Story Points: | --- | ||
| Clone Of: | ||||
| : | 1243872 (view as bug list) | Environment: | ||
| Last Closed: | 2015-12-08 13:09:19 UTC | Type: | Bug | |
| Regression: | --- | Mount Type: | --- | |
| Documentation: | --- | CRM: | ||
| Verified Versions: | Category: | --- | ||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | ||
| Cloudforms Team: | --- | Target Upstream Version: | ||
| Embargoed: | ||||
| Bug Depends On: | ||||
| Bug Blocks: | 1243872 | |||
|
Description
Aziza Karol
2015-05-14 10:06:51 UTC
Ultimately this is a 'remove the option' until we have this in place. Perhaps we need to add a RFE as well but I think that will happen on its own. New commit detected on manageiq/master: https://github.com/ManageIQ/manageiq/commit/283fe0819e67fbe6e48aee2df0980d4c40774b16 commit 283fe0819e67fbe6e48aee2df0980d4c40774b16 Author: Harpreet Kataria <hkataria> AuthorDate: Fri May 15 10:28:56 2015 -0400 Commit: Harpreet Kataria <hkataria> CommitDate: Wed Jul 15 23:06:26 2015 -0400 Remove Migrate button for SCVMM VMs - Added new model methods, changed code to use is_available? method to determine availability of "migrate" task for VMs similar to power buttons for VMs - Added "validate_migrate" method to VM subclasses and have it return false for Redhat & SCVMM VMs, to hide Migrate button on appropriate VM summary screens. - Added "batch_operation_supported?" method to determine if any of the selected VMs are not allowed to perform migrate task. - Added spec tests around new methods. https://bugzilla.redhat.com/show_bug.cgi?id=1221532 app/controllers/application_controller.rb | 30 ++++++++++++++----- app/helpers/application_helper/toolbar_builder.rb | 4 ++- .../manageiq/providers/amazon/cloud_manager/vm.rb | 4 +++ .../manageiq/providers/redhat/infra_manager/vm.rb | 4 +++ .../manageiq/providers/vmware/infra_manager/vm.rb | 4 +++ app/models/vm_microsoft.rb | 4 +++ app/models/vm_openstack.rb | 4 +++ app/models/vm_or_template.rb | 8 +++++ app/models/vm_xen.rb | 3 ++ spec/controllers/application_controller_spec.rb | 34 ++++++++++++++++++++++ .../application_helper/toolbar_builder_spec.rb | 5 ++-- spec/models/vm_or_template_spec.rb | 26 +++++++++++++++++ 12 files changed, 118 insertions(+), 12 deletions(-) New commit detected on cfme/5.4.z: https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=d546ed7b4bc8a3276360064d182d08a69a62f4e7 commit d546ed7b4bc8a3276360064d182d08a69a62f4e7 Author: Harpreet Kataria <hkataria> AuthorDate: Thu Jul 16 10:13:27 2015 -0400 Commit: Harpreet Kataria <hkataria> CommitDate: Thu Jul 16 13:50:16 2015 -0400 Remove Migrate button for SCVMM VMs - Added new model methods, changed code to use is_available? method to determine availability of "migrate" task for VMs similar to power buttons for VMs - Added "validate_migrate" method to VM subclasses and have it return false for Redhat & SCVMM VMs, to hide Migrate button on appropriate VM summary screens. - Added "batch_operation_supported?" method to determine if any of the selected VMs are not allowed to perform migrate task. - Added spec tests around new methods. https://bugzilla.redhat.com/show_bug.cgi?id=1221532 https://bugzilla.redhat.com/show_bug.cgi?id=1243872 vmdb/app/controllers/application_controller.rb | 30 ++++++++++++++----- vmdb/app/helpers/application_helper.rb | 4 ++- vmdb/app/models/vm.rb | 3 ++ vmdb/app/models/vm_amazon.rb | 4 +++ vmdb/app/models/vm_microsoft.rb | 4 +++ vmdb/app/models/vm_openstack.rb | 4 +++ vmdb/app/models/vm_or_template.rb | 8 +++++ vmdb/app/models/vm_redhat.rb | 4 +++ vmdb/app/models/vm_vmware.rb | 4 +++ vmdb/app/models/vm_xen.rb | 3 ++ .../controllers/application_controller_spec.rb | 34 ++++++++++++++++++++++ vmdb/spec/helpers/application_helper_spec.rb | 31 ++++++++++++++++++-- vmdb/spec/models/vm_or_template_spec.rb | 26 +++++++++++++++++ 13 files changed, 147 insertions(+), 12 deletions(-) New commit detected on cfme/5.4.z: https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=263fc0366a35b74ce93569d1a032527c0c3e08e9 commit 263fc0366a35b74ce93569d1a032527c0c3e08e9 Merge: 577bad3 d546ed7 Author: Dan Clarizio <dclarizi> AuthorDate: Thu Jul 23 11:27:25 2015 -0400 Commit: Dan Clarizio <dclarizi> CommitDate: Thu Jul 23 11:27:25 2015 -0400 Merge branch '54z_bz_1243872' into '5.4.z' Remove Migrate button for SCVMM VMs - Added new model methods, changed code to use is_available? method to determine availability of "migrate" task for VMs similar to power buttons for VMs - Added "validate_migrate" method to VM subclasses and have it return false for Redhat & SCVMM VMs, to hide Migrate button on appropriate VM summary screens. - Added "batch_operation_supported?" method to determine if any of the selected VMs are not allowed to perform migrate task. - Added spec tests around new methods. https://bugzilla.redhat.com/show_bug.cgi?id=1221532 https://bugzilla.redhat.com/show_bug.cgi?id=1243872 @dclarizio please review, had to merge some conflicts manually after applying changes from the patch toolbar code has been moved out of application_helper on upstream codebase, original PR: https://github.com/ManageIQ/manageiq/pull/2928 See merge request !184 vmdb/app/controllers/application_controller.rb | 30 ++++++++++++++----- vmdb/app/helpers/application_helper.rb | 4 ++- vmdb/app/models/vm.rb | 3 ++ vmdb/app/models/vm_amazon.rb | 4 +++ vmdb/app/models/vm_microsoft.rb | 4 +++ vmdb/app/models/vm_openstack.rb | 4 +++ vmdb/app/models/vm_or_template.rb | 8 +++++ vmdb/app/models/vm_redhat.rb | 4 +++ vmdb/app/models/vm_vmware.rb | 4 +++ vmdb/app/models/vm_xen.rb | 3 ++ .../controllers/application_controller_spec.rb | 34 ++++++++++++++++++++++ vmdb/spec/helpers/application_helper_spec.rb | 31 ++++++++++++++++++-- vmdb/spec/models/vm_or_template_spec.rb | 26 +++++++++++++++++ 13 files changed, 147 insertions(+), 12 deletions(-) Detected commit referencing this ticket while ticket status is POST. Detected commit referencing this ticket while ticket status is POST. New commit detected on manageiq/master: https://github.com/ManageIQ/manageiq/commit/808f5b661a0f60c45ab1ba1d5e212d045eb0695a commit 808f5b661a0f60c45ab1ba1d5e212d045eb0695a Author: Harpreet Kataria <hkataria> AuthorDate: Thu Aug 27 12:05:47 2015 -0400 Commit: Harpreet Kataria <hkataria> CommitDate: Thu Aug 27 14:57:59 2015 -0400 Fixed flash message to be more generic. Fixed existing spec test to check for updated flash message. https://bugzilla.redhat.com/show_bug.cgi?id=1221532 https://bugzilla.redhat.com/show_bug.cgi?id=1243872 app/controllers/application_controller.rb | 8 +++++--- spec/controllers/application_controller_spec.rb | 2 +- 2 files changed, 6 insertions(+), 4 deletions(-) New commit detected on cfme/5.4.z: https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=c6277a441b89e6dd7321344e9b20c6b3d498d217 commit c6277a441b89e6dd7321344e9b20c6b3d498d217 Author: Harpreet Kataria <hkataria> AuthorDate: Thu Aug 27 17:32:25 2015 -0400 Commit: Harpreet Kataria <hkataria> CommitDate: Thu Aug 27 17:32:25 2015 -0400 Fixed flash message to be more generic. Fixed existing spec test to check for updated flash message. https://bugzilla.redhat.com/show_bug.cgi?id=1221532 https://bugzilla.redhat.com/show_bug.cgi?id=1243872 vmdb/app/controllers/application_controller.rb | 8 +++++--- vmdb/spec/controllers/application_controller_spec.rb | 2 +- 2 files changed, 6 insertions(+), 4 deletions(-) New commit detected on cfme/5.4.z: https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=3783724bf2e97bd545537858ab7e8b009202a412 commit 3783724bf2e97bd545537858ab7e8b009202a412 Merge: 1196c64 c6277a4 Author: Dan Clarizio <dclarizi> AuthorDate: Fri Aug 28 11:02:23 2015 -0400 Commit: Dan Clarizio <dclarizi> CommitDate: Fri Aug 28 11:02:23 2015 -0400 Merge branch '54z_bz_1243872' into '5.4.z' Fixed flash message to be more generic. Fixed existing spec test to check for updated flash message. https://bugzilla.redhat.com/show_bug.cgi?id=1221532 https://bugzilla.redhat.com/show_bug.cgi?id=1243872 @dclarizio please review, clean path was applied original PR: https://github.com/ManageIQ/manageiq/pull/4079 See merge request !240 vmdb/app/controllers/application_controller.rb | 8 +++++--- vmdb/spec/controllers/application_controller_spec.rb | 2 +- 2 files changed, 6 insertions(+), 4 deletions(-) New commit detected on cfme/5.4.z: https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=c6277a441b89e6dd7321344e9b20c6b3d498d217 commit c6277a441b89e6dd7321344e9b20c6b3d498d217 Author: Harpreet Kataria <hkataria> AuthorDate: Thu Aug 27 17:32:25 2015 -0400 Commit: Harpreet Kataria <hkataria> CommitDate: Thu Aug 27 17:32:25 2015 -0400 Fixed flash message to be more generic. Fixed existing spec test to check for updated flash message. https://bugzilla.redhat.com/show_bug.cgi?id=1221532 https://bugzilla.redhat.com/show_bug.cgi?id=1243872 vmdb/app/controllers/application_controller.rb | 8 +++++--- vmdb/spec/controllers/application_controller_spec.rb | 2 +- 2 files changed, 6 insertions(+), 4 deletions(-) New commit detected on cfme/5.4.z: https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=3783724bf2e97bd545537858ab7e8b009202a412 commit 3783724bf2e97bd545537858ab7e8b009202a412 Merge: 1196c64 c6277a4 Author: Dan Clarizio <dclarizi> AuthorDate: Fri Aug 28 11:02:23 2015 -0400 Commit: Dan Clarizio <dclarizi> CommitDate: Fri Aug 28 11:02:23 2015 -0400 Merge branch '54z_bz_1243872' into '5.4.z' Fixed flash message to be more generic. Fixed existing spec test to check for updated flash message. https://bugzilla.redhat.com/show_bug.cgi?id=1221532 https://bugzilla.redhat.com/show_bug.cgi?id=1243872 @dclarizio please review, clean path was applied original PR: https://github.com/ManageIQ/manageiq/pull/4079 See merge request !240 vmdb/app/controllers/application_controller.rb | 8 +++++--- vmdb/spec/controllers/application_controller_spec.rb | 2 +- 2 files changed, 6 insertions(+), 4 deletions(-) Good to go. Verified in 5.5.0.1-alpha1.20150915151429_2230bd4 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-2015:2551 |