Bug 1436367

Summary: "Save" and "Reset" buttons are absent when adding log collection configuration
Product: Red Hat CloudForms Management Engine Reporter: Oleksii Tsuman <otsuman>
Component: UI - OPSAssignee: Aparna Karve <akarve>
Status: CLOSED CURRENTRELEASE QA Contact: Oleksii Tsuman <otsuman>
Severity: high Docs Contact:
Priority: high    
Version: 5.8.0CC: dajohnso, hkataria, jhardy, mpovolny, obarenbo, otsuman, simaishi
Target Milestone: GAKeywords: Regression, TestOnly
Target Release: 5.9.0Flags: otsuman: automate_bug+
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: ui:button
Fixed In Version: 5.9.0.1 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1437907 1437909 (view as bug list) Environment:
Last Closed: 2018-03-06 14:45:18 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: 1437907, 1437909    
Attachments:
Description Flags
screenshot none

Description Oleksii Tsuman 2017-03-27 18:15:44 UTC
Created attachment 1266731 [details]
screenshot

Description of problem:
"Save" and "Reset" buttons are absent when adding log collection configuration for a depot after selecting depot type. 

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1. Got to Configuration -> Diagnostics -> Collect Logs -> Edit
2. Select depot type (for ex. NFS)


Actual results:
"Save" and "Reset" buttons are absent

Expected results:
"Save" and "Reset" buttons are present and clickable after setting all fields

Additional info:
present in 5.8.0.7.20170321164727_1c97ccd

Comment 5 Oleksii Tsuman 2017-03-30 18:19:28 UTC
Yes, it's a regression and bug present only in 5.8, in 5.7.x these buttons in place

Comment 6 Oleksii Tsuman 2017-03-30 18:20:55 UTC
Even in 5.8.0.4 buttons was present

Comment 7 CFME Bot 2017-03-30 20:43:07 UTC
New commit detected on ManageIQ/manageiq-ui-classic/master:
https://github.com/ManageIQ/manageiq-ui-classic/commit/2778c68ac154d71b437c3a55529ae47246e58130

commit 2778c68ac154d71b437c3a55529ae47246e58130
Author:     Aparna Karve <akarve>
AuthorDate: Wed Mar 29 14:46:16 2017 -0700
Commit:     Aparna Karve <akarve>
CommitDate: Wed Mar 29 14:46:16 2017 -0700

    Removed code that sets newRecord to true
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1436367

 .../javascripts/controllers/ops/log_collection_form_controller.js       | 2 --
 1 file changed, 2 deletions(-)

Comment 10 Oleksii Tsuman 2017-10-13 12:27:28 UTC
verified in 5.9.0.2.20171010190026_0413a06