Repository navigation
fix(ci): lowercase image ref in yt-dlp smoke test - #866
Conversation
Docker registry refs must be lowercase, but $IMAGE_PREFIX preserves the GitHub owner's case (LucasSantana-Dev), causing 'invalid reference format' on every main-branch build since #854. Pipe through tr to normalize before docker run.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
There was a problem hiding this comment.
LucasSantana-Dev has reached the 50-review limit for trial accounts. To continue receiving code reviews, upgrade your plan.
|
Warning Rate limit exceeded
You’ve run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Failed to generate code suggestions for PR |
|
Hotfix #866 changed only the workflow file, so docker-publish did not retrigger and v2.11.0 was never re-published. Add workflow_dispatch and include the workflow file in the push path filter.
This branch was successfully deployed
1 participant
Hotfix for failing docker-publish on main (post-v2.11.0).
The yt-dlp smoke test introduced in #854 references
${{ env.IMAGE_PREFIX }}-botdirectly, which preserves the GitHub owner's case (LucasSantana-Dev). Docker registry refs must be lowercase, so every main-branch build since #854 has failed with:This cascades into homelab deploy being skipped (workflow_run guard fails when docker-publish fails).
Fix: pipe the image ref through
tr '[:upper:]' '[:lower:]'beforedocker run. The image was already pushed to the registry under the lowercase tag (docker/metadata-actionlowercases automatically) — only the smoke-test step referenced the wrong case.Blocks v2.11.0 deploy until merged.