Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Have origin advertise to the director service #18

Merged

Conversation

bbockelm
Copy link
Collaborator

This draft is the first piece of the periodic advertisement of the origin to the director service. When complete, it'll contain:

  • New APIs in the director for advertising services
  • Periodic go routine in the origin code to invoke new APIs
  • Bearer token generation / verification for authorizing the advertisement.

Goal is that all origins connecting to the director advertise their capabilities directly (and verified by the namespace registration service) instead of relying on OSG Topology to bootstrap the namespace.

@bbockelm bbockelm force-pushed the origin_director_registration branch from b73113a to f8d549b Compare July 28, 2023 21:54
bbockelm added 3 commits July 29, 2023 08:11
This implements token creation/verification routines which should
allow the origin to advertise its presence to the director.  This
includes the director querying the namespace registration service
for the corresponding public key.
This work-in-progress commit sketches out a new REST API for the
origin to register itself (and its available namespaces) with the
director.

The bones on the server side are there; the invocation from the
origin (and the generation of the data at the origin) are not.
@bbockelm bbockelm force-pushed the origin_director_registration branch from 11d8c1b to e05b757 Compare July 29, 2023 13:13
bbockelm added 3 commits July 29, 2023 08:19
- Skip the topology queries if we're not the OSDF director.
- Rename the topology endpoint config param from `NamespaceURL` to
  `TopologyNamespaceURL`; `NamespaceURL` is a duplicate param name.
- Have a default value present for the topology endpoint in the
  OSDF configuration.
@bbockelm bbockelm force-pushed the origin_director_registration branch from e05b757 to 8376c19 Compare July 29, 2023 13:20
@bbockelm bbockelm marked this pull request as ready for review August 1, 2023 13:34
@bbockelm
Copy link
Collaborator Author

bbockelm commented Aug 1, 2023

Going to self-merge this one in order to get the federation auto-discovery pieces in place. This will help improve the namespace URL discovery (necessary for a separate cleanup of the namespace registration).

The current state of the branch is the origin periodically advertises and the director responds on its URL -- however, the origin still needs to use the token it generates as part of the API call. Therefore, all the attempts to advertise end in an "Authorization denied".

@bbockelm bbockelm merged commit f0f5677 into PelicanPlatform:main Aug 1, 2023
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.

1 participant