Skip to content

Use grid-mapfile downloaded from Topology#151

Merged
matyasselmeci merged 3 commits intoopensciencegrid:V3-branchfrom
matyasselmeci:pr/authfile-update-merge-grid-mapfile.SOFTWARE-5468
Mar 20, 2024
Merged

Use grid-mapfile downloaded from Topology#151
matyasselmeci merged 3 commits intoopensciencegrid:V3-branchfrom
matyasselmeci:pr/authfile-update-merge-grid-mapfile.SOFTWARE-5468

Conversation

@matyasselmeci
Copy link
Copy Markdown
Contributor

Two changes necessary to actually use the grid-mapfile downloaded from Topology for the cache and origi n (instead of /etc/grid-security/grid-mapfile):

  1. For backward compat, have authfile-update add local changed from /etc/grid-security/grid-mapfile to the downloaded mapfile
  2. Set some variables in the XRootD config
    The variables are necessary because the configs in the osg-xrootd metapackage are actually what control how the grid-mapfile is used, so we'll need to modify those to make use of the new variables here.

…d from Topology (SOFTWARE-5468)

This lets us change the default grid-mapfile location in the xrootd config
while preserving backward compatibility.
These are variables that an upcoming version of osg-xrootd will use to
define where the grid-mapfile is (instead of hardcoding
`/etc/grid-security/grid-mapfile`) and what the behavior should be if
there's no mapping (instead of hardcoding `trymap`). It has no effect
with previous versions of osg-xrootd.

It has to come before 50-osg-http.cfg and 50-osg-xrdvoms.cfg.
@matyasselmeci matyasselmeci requested a review from brianhlin March 18, 2024 23:45
Copy link
Copy Markdown
Member

@brianhlin brianhlin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@matyasselmeci matyasselmeci merged commit 24d7277 into opensciencegrid:V3-branch Mar 20, 2024
@matyasselmeci matyasselmeci deleted the pr/authfile-update-merge-grid-mapfile.SOFTWARE-5468 branch March 20, 2024 01:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants