Skip to content

Point the SSH password scripts at runpod/pod-ssh-password - #952

Merged
justinwlin merged 1 commit into
mainfrom
justinlin/ssh-password-org-repo
Nov 6, 2026
Merged

justinwlin merged 1 commit into
mainfrom
justinlin/ssh-password-org-repo

Conversation

@justinwlin

Copy link
Copy Markdown
Collaborator

The SSH password and pod-to-pod transfer scripts moved from a personal repo to runpod/pod-ssh-password. This updates the links and wget commands on two pages:

  • pods/configuration/use-ssh.mdx
  • community-solutions/ssh-password-migration/overview.mdx

The scripts are the same. The only changes are their own URLs and placeholder IPs in the README. Both raw URLs return 200.

@mintlify

mintlify Bot commented Nov 6, 2026 •

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated
runpod-docs 🟢 Ready View Preview Nov 6, 2026, 4:47 AM

💡 Tip: Enable Automations to automatically generate PRs for you.

@OneAsterisk OneAsterisk left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Claude review, round 1

Verdict: Approved — two pages now point at runpod/pod-ssh-password; every new URL resolves and the scripts behave the same.
Approval gate: 0 Critical, 0 High, 0 Medium+Low → approved.

Existing feedback

  • mintlify[bot]: preview deployment only, no claims. The Mintlify Deployment status reports skipped at this head while the preview comment shows the preview as Ready; nothing to act on.

PR description vs code

Claim Status Where
Links and wget commands updated on pods/configuration/use-ssh.mdx ✅ use-ssh.mdx:146, use-ssh.mdx:149
Links and wget commands updated on community-solutions/ssh-password-migration/overview.mdx ✅ overview.mdx:12, overview.mdx:49, overview.mdx:64
Both raw URLs return 200 ✅ raw.githubusercontent.com/runpod/pod-ssh-password/main/passwordrunpod.sh and .../SCPMigration both 200; the blob/main/passwordrunpod.sh page link also 200
"The scripts are the same. The only changes are their own URLs and placeholder IPs in the README." ⚠️ Functionally identical. passwordrunpod.sh changes are URL-only (header lines 4-5, self-referencing wget at line 317). SCPMigration also changed "RunPod" to "Runpod" in the docstring, SCRIPT_NAME (line 17), and the welcome banner (line 551), and dropped the Author: Assistant line. No logic changes.

Findings

None.

Nits

  • PR body: "the scripts are the same" slightly understates the SCPMigration diff (casing and author-line changes, see table above). Worth one clause so the next reader does not have to diff the repos.

Simplification opportunities

  • Follow-up, not this PR: the new README switched its sample output to RFC 5737 placeholder IPs (203.0.113.x), but overview.mdx:54, overview.mdx:93, and use-ssh.mdx:100, :164, :171, :174, :177 still show real-looking public IPs. Same treatment there would keep the docs consistent with the repo. Pre-existing on main, so out of scope here.

Verified clean

  • The diff touches only the five URL lines; no unrelated changes in either page.
  • runpod/pod-ssh-password is public, unarchived, default branch main, and contains README.md, SCPMigration, passwordrunpod.sh.
  • The SCPMigration URL dropped the refs/heads/main form in favor of main; both resolve, and main matches the other links.
  • No remaining references to justinwlin/Runpod-SSH-Password, passwordrunpod, or SCPMigration anywhere else in the repo (docs.json and community-solutions/overview.mdx only reference the docs page path, which is unchanged).
  • Check tooltip imports passed at this head.
  • The two other justinwlin/* repos still linked from the docs (Runpod-Network-Volume-Storage-Tool, Runpod-GPU-And-Serverless-Base) are different projects and are not part of this move.

Questions for the author

  • justinwlin/Runpod-SSH-Password is still live and not archived, so it is now a second copy rather than a redirect. Is the plan to archive it or add a pointer to the org repo, so the two do not drift?

Reviewed by Claude (claude-fable-5-1) via the pr-review skill. Tokens: 228 in / 5,451 out / 493,982 cache read / 72,829 cache write = 572,490. Estimated cost: $1.86 at $10/$50 per MTok in/out, $0.25 cache read, $20 cache write (1-hour TTL, 2× input). Round 1 of this PR. This is automated assistance; a human owns the final call.

@justinwlin
justinwlin merged commit 4ec7ea8 into main Nov 6, 2026
2 checks passed
@justinwlin
justinwlin deleted the justinlin/ssh-password-org-repo branch November 6, 2026 04:52

This branch was successfully deployed

1 active deployment
staging — d055bc3e Deployed Nov 6, 2026 by mintlify[bot]
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