Project

General

Profile

Bug #23674

undefined method `include?' for nil:NilClass on selecting some kickstart repos on RH repos page

Added by Partha Aji 6 months ago. Updated 4 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Repositories
Target version:
Difficulty:
Triaged:
Bugzilla link:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

Steps to Reproduce:
1. Navigate to RH Repos page
2. Select content_type = kickstart
3. Try enabling
Red Hat Enterprise Linux Atomic Host (Kickstart)
Red Hat Enterprise Linux Atomic Host Beta (Kickstart)
Red Hat Enterprise Linux 7 Server Beta (Kickstart)
Red Hat Enterprise Linux 7 for IBM Power LE Beta (Kickstart)

Actual results:
The option throws undefined method `include?' for nil:NilClass error

Expected results:
The page should give us list of repos under label and ability to enable those

Additional info:
PFA the error log in comment #1
I am still able to correctly select and enable some kickstarts repos like
Red Hat Enterprise Linux 6 Server (Kickstart)
Red Hat Enterprise Linux 7 Server (Kickstart)

Associated revisions

Revision e98bc550 (diff)
Added by Partha Aji 6 months ago

Fixes #23674 - Lists no sub KS repos on RH Repos

The new RH Repos page made the assumption that all KS repositories had a
release version set. Turns out that some kickstart repositories do not.
Examples include
Red Hat Enterprise Linux 7 Server Beta (Kickstart)
Red Hat Enterprise Linux 7 for IBM Power LE Beta (Kickstart)

This commit properly handles the case of KS repos that do not have a
release version set.

History

#1 Updated by Partha Aji 6 months ago

  • Bugzilla link set to 1580438

#2 Updated by The Foreman Bot 6 months ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/Katello/katello/pull/7393 added

#3 Updated by Jonathon Turel 6 months ago

  • Legacy Backlogs Release (now unused) set to 338

#4 Updated by Partha Aji 6 months ago

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

Also available in: Atom PDF