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 1463893 - Missing the URL link, failed to click and open "1 Content Host" for a hypervisor
Summary: Missing the URL link, failed to click and open "1 Content Host" for a hyperv...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: Hosts - Content
Version: 6.3.0
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: Unspecified
Assignee: Walden Raines
QA Contact: Katello QA List
URL: http://projects.theforeman.org/issues...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2017-06-22 03:31 UTC by Eko
Modified: 2019-09-26 14:03 UTC (History)
9 users (show)

Fixed In Version: tfm-rubygem-katello-3.4.4
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2018-02-21 16:54:17 UTC
Target Upstream Version:
Embargoed:

Attachments (Terms of Use)
Missing link (102.13 KB, image/png)
2017-06-22 03:31 UTC, Eko 		no flags Details
View All


Links
System ID Private Priority Status Summary Last Updated
Foreman Issue Tracker 20279 0 None None None 2017-07-11 19:00:48 UTC
Foreman Issue Tracker 20598 0 None None None 2017-08-14 19:42:15 UTC

Description Eko 2017-06-22 03:31:08 UTC 
Created attachment 1290467 [details]
Missing link

Version-Release number of selected component (if applicable):
Satellite6.3.0-snap3.0
virt-who-0.19-2.el7sat.noarch.rpm


Steps to Reproduce:
1. Running virt-who to send the H/G info to satellite6.3, make sure the hypervisor and guest are registered in satellite webUI(Hosts => Content Host)

2. open the hypervisor page to check the link as the attached file


Actual results:
link missing, can't open the url


Expected results:
the url can be opened normally and refer to the guest page.
Comment 4 Walden Raines 2017-07-11 18:36:26 UTC 
Created redmine issue http://projects.theforeman.org/issues/20279 from this bug
Comment 5 Walden Raines 2017-07-11 19:03:15 UTC 
PR: https://github.com/Katello/katello/pull/6865
Comment 6 Satellite Program 2017-07-17 20:01:13 UTC 
Moving this bug to POST for triage into Satellite 6 since the upstream issue http://projects.theforeman.org/issues/20279 has been resolved.
Comment 7 Satellite Program 2017-08-03 22:01:08 UTC 
Moving this bug to POST for triage into Satellite 6 since the upstream issue http://projects.theforeman.org/issues/20279 has been resolved.
Comment 8 Eko 2017-08-09 02:50:00 UTC 
Check this issue with sat6.3-snap10, click "1 Content Host" on the hypervisor page, it will redirect to the "Content Host" home page, not to the "guest" page, so I change the state to NEW.
Comment 9 Satellite Program 2017-08-09 04:01:20 UTC 
Moving this bug to POST for triage into Satellite 6 since the upstream issue http://projects.theforeman.org/issues/20279 has been resolved.
Comment 10 Walden Raines 2017-08-09 13:49:27 UTC 
That's not how this functionality worked before it was broken.(In reply to Eko from comment #8)
> Check this issue with sat6.3-snap10, click "1 Content Host" on the
> hypervisor page, it will redirect to the "Content Host" home page, not to
> the "guest" page, so I change the state to NEW.

That's not how this functionality worked before it was broken.  Before it was broken the link just took you to the content host list with the search including the virtual guest names.  This purpose of this fix is to simply restore the former functionality.  If we want a conditional link that changes based on the number of content hosts then we need an RFE.
Comment 11 Eko 2017-08-10 02:25:27 UTC 
(In reply to Walden Raines from comment #10)
> That's not how this functionality worked before it was broken.(In reply to
> Eko from comment #8)
> > Check this issue with sat6.3-snap10, click "1 Content Host" on the
> > hypervisor page, it will redirect to the "Content Host" home page, not to
> > the "guest" page, so I change the state to NEW.
> 
> That's not how this functionality worked before it was broken.  Before it
> was broken the link just took you to the content host list with the search
> including the virtual guest names.  This purpose of this fix is to simply
> restore the former functionality.  If we want a conditional link that
> changes based on the number of content hosts then we need an RFE.

Yes, this link should take me to the content host list according to the searching guest name, but actually it will list all the host and guest.

The interesting action is:
if I click "1 Content Host" link directly, it will feedback all the host and guest list, missing the search keyword.

If I right click "1 Content Host" link and choose "open Link in New Tab", it will only feedback the list according to the searching guest name, that's expected result.
Comment 12 Walden Raines 2017-08-10 14:46:46 UTC 
(In reply to Eko from comment #11)
> (In reply to Walden Raines from comment #10)
> > That's not how this functionality worked before it was broken.(In reply to
> > Eko from comment #8)
> > > Check this issue with sat6.3-snap10, click "1 Content Host" on the
> > > hypervisor page, it will redirect to the "Content Host" home page, not to
> > > the "guest" page, so I change the state to NEW.
> > 
> > That's not how this functionality worked before it was broken.  Before it
> > was broken the link just took you to the content host list with the search
> > including the virtual guest names.  This purpose of this fix is to simply
> > restore the former functionality.  If we want a conditional link that
> > changes based on the number of content hosts then we need an RFE.
> 
> Yes, this link should take me to the content host list according to the
> searching guest name, but actually it will list all the host and guest.
> 
> The interesting action is:
> if I click "1 Content Host" link directly, it will feedback all the host and
> guest list, missing the search keyword.
> 
> If I right click "1 Content Host" link and choose "open Link in New Tab", it
> will only feedback the list according to the searching guest name, that's
> expected result.

Oh I see.  Thanks for the clarification.
Comment 13 Walden Raines 2017-08-14 20:54:19 UTC 
New PR: https://github.com/Katello/bastion/pull/204
Comment 14 Walden Raines 2017-08-15 22:11:32 UTC 
commit 0cd73f9f2fccad6d0608f109231fcd21004a15a6
Author: Walden Raines <walden>
Date:   Mon Aug 14 16:11:24 2017 -0400

    Fixes #20598: use state params for search.
    
    We should respect the state params for searches when an href is
    generated with ui-sref. This commit ensures that $stateParams is
    preferred over the table cache and what is in the query strings.
    
    http://projects.theforeman.org/issues/20598
Comment 15 Jonathon Turel 2017-08-23 19:22:37 UTC 
The second PR is not in the latest snap (6.3 Snap 12) and therefore this is failing at least partially: the link works but the table is not filtered according to the guests within the hypervisor (unless opening a new tab).

I think the correct status is back to POST since it seemingly does not require futher dev work but a cherry pick of 0cd73f9f2fccad6d0608f109231fcd21004a15a6
Comment 16 Eko 2017-08-28 08:37:23 UTC 
verified in sat6.3-snap13, it can filter the list as expected.
Comment 17 Satellite Program 2018-02-21 16:54:17 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-2018:0336
Note You need to log in before you can comment on or make changes to this bug.