New commit detected on ManageIQ/manageiq-ui-classic/gaprindashvili: https://github.com/ManageIQ/manageiq-ui-classic/commit/29f756dea3e512d389cfa661f6b5c54fd45bc82b commit 29f756dea3e512d389cfa661f6b5c54fd45bc82b Author: Milan Zázrivec <mzazrivec> AuthorDate: Mon Dec 4 08:42:15 2017 +0100 Commit: Satoe Imaishi <simaishi> CommitDate: Mon Dec 4 11:35:17 2017 -0500 Merge pull request #2890 from mansam/fix-provisioning-workflow-volume-form Fix loop conditional on provisioning workflow volume form (cherry picked from commit 45d70e83af28dc3e9bed36cd45b8de7b6503bfed) https://bugzilla.redhat.com/show_bug.cgi?id=1520540 app/views/miq_request/_prov_dialog_volume_fieldset.html.haml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
The fix is half correct Please check the video attached to see exact behavior. If you go to create volumes, create multiple volumes, and order them, it works. But if you create multiple volumes, and go back to a different tab, and come back, only the first volume is visible. with this only 1 volume seen, created the request to provision, the output from `rails c` : MiqRequest.find(1) #<MiqProvisionRequest id: 1, description: "Provision from [cirros] to [vparekh_test]", approval_state: "approved", type: "MiqProvisionRequest", created_on: "2017-12-12 08:53:23", updated_on: "2017-12-12 08:54:20", fulfilled_on: nil, requester_id: 1, requester_name: "Administrator", request_type: "template", request_state: "active", message: "[EVM] VM [vparekh_test] Step [CheckProvisioned] St...", status: "Ok", options: {:initial_pass=>true, :src_vm_id=>[2, "cirros"], :miq_request_dialog_name=>"miq_provision_openstack_dialogs_template", :customize_enabled=>["enabled"], :current_tab_key=>:requester, :owner_phone=>nil, :owner_country=>nil, :owner_phone_mobile=>nil, :owner_title=>nil, :owner_first_name=>nil, :owner_manager=>nil, :owner_address=>nil, :owner_company=>nil, :owner_last_name=>nil, :owner_manager_mail=>nil, :owner_city=>nil, :owner_department=>nil, :owner_load_ldap=>nil, :owner_manager_phone=>nil, :owner_state=>nil, :owner_office=>nil, :owner_zip=>nil, :owner_email=>"vparekh", :request_notes=>nil, :vm_tags=>[], :placement_auto=>[true, 1], :placement_availability_zone=>[nil, nil], :cloud_tenant=>[nil, nil], :cloud_network=>[nil, nil], :security_groups=>[nil, nil], :floating_ip_address=>[nil, nil], :number_of_vms=>[1, "1"], :vm_description=>nil, :vm_prefix=>nil, :vm_name=>"vparekh_test", :schedule_type=>["immediately", "Immediately on Approval"], :schedule_time=>Wed, 13 Dec 2017 00:00:00 UTC +00:00, :retirement=>[0, "Indefinite"], :retirement_warn=>[604800, "1 Week"], :instance_type=>[1, "m1.tiny"], :guest_access_key_pair=>[nil, nil], :name=>nil, :size=>nil, :delete_on_terminate=>false, :customization_template_script=>nil, :dns_servers=>nil, :dns_suffixes=>nil, :addr_mode=>["dhcp", "DHCP"], :linux_host_name=>nil, :gateway=>nil, :linux_domain_name=>nil, :subnet_mask=>nil, :customization_template_id=>[nil, nil], :root_password=>nil, :hostname=>nil, :start_date=>"12/13/2017", :start_hour=>"00", :start_min=>"00", :src_vm_nics=>[], :src_vm_lans=>[], :src_ems_id=>[3, "RHOS11"], :name_1=>"vol1", :size_1=>"2", :volumes=>[{:name=>"vol1", :size=>"2"}], :requester_group=>"EvmGroup-super_administrator", :owner_group=>nil, :delivered_on=>2017-12-12 08:53:44 UTC, :user_message=>"[EVM] VM [vparekh_test] Step [CheckProvisioned] Status [Creating VM] Message [Creating VM] Current Retry Number [1]"}, userid: "admin", source_id: 2, source_type: "VmOrTemplate", destination_id: nil, destination_type: nil, tenant_id: 2, service_order_id: nil, process: true>
Created attachment 1366502 [details] recording Volume half fix
It appears that if the "Add Volume" button was added after interacting with the fields that were already present, then a flag could be set that would prevent the new fields from being observed. A new patch is available to address this issue: https://github.com/ManageIQ/manageiq-ui-classic/pull/3043
New commit detected on ManageIQ/manageiq-ui-classic/master: https://github.com/ManageIQ/manageiq-ui-classic/commit/1648446c42250bc815da6cc31c0dcfaedabb855d commit 1648446c42250bc815da6cc31c0dcfaedabb855d Author: Sam Lucidi <slucidi> AuthorDate: Tue Dec 12 15:24:40 2017 -0500 Commit: Sam Lucidi <slucidi> CommitDate: Tue Dec 12 15:24:40 2017 -0500 Reset isObserved flag on cloned volume fieldsets If the volume fieldset was cloned after focusing a field, the isObserved flag would be true on the original field, causing the clone not to be watched for changes. Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1520540 app/views/miq_request/_prov_dialog_volume_fieldset.html.haml | 3 +++ 1 file changed, 3 insertions(+)
Moving back to ON_DEV as this is cfme-5.9 BZ and the PR isn't backported to Gaprindashvili branch yet.
New commit detected on ManageIQ/manageiq-ui-classic/gaprindashvili: https://github.com/ManageIQ/manageiq-ui-classic/commit/544df3b39979f0b5801d8398de677fd7141aa650 commit 544df3b39979f0b5801d8398de677fd7141aa650 Author: Martin Hradil <himdel> AuthorDate: Fri Dec 15 14:48:23 2017 +0100 Commit: Satoe Imaishi <simaishi> CommitDate: Fri Dec 15 14:21:19 2017 -0500 Merge pull request #3043 from mansam/fix-observation-of-cloned-volume-fieldsets Fix observation of cloned volume fieldsets (cherry picked from commit d4536f703bdbe64fdd865f1c24ede1fb62468961) Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1520540 .../miq_request/_prov_dialog_volume_fieldset.html.haml | 15 +++++++++------ 1 file changed, 9 insertions(+), 6 deletions(-)
Able to verify this on Version 5.9.0.14.20171219165537_e3dea29