Skip to content
This repository has been archived by the owner on May 23, 2022. It is now read-only.

SOFTWARE-3342 Xrootd http #411

Merged
merged 4 commits into from
Aug 3, 2018
Merged

Conversation

efajardo
Copy link
Contributor

@efajardo efajardo commented Aug 2, 2018

Adding the HTTP documentation as optional part of the XRootD configuration without the specific CMS parts.

@efajardo efajardo requested a review from brianhlin August 2, 2018 20:50
@bbockelm
Copy link

bbockelm commented Aug 2, 2018

@efajardo - can you squash out the merge commit, rewrite the commit message to be more descriptive, and provide a description in the PR?

Copy link

@bbockelm bbockelm left a comment

Choose a reason for hiding this comment

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

It's a good start (based on the XrootdHTTP document, right?). However, I think we need to refine it a bit to integrate it better in the OSG docs.

1. Modify `/etc/xrootd/xrootd-clustered.cfg` and add the following lines:

:::file
sec.protocol /usr/lib64 gsi \
Copy link

Choose a reason for hiding this comment

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

This line is correct for enabling authorization, but is unrelated to HTTP / HTTPS. We should find the xrootd-lcmaps documentation and reference that -- make sure that doc uses the same line too!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

sure

if exec xrootd
xrd.protocol http:1094 libXrdHttp.so
http.cadir /etc/grid-security/certificates
http.cert /etc/grid-security/xrd/xrdcert.pem
Copy link

Choose a reason for hiding this comment

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

Note the hostcert path doesn't match the documentation earlier in the page. Which one do we suggest in the shipping configs?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good catch I will make it match

User-agent: *
Disallow: /

1. Testing the configuration
Copy link

Choose a reason for hiding this comment

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

Start the service, of course!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good catch, I will link to the manage services

-authzfunparms:--lcmapscfg,/etc/xrootd/lcmaps.cfg,--loglevel,4 \
-gmapopt:10 \
-gmapto:0
if exec xrootd
Copy link

Choose a reason for hiding this comment

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

What happens if if exec xrootd is left out? Does it mess up the cmsd? Or is this a line we've been copy/pasting for years and never was really required?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yep it does:

180802 14:15:02 27686 Meter: Write access and staging prohibited.
------ server@gftp-2.t2.ucsd.edu phase 2 server initialization completed.
180802 14:15:02 27686 XrdOpen: Unable to bind socket to port 1094; address already in use
------ cmsd server@gftp-2.t2.ucsd.edu:1094 initialization failed.````


#### (Optional) Enable HTTP based Writes

The primary changes are to the Authfile; you will need to add several a (all) authorizations to where users need to be able to write. Here's an example:
Copy link

Choose a reason for hiding this comment

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

This is all about authorization, which probably belongs to a different section in the documentation.

The CMS doc made this a separate section because so many CMS T2s had authorization completely disabled (which didn't matter that much when it was read-only).


The [upstream documentation](http://xrootd.org/doc/dev49/sec_config.htm#_Toc517294132) has further information on the Authfile format.

!!! warning
Copy link

Choose a reason for hiding this comment

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

This is a useful warning to keep around in the HTTP section.

Copy link

@bbockelm bbockelm left a comment

Choose a reason for hiding this comment

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

I still think of this as a good start, but not completing the ticket. I plan to merge this but make notes in the ticket that it should be one of several topical XRootD documents.

@bbockelm bbockelm merged commit 612939e into opensciencegrid:master Aug 3, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants