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

Add a couple clarifications regarding access requirements #164

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mwestphall
Copy link
Contributor

No description provided.

@@ -240,7 +250,7 @@ Once the web page is updated, run the following command to update the CA certifi
verify that the version of the CA certificates match the version that was promoted to release.

```bash
# moria.cs.wisc.edu
# dumbo.chtc.wisc.edu
Copy link
Collaborator

Choose a reason for hiding this comment

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

This doesn't work. Because we nolonger make an AFS specific tarballs. We should replace this step with something else.

Copy link
Collaborator

Choose a reason for hiding this comment

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

I'll update this step in a different PR.

Copy link
Contributor

Choose a reason for hiding this comment

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

@matyasselmeci @mwestphall @timtheisen one thing that I don't understand here is

run the following command to update the CA certificates in the tarball installation

This step is after the tarballs are built and copied to the relevant locations on AFS / OASIS. It doesn't look like we're running the update and tar'ing everything back up so this seems like a no-op to me? We also do these sorts of tests in the tarball verification script https://github.com/opensciencegrid/release-tools/blob/master/1-verify-tarballs#L35-L63

I think we can just scratch this step except for verifying that the cadist page is updated

Copy link
Collaborator

Choose a reason for hiding this comment

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

Correct. I just ensure that the cadist page updates.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Just make sure that the page https://repo.opensciencegrid.org/cadist/ updates.

!!! note
Ensure you've [added your SSH key to oasis](https://osg-htc.org/docs/common/contact-registration/#oasis-managers-adding-an-ssh-key) prior
to running this step. Also, ensure you've [enabled SSH agent forwarding](https://docs.github.com/en/authentication/connecting-to-github-with-ssh/using-ssh-agent-forwarding)
from a host with your SSH key (such as your laptop) to dumbo.
Copy link
Contributor

Choose a reason for hiding this comment

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

Why do we need SSH agent forwarding? We should avoid prefer ProxyJump wherever we can https://www.infoworld.com/article/2266099/proxyjump-is-safer-than-ssh-agent-forwarding.html

Copy link
Collaborator

Choose a reason for hiding this comment

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

I don't think we need SSH agent forwarding. I don't use it and I can upload tarballs just fine. I'd delete that requirement.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@timtheisen the instructions specify running this step on dumbo, so the presumed setup is that you're ssh'ing from your local laptop to dumbo, then to oasis. Oasis does key-based ssh authentication, and I was running into issues not having my ssh key available on dumbo when running this step. My first thought of how to work around this was agent forwarding from my laptop to dumbo (and then to oasis), we can look into other approaches though if that's not the preferred aproach.

Copy link
Collaborator

Choose a reason for hiding this comment

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

I see, I have my private key on dumbo. That's the difference.

Copy link
Contributor

@brianhlin brianhlin Feb 14, 2025

Choose a reason for hiding this comment

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

We discussed this locally and have ideas to do pulls from Pelican instead of pushes so that we don't have to keep private keys on dumbo

See https://opensciencegrid.atlassian.net/browse/SOFTWARE-6094

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.

3 participants