Project

General

Profile

Actions

Bug #38726

closed

500 error is creating rolling CV via API with no environments

Added by Ian Ballou 8 months ago. Updated 6 months ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
Content Views
Target version:
Difficulty:
Triaged:
Yes
Fixed in Releases:
Found in Releases:

Description

500 error is creating rolling CV via API with no environements
Cloned from 17371764

Katello 4.19.dev

Description of problem:

{color:#1d1c1d}# hammer content-view create --name test2 --organization-id 3 --rolling Could not create the content view: undefined method `any?' for nil:NilClass{color}

 

How reproducible:

100%

Is this issue a regression from an earlier version:

Yes

Steps to Reproduce:

{color:#1d1c1d}hammer content-view create --name test2 --organization-id 3 --rolling{color}

Actual behavior:
Ugly error is thrown

Expected behavior:
Library is assumed if no environments are passed.

Business Impact / *Additional info:*

 

Actions #1

Updated by Ian Ballou 8 months ago

  • Target version changed from Katello 4.20.0 to Katello 4.19.0
Actions #2

Updated by Ian Ballou 8 months ago

We also need to improve the UI allowing users to create a rolling CV with 0 environments.

Actions #3

Updated by The Foreman Bot 8 months ago

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

Updated by Quirin Pamp 8 months ago · Edited

The choice to allow the creation of rolling CVs without any environments (both via API and GUI) was deliberate.

The reasoning was: We do not want to default to Library, because avoiding Library was the entire reason for introducing the rolling CV environments in the first place.

Therefore we let people create rolling CVs without any environments (which are functionally useless because they need at least one environment to be consumable), but it is easy to add any desired environments later.

A possible alternative would be to require that users supply at least one environment both via API and GUI when creating a rolling CV (this would also avoid defaulting to Library), but would require more validation.

Actions #5

Updated by Ian Ballou 8 months ago

  • Category set to Content Views
  • Triaged changed from No to Yes
Actions #6

Updated by The Foreman Bot 8 months ago

  • Fixed in Releases Katello 4.19.0 added
Actions #7

Updated by Quirin Pamp 8 months ago

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

Updated by Ian Ballou 6 months ago

  • Subject changed from 500 error is creating rolling CV via API with no environements to 500 error is creating rolling CV via API with no environments
Actions

Also available in: Atom PDF