Bugzilla will be upgraded to version 5.0. The upgrade date is tentatively scheduled for 2 December 2018, pending final testing and feedback.
Bug 1195628 - Editing interface in modal window resets compute resource NIC attributes
Editing interface in modal window resets compute resource NIC attributes
Status: CLOSED ERRATA
Product: Red Hat Satellite 6
Classification: Red Hat
Component: Provisioning (Show other bugs)
6.0.4
Unspecified Unspecified
unspecified Severity medium (vote)
: Unspecified
: Unused
Assigned To: Tomas Strachota
Tazim Kolhar
http://projects.theforeman.org/issues...
: Triaged
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2015-02-24 03:29 EST by Tomas Strachota
Modified: 2017-02-23 15:26 EST (History)
5 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2015-08-12 01:27:10 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)
compute resource libvirt (68.39 KB, image/png)
2015-04-22 03:18 EDT, Tazim Kolhar
no flags Details
Add interface (36.00 KB, image/png)
2015-04-23 07:06 EDT, Tazim Kolhar
no flags Details


External Trackers
Tracker ID Priority Status Summary Last Updated
Foreman Issue Tracker 9325 None None None 2016-04-22 12:02 EDT
Red Hat Product Errata RHSA-2015:1592 normal SHIPPED_LIVE Important: Red Hat Satellite 6.1.1 on RHEL 6 2015-08-12 05:04:35 EDT

  None (edit)
Description Tomas Strachota 2015-02-24 03:29:39 EST
It seems that when I edit interface of host deployed on libvirt in modal window it resets compute resource attributes. It seems to take default from compute profile. I think it stores values correctly, it just does not sync them to form in modal window when I re-edit the interface.
Comment 1 Tomas Strachota 2015-02-24 03:29:41 EST
Created from redmine issue http://projects.theforeman.org/issues/9325
Comment 2 Tomas Strachota 2015-02-24 03:36:37 EST
Upstream fix has been merged into Foreman:

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

    Fixes #9325 - preserve compute resource attrs after NIC modal is closed
Comment 7 Tazim Kolhar 2015-04-21 06:21:17 EDT
hi

please provide verification steps

thanks
Comment 8 Tomas Strachota 2015-04-21 09:51:49 EDT
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.
Comment 9 Tazim Kolhar 2015-04-22 03:18:06 EDT
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:
Comment 10 Tazim Kolhar 2015-04-22 03:18:49 EDT
Created attachment 1017292 [details]
compute resource libvirt
Comment 11 Tomas Strachota 2015-04-22 09:36:36 EDT
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!
Comment 12 Tazim Kolhar 2015-04-23 07:03:17 EDT
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
Comment 13 Tazim Kolhar 2015-04-23 07:05:06 EDT
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
Comment 14 Tazim Kolhar 2015-04-23 07:06:01 EDT
Created attachment 1017902 [details]
Add interface
Comment 15 Bryan Kearney 2015-08-11 09:24:09 EDT
This bug is slated to be released with Satellite 6.1.
Comment 16 errata-xmlrpc 2015-08-12 01:27:10 EDT
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

Note You need to log in before you can comment on or make changes to this bug.