Bug 640326 - condor_userprio is case sensitive to accounting groups
Summary: condor_userprio is case sensitive to accounting groups
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise MRG
Classification: Red Hat
Component: condor
Version: 1.3
Hardware: All
OS: All
low
medium
Target Milestone: 1.3.2
: ---
Assignee: Erik Erlandson
QA Contact: Martin Kudlej
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2010-10-05 14:59 UTC by Erik Erlandson
Modified: 2011-02-15 12:16 UTC (History)
5 users (show)

Fixed In Version: condor-7.4.5-0.2
Doc Type: Bug Fix
Doc Text:
Previously, the accountant treated accounting groups as case sensitive when accounting groups with multiple cases (e.g. "group" versus "Group" versus "GROUP") were submitted. Due to this behavior, different cases were incorrectly managed as different groups, which shows up in condor_userprio and affects the internal use of group priorities. With this update, the accountant uses the correct case-folding logic so that accounting groups are case insensitive. Now, accounting groups can be submitted using any case, and are correctly treated as case insensitive by the accountant.
Clone Of:
Environment:
Last Closed: 2011-02-15 12:16:03 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2011:0217 0 normal SHIPPED_LIVE Red Hat Enterprise MRG Messaging and Grid bug fix and enhancement update 2011-02-15 12:10:15 UTC

Description Erik Erlandson 2010-10-05 14:59:41 UTC
Description of problem:
condor_userprio appears to be improperly distinguishing between "a.user" and "A.user"

Steps to Reproduce:
configuration:
NEGOTIATOR_DEBUG = D_FULLDEBUG | D_MATCH
SCHEDD_INTERVAL = 15
NEGOTIATOR_USE_SLOT_WEIGHTS = FALSE
NUM_CPUS = 20
GROUP_NAMES = A, B
GROUP_QUOTA_DYNAMIC_A = 0.25
GROUP_QUOTA_DYNAMIC_B = 0.2
GROUP_AUTOREGROUP_A = FALSE
GROUP_AUTOREGROUP_B = FALSE

submit file:
$ cat test.submit 
universe = vanilla
cmd = /bin/sleep
args = 5m
+AccountingGroup = "a.user"
queue 5
+AccountingGroup = "A.user"
queue 5
+AccountingGroup = "b.user"
queue 5
+AccountingGroup = "B.user"
queue 5

Scenario:
1. submit jobs
$ condor_submit test.submit 
Submitting job(s)....................
20 job(s) submitted to cluster 1.


  
Actual results:
when I list condor_userprio
(classads from negotiator) it still distinguish group "A" from group "a".

$ condor_userprio -l
LastUpdate = 1286262730
Name1 = "b.user.eng.bos.redhat.com"
Priority1 = 0.504585
ResourcesUsed1 = 2
...
Name2 = "B.user.eng.bos.redhat.com"
Priority2 = 0.504585
ResourcesUsed2 = 2
...
Name3 = "a.user.eng.bos.redhat.com"
Priority3 = 0.505067
ResourcesUsed3 = 3
...
Name4 = "A.user.eng.bos.redhat.com"
Priority4 = 0.507183
ResourcesUsed4 = 2
...
Name5 = "A"
Priority5 = 0.507183
ResourcesUsed5 = 2
...
Name6 = "B"
Priority6 = 0.504585
ResourcesUsed6 = 2
...
Name7 = "a"
Priority7 = 0.505067
ResourcesUsed7 = 3
...
Name8 = "b"
Priority8 = 0.504585
ResourcesUsed8 = 2


Expected results:
I expect it should show that group "A" ("a") uses 5 resources.


Additional Info:
See also: 
https://bugzilla.redhat.com/show_bug.cgi?id=632476
https://bugzilla.redhat.com/show_bug.cgi?id=617709

Comment 1 Erik Erlandson 2010-11-19 21:24:52 UTC
Updated accountant logic includes case folding for groups:
V7_4-BZ619557-HFS-tree-structure

Comment 3 Erik Erlandson 2010-12-21 18:53:33 UTC
    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:
Cause:
Submission of accounting groups with multiple cases (e.g. "group" versus "Group" versus "GROUP").

Consequence:
The accountant treated accounting groups as case sensitive, so different cases were incorrectly managed as different groups, which shows up in condor_userprio and also affects internal use of group priorities.

Fix:
The accountant was updated with proper case-folding logic so that accounting groups are case insensitive.

Result:
Accounting groups can be submitted using any case, and are correctly treated as case insensitive by the accountant.

Comment 4 Martin Kudlej 2011-01-14 14:10:04 UTC
Tested on RHEL5.6/4.9 x x86_64/i386 with condor-7.4.4-0.17 and it is case sensitive.

Comment 5 Martin Kudlej 2011-01-14 15:55:26 UTC
Tested on RHEL5.6/4.9 x x86_64/i386 with condor-7.4.5-0.6 and it is case
insensitive. --> VERIFIED

Comment 6 Florian Nadge 2011-02-09 16:25:41 UTC
    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,11 +1 @@
-Cause:
+Previously, the accountant treated accounting groups as case sensitive when accounting groups with multiple cases (e.g. "group" versus "Group" versus "GROUP") were submitted. Due to this behavior, different cases were incorrectly managed as different groups, which shows up in condor_userprio and affects the internal use of group priorities. With this update, the accountant uses the correct case-folding logic so that accounting groups are case insensitive. Now, accounting groups can be submitted using any case, and are correctly treated as case insensitive by the accountant.-Submission of accounting groups with multiple cases (e.g. "group" versus "Group" versus "GROUP").
-
-Consequence:
-The accountant treated accounting groups as case sensitive, so different cases were incorrectly managed as different groups, which shows up in condor_userprio and also affects internal use of group priorities.
-
-Fix:
-The accountant was updated with proper case-folding logic so that accounting groups are case insensitive.
-
-Result:
-Accounting groups can be submitted using any case, and are correctly treated as case insensitive by the accountant.

Comment 7 errata-xmlrpc 2011-02-15 12:16:03 UTC
An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on therefore solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.

http://rhn.redhat.com/errata/RHBA-2011-0217.html


Note You need to log in before you can comment on or make changes to this bug.