Project

General

Profile

Bug #15333

ProjectId is missing double quotes

Added by Parag Mhashilkar almost 4 years ago. Updated about 2 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Parag Mhashilkar
Category:
-
Target version:
Start date:
01/26/2017
Due date:
% Done:

0%

Estimated time:
First Occurred:
Occurs In:
Stakeholders:

OSG

Duration:

Description

Hi GlideinWMs Folks,

I know you are working on the 3.2.17 release and I wonder if you can include this small pull request on it:

https://github.com/holzman/glideinWMS/pull/9

It’s a bug fix. And is already on test and working on the ITB GOC factory,

Thanks,

Edgar M Fajardo Hernandez

History

#1 Updated by Parag Mhashilkar almost 4 years ago

  • Status changed from New to Resolved

Merged relevant changes to v3/15333 rather than pull request.

git pull https://github.com/efajardo/glideinWMS.git ca6ec6f0c7f599f8400e216b449047aa09a2a10f

#2 Updated by Marco Mambelli over 3 years ago

  • Status changed from Resolved to Closed

#3 Updated by Parag Mhashilkar almost 3 years ago

  • Stakeholders updated (diff)

#4 Updated by Marco Mambelli about 2 years ago

Adding also an email exchange that clarifies why only one of the changes in the pull request was accepted:

Hi Edgar,
yes, a group credential of type=“grid_proxy+project_id” would work, you could also add other information in the credential, e.g. type=“grid_proxy+my_other_id+project_id” 
As long as all the + separated tokens required by the entry are provided by the credential and are appearing in some order in the credential type, this credential will be valid for that entry.

Cheers,
Marco

On Feb 3, 2017, at 4:30 PM, Edgar M Fajardo Hernandez <emfajardohernandez@physics.ucsd.edu> wrote:

Hi Parag,

Well the Comet entry looks like:

http://glidein-itb.grid.iu.edu/factory/monitor/factoryEntryStatusNow.html?entry=OSG_T2_US_UCSD_gw6_Comet_mcore

AuthMethod    grid_proxy+project_id

And the credentials in the group look like:

<credential absfname=“/home/frontend/.globus/x509_cmspilot01_uscms_pilot.proxy" project_id="csd428" security_class="cmspilot" trust_domain="grid" type="grid_proxy"/>

Btw, I tested the change and it worked :)

So are you saying that the group credential should be type=“grid_proxy+project_id”

Edgar M Fajardo Hernandez
emfajardohernandez@physics.ucsd.edu

On Feb 3, 2017, at 2:24 PM, Parag A Mhashilkar <parag@fnal.gov> wrote:

Hi Edgar

I don’t think that your changes are correct. Factory will use only one credential and that credential should provide all the required pieces required by the entry’s supported auth methods. However, a credential can have additional bits that factory may not care about. So, to me, auth_methods should be subset of the options supported by the credential, i.e type

I suspect if your config may need to be looked at. What does your factory entry look like? Also corresponding group/credential.

-Parag
On Feb 3, 2017, at 3:40 PM, Edgar Fajardo <notifications@github.com> wrote:

Looking at the actual printout:

[2017-02-03 13:27:35,436] DEBUG: glideinFrontendInterface:448: am_set in support_auth_method: set(['project_id', 'grid_proxy']) [2017-02-03 13:27:35,436] DEBUG: glideinFrontendInterface:449: type_set in support_auth_method: set([u'grid_proxy'])

It is was clear the comparison was not correctly done.

You can view, comment on, or merge this pull request online at:

https://github.com/holzman/glideinWMS/pull/10

Commit Summary

    • Merge pull request #1 from holzman/master
    • Fix bug with several authentication methods
File Changes

    • M frontend/glideinFrontendInterface.py (2)
Patch Links:

    • https://github.com/holzman/glideinWMS/pull/10.patch
    • https://github.com/holzman/glideinWMS/pull/10.diff
—

Also available in: Atom PDF