Bug 647563

Summary: Unable to remove "numeric" flag from Key Types
Product: [Retired] Beaker Reporter: Sean Waite <swaite>
Component: web UIAssignee: Roman Joost <rjoost>
Status: CLOSED CURRENTRELEASE QA Contact: tools-bugs <tools-bugs>
Severity: medium Docs Contact:
Priority: low    
Version: 0.7CC: bpeck, dcallagh, dowang, ebaak, ineilsen, jnicolet, mcsontos, rjoost, tools-bugs, xtian
Target Milestone: 22.0Keywords: NeedsTestCase, Patch, Triaged
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: Misc
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-01-14 05:32:58 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Sean Waite 2010-10-28 19:12:14 UTC 
Description of problem:
Cannot uncheck "numeric" from an existing key type.


How reproducible:
100%

Steps to Reproduce:
1. Create a key type, selecting the "numeric" flag
2. Save it.
3. Open the key type, uncheck "numeric"
4. Save it.
  
Actual results:
Beaker returns "OK" - but if you open the key type again, "numeric" is still set

Expected results:
Beaker returns "OK," "numeric" should no longer be set

Additional info:
I haven't checked to see if this works (or not) the other way around - taking a key type without numeric set, and setting it.
Comment 4 Roman Joost 2015-11-18 06:50:51 UTC 
Patch available:

https://gerrit.beaker-project.org/#/c/4496/
Comment 5 Roman Joost 2015-11-30 05:08:15 UTC 
The attached patch in comment #4 was abandoned since the test I had added leaked into

https://gerrit.beaker-project.org/#/c/4495/

Therefore setting this bug to MODIFIED.
Comment 6 Roman Joost 2015-12-07 00:38:49 UTC 
This bug fix is included in beaker-server-22.0-0.git.110.c41cd4c which is currently available for testing here:

https://beaker-devel.app.eng.bos.redhat.com/
Comment 8 Dan Callaghan 2015-12-18 01:35:25 UTC 
This fix was merged into the release-21 branch but we won't be producing any further 21.x bug fix releases. This will be included with 22.0 instead.
Comment 9 Dan Callaghan 2016-01-14 05:32:58 UTC 
Beaker 22.0 has been released.