Project

General

Profile

Actions
Copy link

Bug #35683

closed

Content host status refreshed even for hosts without content

Added by Jeremy Lenz over 2 years ago. Updated over 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Jeremy Lenz
Category:
Hosts
Target version:
Katello 4.7.1
Difficulty:
Triaged:
Yes
Bugzilla link:
Pull request:
https://github.com/Katello/katello/pull/10353, https://github.com/Katello/katello/pull/10426, https://github.com/Katello/katello/pull/10743
Fixed in Releases:
Katello 4.8.0
Found in Releases:
Red Hat JIRA:

Description

see https://community.theforeman.org/t/build-status-for-host-without-content-can-not-be-set-to-build-refresh-content-host-statuses-fails/30487

It hits an error here because `errata` is nil: https://github.com/Katello/katello/blob/ee84716c4c94b28b0bb1302e63e6e81f3615c710/app/models/katello/errata_status.rb#L61

But the real question is why does `refresh_content_host_status` not first check to see if the host has a content facet? https://github.com/Katello/katello/blob/ee84716c4c94b28b0bb1302e63e6e81f3615c710/app/models/katello/concerns/host_managed_extensions.rb#L133

Actions
Copy link
 #1

Updated by Chris Roberts over 2 years ago

  • Target version set to Katello 4.7.0
  • Triaged changed from No to Yes
Actions
Copy link
 #2

Updated by Chris Roberts over 2 years ago

  • Target version changed from Katello 4.7.0 to Katello 4.8.0
Actions
Copy link
 #3

Updated by The Foreman Bot over 2 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Jeremy Lenz
  • Pull request https://github.com/Katello/katello/pull/10353 added
Actions
Copy link
 #4

Updated by Jeremy Lenz over 2 years ago

  • Target version changed from Katello 4.8.0 to Katello 4.7.1
  • Triaged changed from Yes to No
Actions
Copy link
 #5

Updated by Samir Jha over 2 years ago

  • Triaged changed from No to Yes
Actions
Copy link
 #6

Updated by The Foreman Bot over 2 years ago

  • Fixed in Releases Katello 4.8.0 added
Actions
Copy link
 #7

Updated by Jeremy Lenz over 2 years ago

  • Status changed from Ready For Testing to Closed
Actions
Copy link
 #8

Updated by The Foreman Bot about 2 years ago

  • Pull request https://github.com/Katello/katello/pull/10426 added
Actions
Copy link
 #9

Updated by The Foreman Bot over 1 year ago

  • Pull request https://github.com/Katello/katello/pull/10743 added
Actions
Copy link

Also available in: Atom PDF