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

Added SSH over HTTPS port warning to github bootstrapping docs #1779

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

Conversation

dlactin
Copy link

@dlactin dlactin commented Jan 19, 2024

Added a warning for an unlikely but potential issue for folks who are using SSH over port 443 when bootstrapping flux.

I ran into this issue and even with the host keys set to the ones in that link I was unable to get this to work until I changed the port back to 22.

@kingdonb kingdonb force-pushed the add-ssh-over-https-port-docs branch from 7bd4255 to a9c48bc Compare August 25, 2024 16:34
@kingdonb
Copy link
Member

kingdonb commented Aug 25, 2024

Sorry for the delay reviewing this! 😅

The issue being addressed isn't really clear to me, could you say a bit more about who this issue affects? How does it happen? I looked at the docs linked and it doesn't clarify the problem, or how to solve it from a Flux user perspective.

Are the affected users Enterprise GitHub customers who have set up SSH on port 443, for some firewall reason?

@dlactin
Copy link
Author

dlactin commented Aug 29, 2024

Hello!

Not a problem.

It's been a while since I've looked at this, but from what I recall, your second comment is spot on.

Are the affected users Enterprise GitHub customers who have set up SSH on port 443, for some firewall reason?

This was implemented while a few of us were at a conference that had port 22 blocked, and it caused our attempts at bootstrapping a cluster with flux to fail without a generic host key mismatch error.

Definitely not a flux problem, but something I thought would be worth mentioning in the docs for folks bootstrapping flux this way.

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