Project

General

Profile

Actions

Bug #36041

closed

Add some validation for name in Simplified ACS creation via hammer

Added by Quinn James about 1 year ago. Updated about 1 year ago.

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

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=2159967

Description of problem:
Not providing a name on Simplified ACS creation via hammer shows an unsightly error. Please consider adding some validation and message like 'Missing values for alternate_content_source.'

Version-Release number of selected component (if applicable):
6.13.0 snap 5

How reproducible:
always

Steps to Reproduce:
1. Try to create a simplified ACS, not providing name for it:
  1. hammer alternate-content-source create --alternate-content-source-type simplified

Actual results:
Could not create the Alternate Content Source.:
PG::NotNullViolation: ERROR: null value in column "name" violates not-null constraint
DETAIL: Failing row contains (42, null, null, null, null, null, null, null, {}, yum, simplified, null, null, null, null).

Expected results:
Validation added and improved error message shown.

Actions #1

Updated by The Foreman Bot about 1 year ago

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

Updated by The Foreman Bot about 1 year ago

  • Fixed in Releases Katello 4.8.0 added
Actions #3

Updated by Anonymous about 1 year ago

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

Updated by Ian Ballou about 1 year ago

  • Subject changed from Add some validation for name in Simplified ACS creation via hammer to Add some validation for name in Simplified ACS creation via hammer
  • Category set to Repositories
  • Assignee set to Quinn James
  • Target version set to Katello 4.8.0
  • Triaged changed from No to Yes
Actions

Also available in: Atom PDF