Bug 1105061

Summary: product label not unique
Product: Red Hat Satellite Reporter: Tomas Lestach <tlestach>
Component: Content ManagementAssignee: Katello Bug Bin <katello-bugs>
Status: CLOSED CURRENTRELEASE QA Contact: sthirugn <sthirugn>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 6.0.3CC: bbuckingham, jmontleo, sthirugn
Target Milestone: UnspecifiedKeywords: Triaged
Target Release: Unused   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-09-11 12:26:38 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Tomas Lestach 2014-06-05 10:09:21 UTC
Description of problem:
I am able to create two products within one organization with the same product label.

Version-Release number of selected component (if applicable):
Satellite-6.0.3-RHEL-6-20140529.0-Satellite-x86_64

How reproducible:
always

Steps to Reproduce:
1.
$ hammer product create --organization-id=19 --name product_name1 --label product_label
$ hammer product create --organization-id=19 --name product_name2 --label product_label

Actual results:
$ hammer product create --organization-id=19 --name product_name1 --label product_label
Product created
$ hammer product create --organization-id=19 --name product_name2 --label product_label
Product created

Having two products within one organization does not really make sense, rather points on the lack of unique constraints.

Expected results:
Label shall be unique at least within an organization.

Comment 1 RHEL Program Management 2014-06-05 10:23:29 UTC
Since this issue was entered in Red Hat Bugzilla, the release flag has been
set to ? to ensure that it is properly evaluated for this release.

Comment 3 Brad Buckingham 2014-06-24 20:32:12 UTC
Based on the current code, it looks like this one has already been solved; therefore, going to move the BZ to POST and it can be retested after the beta rebase.


hammer> product create --organization-id=1 --name product_name1 --label product_label
Product created
hammer> product create --organization-id=1 --name product_name2 --label product_label
Could not create the product:
  Validation failed: Label Product with label 'product_label' already exists in this organization.
hammer>

Comment 5 sthirugn@redhat.com 2014-07-29 18:45:33 UTC
Verified.

hammer> product create --organization-id=3 --name newproduct1 --label unique
Product created

hammer> product create --organization-id=3 --name newproduct2 --label unique
Could not create the product:
  Validation failed: Label Product with label 'unique' already exists in this organization.

Version Tested:
Satellite-6.0.4-RHEL-6-20140723.0

* apr-util-ldap-1.3.9-3.el6_0.1.x86_64
* candlepin-0.9.19-1.el6_5.noarch
* candlepin-scl-1-5.el6_4.noarch
* candlepin-scl-quartz-2.1.5-5.el6_4.noarch
* candlepin-scl-rhino-1.7R3-1.el6_4.noarch
* candlepin-scl-runtime-1-5.el6_4.noarch
* candlepin-selinux-0.9.19-1.el6_5.noarch
* candlepin-tomcat6-0.9.19-1.el6_5.noarch
* elasticsearch-0.90.10-4.el6sat.noarch
* foreman-1.6.0.29-1.el6sat.noarch
* foreman-compute-1.6.0.29-1.el6sat.noarch
* foreman-gce-1.6.0.29-1.el6sat.noarch
* foreman-libvirt-1.6.0.29-1.el6sat.noarch
* foreman-ovirt-1.6.0.29-1.el6sat.noarch
* foreman-postgresql-1.6.0.29-1.el6sat.noarch
* foreman-proxy-1.6.0.21-1.el6sat.noarch
* foreman-selinux-1.6.0-8.el6sat.noarch
* foreman-vmware-1.6.0.29-1.el6sat.noarch
* katello-1.5.0-27.el6sat.noarch
* katello-ca-1.0-1.noarch
* katello-certs-tools-1.5.6-1.el6sat.noarch
* katello-installer-0.0.56-1.el6sat.noarch
* openldap-2.4.23-32.el6_4.1.x86_64
* pulp-katello-0.3-3.el6sat.noarch
* pulp-nodes-common-2.4.0-0.23.beta.el6sat.noarch
* pulp-nodes-parent-2.4.0-0.23.beta.el6sat.noarch
* pulp-puppet-plugins-2.4.0-0.23.beta.el6sat.noarch
* pulp-puppet-tools-2.4.0-0.23.beta.el6sat.noarch
* pulp-rpm-plugins-2.4.0-0.23.beta.el6sat.noarch
* pulp-selinux-2.4.0-0.23.beta.el6sat.noarch
* pulp-server-2.4.0-0.23.beta.el6sat.noarch
* python-ldap-2.3.10-1.el6.x86_64
* ruby193-rubygem-net-ldap-0.3.1-3.el6sat.noarch
* ruby193-rubygem-runcible-1.1.0-2.el6sat.noarch

Comment 6 sthirugn@redhat.com 2014-07-29 18:46:36 UTC
Note that an other bug is created to fix the confusing error message: https://bugzilla.redhat.com/show_bug.cgi?id=1124549

Comment 8 Bryan Kearney 2014-09-11 12:26:38 UTC
This was delivered with Satellite 6.0 which was released on 10 September 2014.