Bug 911386
| Summary: | Service Level, Type does not display cleanly for subscriptions with no set service level | ||||||
|---|---|---|---|---|---|---|---|
| Product: | Red Hat Enterprise Linux 5 | Reporter: | J.C. Molet <jmolet> | ||||
| Component: | subscription-manager | Assignee: | Bryan Kearney <bkearney> | ||||
| Status: | CLOSED ERRATA | QA Contact: | IDM QE LIST <seceng-idm-qe-list> | ||||
| Severity: | medium | Docs Contact: | |||||
| Priority: | unspecified | ||||||
| Version: | 5.10 | CC: | bkearney, fsharath, jgalipea, jsefler | ||||
| Target Milestone: | beta | ||||||
| Target Release: | 5.10 | ||||||
| Hardware: | Unspecified | ||||||
| OS: | Unspecified | ||||||
| Whiteboard: | |||||||
| Fixed In Version: | Doc Type: | Bug Fix | |||||
| Doc Text: |
No description necessary
|
Story Points: | --- | ||||
| Clone Of: | Environment: | ||||||
| Last Closed: | 2013-09-30 22:56:21 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: | |||||||
| Bug Depends On: | |||||||
| Bug Blocks: | 840995, 920191 | ||||||
| Attachments: |
|
||||||
commit 76a1f4b65f3bdd184887ab43bc5adc4219b0d37b
Author: J.C. Molet <jmolet>
Date: Mon Feb 18 15:37:34 2013 -0500
911386: Displaying combined Service Level and Type should handle empty values for both items
Created attachment 709640 [details]
screenshot of RHEL-70-GUI
# rpm -qa | egrep "subscription-manager|python-rhsm"
subscription-manager-firstboot-1.8.4-1.el7.x86_64
python-rhsm-1.8.7-1.el7.x86_64
subscription-manager-1.8.4-1.el7.x86_64
subscription-manager-debuginfo-1.8.4-1.el7.x86_64
subscription-manager-gui-1.8.4-1.el7.x86_64
subscription-manager-migration-1.8.4-1.el7.x86_64
Please see the images attached. Verified for both senarios where only a service_level and no service_type or or that has neither.
VERIFIED
This request was evaluated by Red Hat Product Management for inclusion in a Red Hat Enterprise Linux release. Product Management has requested further review of this request by Red Hat Engineering, for potential inclusion in a Red Hat Enterprise Linux release for currently deployed products. This request is not yet committed for inclusion in a release. Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. http://rhn.redhat.com/errata/RHBA-2013-1332.html |
Description of problem: In subscription-manager-gui, in the list of all available subscriptions, there is a field "Service Level, Type" that lists the corresponding subscription's Version-Release number of selected component (if applicable): subscription-manager-gui-1.8.2-1.git.13.3fdfb73.el6.x86_64 How reproducible: always Steps to Reproduce: 1. Register rhsm to a standalone candlepin with TESTDATA enabled. 2. Go to the all available subscriptions tab and update the search 3. click on any product with only a service_level and no service_type or or that has neither Actual results: With the case of subscriptoins that have neither (such as "Awesome OS Modifier") you will get "Support Level, Type: Not Set, " where there is an extra unneeded comma When you get a subscription with only a service level like "Shared File System" you will get "Support Level, Type: Layered, " also with an extra comma Expected results: There shouldn't be a comma for a list with one item. Additional info: This code is in /src/subscription_manager/gui/widgets.py Currently it looks like this: display_level = support_level if support_level == "": display_level = _("Not Set") self._set(self.support_level_and_type_text, ", ".join([display_level, support_type])) It should look something like this: display_level = support_level if support_level == "": display_level = _("Not Set") if support_type != "": display_level_and_type = ", ".join([display_level, support_type]) else: display_level_and_type = display_level self._set(self.support_level_and_type_text, display_level_and_type)