Bug 1379291
| Summary: | [RFE] Inform user that all Puppet Class sub entities affect saving the whole puppet class | ||||||
|---|---|---|---|---|---|---|---|
| Product: | Red Hat Satellite | Reporter: | Oleksandr Shtaier <oshtaier> | ||||
| Component: | Smart Variables | Assignee: | Jitendra Yejare <jyejare> | ||||
| Status: | CLOSED ERRATA | QA Contact: | Jitendra Yejare <jyejare> | ||||
| Severity: | low | Docs Contact: | |||||
| Priority: | low | ||||||
| Version: | 6.3.0 | CC: | bkearney, inecas, jcallaha, jyejare, orabin, oshtaier, rohoover | ||||
| Target Milestone: | 6.4.0 | Keywords: | FutureFeature, Triaged | ||||
| Target Release: | Unused | ||||||
| Hardware: | x86_64 | ||||||
| OS: | Linux | ||||||
| Whiteboard: | |||||||
| Fixed In Version: | Doc Type: | If docs needed, set a value | |||||
| Doc Text: | Story Points: | --- | |||||
| Clone Of: | Environment: | ||||||
| Last Closed: | 2018-10-16 15:26:32 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: | |||||||
| Attachments: |
|
||||||
|
Description
Oleksandr Shtaier
2016-09-26 09:45:42 UTC
Does this happen with any other errors or just the one from BZ 1379277? It happens with any error, but it is probably impossible to create another precondition rather than described in BZ 1379277 as we need to have invalid smart variable in advance. For example, we can choose type 'boolean' and type string into value field for one variable and correct values for second variable. Of course, you will never be able to save second variable, no matter what, but it is not so critical as if we have invalid variable in puppet class already Basically, issue can be treated as RFE or a bug in the same aspects. Problem here is only in informing user that all Puppet Class sub entities (e.g. Smart Variables) will not be saved when you push 'Submit' button in case one of such sub entities has invalid data. It doesn't matter whether you editing existing entity, sub entity or creating new sub entities. Any triggered validation prevents you to save any changes that you made within last transaction. Of course, as a workaround, you can go to Smart Variable interface derectly (Configure-> Smart Variable) and do whatever action you want It sounds like this is the appropriate use case for the inline notification (http://www.patternfly.org/pattern-library/communication/inline-notifications/) since it's a fairly critical time sensitive piece of information. It would appear at the top of the page after validation. The messaging should alert the user to the fact that nothing on the page has been saved because of an existing error. Created redmine issue http://projects.theforeman.org/issues/19582 from this bug Upstream bug assigned to orabin Upstream bug assigned to orabin Moving this bug to POST for triage into Satellite 6 since the upstream issue http://projects.theforeman.org/issues/19582 has been resolved. Created attachment 1475494 [details]
[Verification] Screenshot Attached
Verified!
@ Satellite 6.4 snap 17
Steps:
1. Go to Puppet Class -> Smart Variables.
2. Create one invalid smart variable but don't submit.
3. Create another smart variable but the valid one.
4. Now submit.
Observation:
1. Both smart variables are not saved as one of the smart variable is invalid.
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. https://access.redhat.com/errata/RHSA-2018:2927 |