Project

General

Profile

Bug #26049

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

Added by Samir Jha over 3 years ago. Updated about 3 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Hammer
Target version:
Difficulty:
Triaged:
Yes
Bugzilla link:
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

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

Associated revisions

Revision a4870668 (diff)
Added by Samir Jha over 3 years ago

Fixes #26049 - Docker tag whitelist error message

History

#1 Updated by The Foreman Bot over 3 years ago

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

#2 Updated by Anonymous over 3 years ago

  • Status changed from Ready For Testing to Closed

#3 Updated by Chris Roberts about 3 years ago

  • Found in Releases Katello 3.10.0 added
  • Fixed in Releases Katello 3.12.0 added

#4 Updated by Chris Roberts about 3 years ago

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

Also available in: Atom PDF