Project

General

Profile

Bug #25219

frontend over-enthusiastic generating tokens for entries

Added by Dennis Box 2 months ago. Updated about 1 month ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
-
Target version:
Start date:
11/17/2020
Due date:
% Done:

0%

Estimated time:
First Occurred:
Occurs In:
Stakeholders:
Duration:

Description

Reported by Marco Mascheroni and Saqib Haleeb. I have not been able to reproduce the first problem they report, but a fix for the other two will be in this branch

Hi Dennis and Marco,

Saqib installed the frontend in itb today.
Here is the first error I detected1. I solved it by moving the definion of tmpnm right after the 'try'.

I also noticed the frontend user has to be in the sudoers2.

Finally, the frontend is terribly slow. Because ITB is a small machine which we often use for scale test puposes, we set group_parallel_workers="1". It seems the frontend is trying to create a condor token for each of the entry3, but each call takes several seconds4. And given that we have hundreds of entries, multiple frontend groups, and a single process that's an issue.

I'll need to look into it more tomorrow, but I wanted to give you a quick feedback before going to bed!

Talk to you soon,

Marco

Associated revisions

Revision 10a2ba7b (diff)
Added by Dennis Box 2 months ago

fix for problems 1) and 3) in redmine issue #25219

History

#1 Updated by Dennis Box 2 months ago

Correction I am checking in a fix for problems (1) and (3) in the email

#2 Updated by Dennis Box 2 months ago

  • Assignee changed from Dennis Box to Marco Mascheroni
  • Status changed from New to Feedback

Hi Marco,
glideinFrontendElement.py in branch v37/25219 is a fix for problems 1) and 3) in your email.
Can you please test it on itb as a patch for v3.7.1

#3 Updated by Marco Mascheroni about 2 months ago

Hi Dennis,

this looks good. I need to digest the part with:

+            and condor_version != 'default' \

What if the default condor version is > than 8.9.2 in the factory?

Anyway, this is definitely an improvement. I would merge it and open a new ticket for "Map the default condor version to its actual number for token handling"

#4 Updated by Marco Mascheroni about 2 months ago

  • Assignee changed from Marco Mascheroni to Dennis Box

#5 Updated by Dennis Box about 1 month ago

  • Status changed from Feedback to Resolved

Also available in: Atom PDF