Bug 971937
Summary: | Trends: Creation of a new Trend Counter displays trend type, not trend name. | ||
---|---|---|---|
Product: | Red Hat Satellite | Reporter: | Corey Welton <cwelton> |
Component: | Provisioning | Assignee: | jmagen <jmagen> |
Status: | CLOSED CURRENTRELEASE | QA Contact: | Sachin Ghai <sghai> |
Severity: | unspecified | Docs Contact: | |
Priority: | unspecified | ||
Version: | Nightly | CC: | bkearney, dcleal, gkhachik, jhutar, jmagen, jmontleo, ohadlevy, sghai |
Target Milestone: | Unspecified | Keywords: | 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-07-02 14:08:01 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
Corey Welton
2013-06-07 15:51:33 UTC
I think we should disable trend name if not a FactTrend. https://github.com/theforeman/foreman/pull/888 https://github.com/theforeman/foreman/pull/888 has been merged upstream. Moving this to be tested during MDP3, not critical for MDP2 success story qa_ack-ed Seems that on Satellite-6.0.3-RHEL-6-20140321.2 compose this is still broken or maybe I'm doing something wrong (?): 1. Go to Monitor -> Trends -> Add Trend Counter 2. Select Trend Type: Model (note that you can not edit "Name" field) 3. Trend gets created with name "Model" Jan, I think it is working correcting. For any Trend Type that is not Facts, a user now cannot enter/edit the name. Before the fix on https://github.com/theforeman/foreman/commit/460215e, the trend name could be added if Trend Type was not "Facts", but the name entered did not show up. This was bud. However, now that a user can't enter the name. We should correct the bug title from "Trends: Creation of a new Trend Counter displays trend type, not trend name" to "Trends: Creation of a new Trend Counter should show trend type for Foreman Trends and trend name for Fact Trends" Let me know if you have any questions. ping As Joseph said, the behaviour described in comment #11 is expected. The fix was to prevent editing of the name field for the built in trend types. It wasn't intended to be editable for these, only for fact-based trends. Verified with sat6 beta snap8. So as per the comment 12. We can enter/edit name for all trend types except "Facts". Looks like name field is disabled. If we choose "Facts" trend type then we can define a name and the same name appears when we view the trends. So Moving this to verified. Correction: Please re-read comment15 We can **not** enter/edit name for all trend types except "Facts" (In reply to jmagen from comment #12) > Jan, I think it is working correcting. For any Trend Type that is not Facts, > a user now cannot enter/edit the name. > [...] Sorry for missing the conversation for so long. Thanks Sachin for QE work. This was delivered with 6.0.3, which is the Satellite 6 Beta. |