Bug 1195628

Created from redmine issue http://projects.theforeman.org/issues/9325

Upstream fix has been merged into Foreman:

commit b18aa5307b9fc407bbd469ca6442553ce012eb78
Author: Tomas Strachota <tstrachota>
Date:   Mon Feb 23 15:47:33 2015 +0000

    Fixes #9325 - preserve compute resource attrs after NIC modal is closed

hi

please provide verification steps

thanks

The reproducer is:
1. On the New host page, select a compute resource that allows for creating interfaces (e.g. libvirt)
2. Go to the the Interfaces tab and edit the first interface in a modal window
3. At the very bottom of the modal window there's a section named by the compute resource. Edit any of the fields in that section.
4. Close the modal, reopen it and check whether the fields you changed have the desired values.

VERIFIED:

# rpm -qa | grep foreman
qe-sat6-rhel71.usersys.redhat.com-foreman-client-1.0-1.noarch
qe-sat6-rhel71.usersys.redhat.com-foreman-proxy-1.0-1.noarch
ruby193-rubygem-foreman_docker-1.2.0.7-1.el7sat.noarch
rubygem-hammer_cli_foreman_discovery-0.0.1.4-1.el7sat.noarch
ruby193-rubygem-foreman-redhat_access-0.1.0-1.el7sat.noarch
foreman-compute-1.7.2.15-1.el7sat.noarch
foreman-vmware-1.7.2.15-1.el7sat.noarch
ruby193-rubygem-foreman_discovery-2.0.0.9-1.el7sat.noarch
rubygem-hammer_cli_foreman_bootdisk-0.1.2.5-1.el7sat.noarch
ruby193-rubygem-foreman_gutterball-0.0.1.9-1.el7sat.noarch
foreman-1.7.2.15-1.el7sat.noarch
foreman-ovirt-1.7.2.15-1.el7sat.noarch
rubygem-hammer_cli_foreman-0.1.4.7-1.el7sat.noarch
foreman-proxy-1.7.2.4-1.el7sat.noarch
qe-sat6-rhel71.usersys.redhat.com-foreman-proxy-client-1.0-1.noarch
foreman-postgresql-1.7.2.15-1.el7sat.noarch
ruby193-rubygem-foreman_hooks-0.3.7-2.el7sat.noarch
foreman-selinux-1.7.2.13-1.el7sat.noarch
foreman-gce-1.7.2.15-1.el7sat.noarch
ruby193-rubygem-foreman-tasks-0.6.12.3-1.el7sat.noarch
rubygem-hammer_cli_foreman_tasks-0.0.3.3-1.el7sat.noarch
foreman-debug-1.7.2.15-1.el7sat.noarch
foreman-libvirt-1.7.2.15-1.el7sat.noarch
ruby193-rubygem-foreman_bootdisk-4.0.2.10-1.el7sat.noarch

steps :

log to sat6 ui 
navigate to compute resource (interface libvirt)
edit the first interface in the modal window
edit for example description
close the window
reopen it . the fields changed retain their values

Screenshot attached:

Created attachment 1017292 [details]
compute resource libvirt

Since the problem was on the "New host" page and the verification steps with the screenshot seem to be taken on "Compute resources" page, I request a re-verification, please.
Thank you!

VERIFIED :

# rpm -qa  | grep foreman
foreman-libvirt-1.7.2.15-1.el6_6sat.noarch
ruby193-rubygem-foreman-tasks-0.6.12.3-1.el6_6sat.noarch
foreman-postgresql-1.7.2.15-1.el6_6sat.noarch
ruby193-rubygem-foreman_docker-1.2.0.7-1.el6_6sat.noarch
ruby193-rubygem-foreman_gutterball-0.0.1.9-1.el6_6sat.noarch
rubygem-hammer_cli_foreman_discovery-0.0.1.4-1.el6_6sat.noarch
foreman-debug-1.7.2.15-1.el6_6sat.noarch
foreman-compute-1.7.2.15-1.el6_6sat.noarch
foreman-gce-1.7.2.15-1.el6_6sat.noarch
ruby193-rubygem-foreman_bootdisk-4.0.2.10-1.el6_6sat.noarch
rubygem-hammer_cli_foreman_tasks-0.0.3.3-1.el6_6sat.noarch
foreman-proxy-1.7.2.4-1.el6_6sat.noarch
dhcp207-193.lab.eng.pnq.redhat.com-foreman-client-1.0-1.noarch
dhcp207-193.lab.eng.pnq.redhat.com-foreman-proxy-1.0-2.noarch
foreman-1.7.2.15-1.el6_6sat.noarch
foreman-vmware-1.7.2.15-1.el6_6sat.noarch
ruby193-rubygem-foreman-redhat_access-0.1.0-1.el6_6sat.noarch
rubygem-hammer_cli_foreman-0.1.4.7-1.el6_6sat.noarch
dhcp207-193.lab.eng.pnq.redhat.com-foreman-proxy-client-1.0-1.noarch
foreman-selinux-1.7.2.13-1.el6_6sat.noarch
ruby193-rubygem-foreman_hooks-0.3.7-2.el6_6sat.noarch
rubygem-hammer_cli_foreman_bootdisk-0.1.2.5-1.el6_6sat.noarch
ruby193-rubygem-foreman_openscap-0.3.2.5-1.el6_6sat.noarch
foreman-ovirt-1.7.2.15-1.el6_6sat.noarch
ruby193-rubygem-foreman_discovery-2.0.0.9-1.el6_6sat.noarch
puppet-foreman_scap_client-0.3.3-6.el6_6sat.noarch

On the New host page,
Go to the the Interfaces tab and edit the first interface in a modal window
Close the modal, reopen it and check whether the fields you changed have the desired values.

screenshot attached

Adding the irc chat for ready reference

tstrachota> tkolhar, hi there
<tstrachota> tkolhar, I didn't meant to select compute resource from a combo box on the very first tab
<tstrachota> tkolhar, sorry, again: I meant to select compute resource from a combo box on the very first tab
<tstrachota> I stopped typing in the middle of sentence and finished differently:)
<tstrachota> tkolhar, than check the modal window on the "Network" tab, please
<tkolhar> tstrachota, hi 
<tkolhar> tstrachota, on Network tab
<tkolhar> tstrachota, update some value ?
<tstrachota> tkolhar, there's "+ Add interface" button
<tkolhar> tstrachota, yeah clicked
<tkolhar> tstrachota, now update any value
<tstrachota> tkolhar, yep
<tstrachota> tkolhar, afair it was combo box issue
<tkolhar> tstrachota, ok it got updated 
<tkolhar> tstrachota, so is it done
<tstrachota> good, verified then
<tkolhar> tstrachota, At the very bottom of the modal window there's a section named by the compute resource. Edit any of the fields in that section.
<tkolhar> tstrachota, i didn't get this step
<tstrachota> tkolhar, disregard this part, please. I forgot we didn't cherry-pick this bit into 6.1
<tkolhar> tstrachota, ah thanks becoz i was looking for compute resource in the interface tab
<tkolhar> tstrachota, thanks

Created attachment 1017902 [details]
Add interface

This bug is slated to be released with Satellite 6.1.

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:1592