Project

General

Profile

Bug #2892

Frontend should not match jobs with no proxy to sites that require glexec to be used

Added by Parag Mhashilkar over 8 years ago. Updated over 7 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Douglas Strain
Category:
Frontend
Target version:
Start date:
08/20/2012
Due date:
% Done:

0%

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

Description

While requesting glideins, frontend ignores that presence of proxy in the job classad. This results in glideins requested from sites that require the use of glexec. Altough jobs still don't match the startds, this is waste of resources which could be avoided at frontend level.

History

#1 Updated by Parag Mhashilkar over 8 years ago

  • Assignee changed from Parag Mhashilkar to Burt Holzman

#2 Updated by Parag Mhashilkar over 8 years ago

  • Target version changed from v2_7_x to v2_6_2

#3 Updated by Douglas Strain about 8 years ago

  • Assignee changed from Burt Holzman to Douglas Strain

#4 Updated by Douglas Strain about 8 years ago

Oops, added an update to the wrong ticket. Stay tuned for updates on this ticket.

#5 Updated by Parag Mhashilkar about 8 years ago

  • Target version changed from v2_6_2 to v2_7_x

#6 Updated by Douglas Strain about 8 years ago

  • Status changed from New to Feedback
  • Assignee changed from Douglas Strain to Parag Mhashilkar

This has been finished in branch_v2plus_2892.
Commit 81607fefcf693c399ce76995a6f9822f5f78a6c4.

Please review at your leisure.

#7 Updated by Parag Mhashilkar almost 8 years ago

  • Assignee changed from Parag Mhashilkar to Douglas Strain

Looks ok to merge after following changes:

glideinFrontendElement.py: For GLIDEIN_REQUIRE_VOMS (line 552)& GLIDEIN_REQUIRE_GLEXEC_USE (line 556), use dict.get() instead of combo has_key()+dict[key]

glideinFrontendLib: Can we eliminate has_key() and use 'in' instead?

#8 Updated by Douglas Strain almost 8 years ago

  • Status changed from Feedback to Resolved

Reviewer comments have been implemented and code has been merged to branch_v2plus and master.

#9 Updated by Parag Mhashilkar almost 8 years ago

  • Target version changed from v2_7_x to 293

#10 Updated by Parag Mhashilkar over 7 years ago

  • Target version changed from 293 to v2_7

#11 Updated by Parag Mhashilkar over 7 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF