Project

General

Profile

Actions

Bug #26049

closed

Non-grammatical error message when docker tags whitelist is being set for non-docker repos

Added by Samir Jha almost 6 years ago. Updated almost 6 years ago.

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

Description

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

Steps:
1. hammer product create --name 'testprod' --organization-id 1
2. hammer repository create --product-id 1 --name test --content-type yum --docker-tags-whitelist 'one,two,three'

Actual output:
Could not create the repository:
Validation failed: Docker tags whitelist White list can be only set for Container Image repositories.

This error message is non-grammatical. Not only "whitelist" is repeated, two different spellings are used.

Error message should be:
"Validation failed: Docker tags whitelist can be only set for Container Image repositories."

Or maybe the intention is to list all fields with errors and specific error message for each of them? Then we could use colon or dash here. And consistent spelling nevertheless.
"Validation failed: Docker tags whitelist: White list can be only set for Container Image repositories."

Version:
Satellite 6.5 snap 11
pulp-server-2.18.0-0.1.rc.el7sat.noarch
foreman-1.20.1.4-1.el7sat.noarch
satellite-6.5.0-5.beta.el7sat.noarch
katello-3.10.0-0.6.rc1.el7sat.noarch

Actions #1

Updated by The Foreman Bot almost 6 years ago

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

Updated by Anonymous almost 6 years ago

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

Updated by Chris Roberts almost 6 years ago

  • Fixed in Releases Katello 3.12.0 added
  • Found in Releases Katello 3.10.0 added
Actions #4

Updated by Chris Roberts almost 6 years ago

  • Target version set to Katello 3.12.0
  • Triaged changed from No to Yes
Actions

Also available in: Atom PDF