Hide Forgot
Description of problem: RHEV-H save invalid configuration. (such as: Invalid NFS Entry; Invalid NetConsole Hostname or Address) Version-Release number of selected component (if applicable): rhev-hypervisor6-6.3-20120215.0.el6 How reproducible: 100% Steps to Reproduce: 1. Install RHEV-H 6.3. 2. Enter logging menu, input invalid server address (e.g. "......") 3. Press "Apply" button. 4. Enter kdump menu, input invalid NFS (e.g. "......") 5. cat /etc/rsysconfig.conf 6. cat /etc/sysconfig/netconsole.conf 7. cat /etc/kdump.conf Actual results: Warning message will pop-up when input invalid info. RHEV-H save invalid configuration. # cat /etc/kdump.conf default reboot net ...... Expected results: RHEV-H only persist valid configuration. Additional info:
My plan is to re-check the validity of all fields when the "Apply" button is pressed. If there are any fields with invalid values, a popup will appear naming the and request the user to fix them before the changes can be applied. Comments?
sounds good, for the most part I believe the values if blank or incorrect will clear out, may want to check that first
also make sure that navigating off the screen (up/down) doesn't cause an issue (it shouldn't, but doesn't hurt to double check)
(In reply to comment #3) > sounds good, for the most part I believe the values if blank or incorrect will > clear out, may want to check that first That sounded like a good idea, the problem is that e.g. if someone enters a character as a port number, I can not just blank it, as this might be needed to write the configuration.
Ok, I just saw (In reply to comment #5) > (In reply to comment #3) > > sounds good, for the most part I believe the values if blank or incorrect will > > clear out, may want to check that first Ok, I just saw how it's done on othe fields right now. This patch also does it this way: http://gerrit.ovirt.org/#change,3711 Tested as follows: - Installed node - Enter the TUI - Select the Logging page - Append an "a" to the max file size field - Leave the field using down-arrow or TAB - A dialog pops up, telling the field is containing an invalid value - Select "OK" = The field is reset to "1024" - Select the KDump page - Enable NFS - Change the NFS server field to "---" - Leave the field using down-arrow or TAB - A dialog pops up, telling the field is containing an invalid value - Select "OK" = The field is reset to "" - Press F2 to drop to shell - /etc/kdump.conf does not exist
Technical note added. If any revisions are required, please edit the "Technical Notes" field accordingly. All revisions will be proofread by the Engineering Content Services team. New Contents: Previously, when configuring certain functionality, invalid input would be saved causing inconsistent or incorrect behavior. Now, when invalid input is detected, it will not be persisted to the configuration files.
Technical note updated. If any revisions are required, please edit the "Technical Notes" field accordingly. All revisions will be proofread by the Engineering Content Services team. Diffed Contents: @@ -1 +1 @@ -Previously, when configuring certain functionality, invalid input would be saved causing inconsistent or incorrect behavior. Now, when invalid input is detected, it will not be persisted to the configuration files.+Previously, when configuring the Hypervisor, invalid input would be saved to disk and persisted, causing inconsistent or incorrect behavior. Now, when invalid input is detected, it is not persisted to the configuration files.
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-2012-0741.html