Red Hat Satellite engineering is moving the tracking of its product development work on Satellite to Red Hat Jira (issues.redhat.com). If you're a Red Hat customer, please continue to file support cases via the Red Hat customer portal. If you're not, please head to the "Satellite project" in Red Hat Jira and file new tickets here. Individual Bugzilla bugs will be migrated starting at the end of May. If you cannot log in to RH Jira, please consult article #7032570. That failing, please send an e-mail to the RH Jira admins at rh-issues@redhat.com to troubleshoot your issue as a user management inquiry. The email creates a ServiceNow ticket with Red Hat. Individual Bugzilla bugs that are migrated will be moved to status "CLOSED", resolution "MIGRATED", and set with "MigratedToJIRA" in "Keywords". The link to the successor Jira issue will be found under "Links", have a little "two-footprint" icon next to it, and direct you to the "Satellite project" in Red Hat Jira (issue links are of type "https://issues.redhat.com/browse/SAT-XXXX", where "X" is a digit). This same link will be available in a blue banner at the top of the page informing you that that bug has been migrated.
Bug 1195628 - Editing interface in modal window resets compute resource NIC attributes
Summary: Editing interface in modal window resets compute resource NIC attributes
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: Provisioning
Version: 6.0.4
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: Unspecified
Assignee: Tomas Strachota
QA Contact: Tazim Kolhar
URL: http://projects.theforeman.org/issues...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-02-24 08:29 UTC by Tomas Strachota
Modified: 2017-02-23 20:26 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-08-12 05:27:10 UTC
Target Upstream Version:
Embargoed:

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


Links
System ID Private Priority Status Summary Last Updated
Foreman Issue Tracker 9325 0 None None None 2016-04-22 16:02:44 UTC
Red Hat Product Errata RHSA-2015:1592 0 normal SHIPPED_LIVE Important: Red Hat Satellite 6.1.1 on RHEL 6 2015-08-12 09:04:35 UTC

Description Tomas Strachota 2015-02-24 08:29:39 UTC 
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 08:29:41 UTC 
Created from redmine issue http://projects.theforeman.org/issues/9325
Comment 2 Tomas Strachota 2015-02-24 08:36:37 UTC 
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
Comment 7 Tazim Kolhar 2015-04-21 10:21:17 UTC 
hi

please provide verification steps

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