Bug #24551
closedPuppet class fails loading in UI when the attribute order is created with upper case characters
Description
Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1596853
Description of problem:
Puppet class fails loading in UI when the attribute order is created with upper case characters
Version-Release number of selected component (if applicable):
satellite-6.3.1-3.el7sat.noarch
How reproducible:
Always
Steps to Reproduce:
1. Navigate to the Satellite UI -> Configure -> Classes
2. Select a puppet class -> Smart Class Parameter. Add a new added a new `Prioritize attribute order` with some upper case letters on it. eg., `test_version,UPPER_TEST`. Add a matcher at the same time with some values. eg., `test_version=4.1.0,UPPER_TEST=true` in the same UI transaction.
3. Click Submit.
Actual results:
The order and matcher would be created, but the puppet class cannot be loaded in Satellite UI anymore. Reason: The lookup_keys table changed the order to all lowercase and stored it. The lookup_values table saved the original upper case letters. This caused inconsistency between the tables and broke the UI.
Expected results:
Either do not allow upper case letters in `Prioritize attribute order` field or save all lower case in lookup_values table too - to keep the data consistent.
Additional info:
Updated by Ori Rabin over 6 years ago
- Subject changed from Puppet class fails loading in UI when the attribute order is created with upper case characters to Puppet class fails loading in UI when the attribute order is created with upper case characters
- Category changed from PuppetCA to Parameters
- Status changed from New to Assigned
- Assignee set to Ori Rabin
- Priority changed from High to Normal
Updated by The Foreman Bot over 6 years ago
- Status changed from Assigned to Ready For Testing
- Pull request https://github.com/theforeman/foreman/pull/5951 added
Updated by Ori Rabin about 6 years ago
- Status changed from Ready For Testing to Closed
Applied in changeset 892fc0757373817bd96c8edcce7781020e41c022.