Bug #15287

link in graph on Smart proxy -> Puppet page have "undefined" as smart proxy name

Added by Bryan Kearney about 2 years ago. Updated 8 days ago.

Status:Closed
Priority:Normal
Assignee:Tomer Brisker
Category:Smart proxies
Target version:1.13.3
Difficulty: Team Backlog:
Triaged: Fixed in Releases:
Bugzilla link:1337483 Found in Releases:
Pull request:https://github.com/theforeman/foreman/pull/4072

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1337483
Description of problem:
Link in graph on capsule -> Puppet page have "undefined" as smart proxy name

Version-Release number of selected component (if applicable):
satellite-6.2.0-12.0.el7sat.noarch

How reproducible:
always

Steps to Reproduce:
1. Have Sat and Capsule with some hosts registered against Capsule
2. Go to Infrastucture -> Capsules -> <capsule> -> Puppet -> General -> Host Configuration Chart
3. Click on some segment of that pie/round graph

Actual results:
You are taken to https://&lt;fqdn&gt;/smart_proxies/undefined which does not exist

Expected results:
There should be no link or there should be a valid one


Related issues

Related to Foreman - Bug #16044: Errors in individual dashboard widgets fail entire page Closed 08/10/2016

Associated revisions

Revision 41be22a1
Added by Tomer Brisker over 1 year ago

Fixes #15287 - Correct link from puppet proxy dashboard chart

Revision 51365bc5
Added by Tomer Brisker over 1 year ago

Fixes #15287 - Correct link from puppet proxy dashboard chart

(cherry picked from commit 41be22a16ba8dbcf17e11661d4eb3fbe9f755a1d)

Revision 3e1900e0
Added by Tomer Brisker over 1 year ago

Fixes #15287 - Correct link from puppet proxy dashboard chart

(cherry picked from commit 41be22a16ba8dbcf17e11661d4eb3fbe9f755a1d)

History

#1 Updated by Dominic Cleal about 2 years ago

  • Subject changed from link in graph on capsule -> Puppet page have "undefined" as smart proxy name to link in graph on Smart proxy -> Puppet page have "undefined" as smart proxy name
  • Category changed from Web Interface to Smart proxies

#2 Updated by Ohad Levy over 1 year ago

  • Assignee set to Tomer Brisker

#3 Updated by The Foreman Bot over 1 year ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/4072 added

#4 Updated by Anonymous over 1 year ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#5 Updated by Tomer Brisker over 1 year ago

  • Related to Bug #16044: Errors in individual dashboard widgets fail entire page added

#6 Updated by Dominic Cleal over 1 year ago

  • Legacy Backlogs Release (now unused) set to 207

#7 Updated by Daniel Lobato Garcia over 1 year ago

  • Target version set to 1.4.3

Also available in: Atom PDF