Additional information and a check_setup.py script to aid with configuring silnlp.#1119
Additional information and a check_setup.py script to aid with configuring silnlp.#1119davidbaines wants to merge 3 commits into
Conversation
check_setup.py verifies env vars, MinIO connectivity (DNS/TCP/health), rclone config and mount, and ClearML auth, printing a remediation hint for each failure. Stdlib-only so it runs in a broken environment; WSL-aware VPN hints. The diagnose-setup skill documents the deeper investigation playbook for Claude Code users. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
check_setup.py now lists every MinIO/B2/ClearML variable on its own line with its value and where it is defined (profile file and line number), and detects vars defined in a profile but absent from the current environment. Secrets are partially masked by default; --show-secrets prints them in full. Also fix bucket_setup.md crontab instructions to use the full rclone path (cron's PATH may not include it). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
mshannon-sil
left a comment
mshannon-sil
left a comment
There was a problem hiding this comment.
Yeah it's a reasonable improvement to ask users specify the full path for rclone to the crontab, even if it's maybe not strictly required.
@mshannon-sil reviewed 2 files and all commit messages, and made 5 comments.
Reviewable status: 2 of 4 files reviewed, 4 unresolved discussions (waiting on davidbaines).
README.md line 158 at r1 (raw file):
## Bucket Setup 1. For MinIO, enable your VPN. On Windows this should be done outside of WSL: * To access the MinIO bucket, you will need VPN access. Reach out to a SILNLP dev team member for access.
Since the issue was needing to update the AllowedIPs, and we've run into that a few times now where users aren't aware that's needed, could you also add an update here about that? It can read something like "To access the MinIO bucket, you will need VPN access. Reach out to a SILNLP dev team member for access. You will receive a configuration file for the WireGuard VPN as well as an ip address to add to the list of AllowedIPs."
README.md line 175 at r1 (raw file):
* You will need to run the `source ./rclone_setup.sh minio` command each time you start your Linux environment. Every other step in these instructions should only be needed one time during initial setup. ## Checking Your Setup
Let's specify "Bucket Setup" here for clarity, since it's not checking other things about the setup like poetry packages.
.claude/skills/diagnose-setup/SKILL.md line 2 at r1 (raw file):
--- name: diagnose-setup
Can we rename the file and name to diagnose-bucket-setup?
silnlp/common/check_setup.py line 2 at r1 (raw file):
#!/usr/bin/env python3 """Diagnose a local SILNLP setup.
Similarly to before, can we name this check_bucket_setup.py?
Also, this file would fit better under silnlp/scripts.
2 participants
I've been trying to get silnlp working on Ubuntu and most of the setup was very smooth. Getting Minio working was tricky though. There was an additional AllowedIPs value that had to be added to the credentials file for wireguard. Fable wrote a skill and a check_setup.py script (which the skill can call). There were also updates to README.md and bucket_setup.md that should be reviewed carefully - the addition essentially says that to use crontab with the full path to the rclone.conf file. Which seems reasonable to me but I can't say I know for sure, except that it is working on my machine.
This change is