Docker image with a registry port cannot be parsed
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 45/100
Direzione di ricerca
Start at DockerImage.parse in lean/models/docker.py and reproduce the two failing cases from the issue. The rule is to treat only a final colon segment without / as the tag; add cases covering registry ports and tags to the existing DockerImage parse tests (look under tests/) so both parse and error paths are covered. Note that linked PR #670 is already open against this issue, so confirm with maintainers before starting.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Expected Behavior
An image hosted on a registry that listens on a port, for example localhost:5000/lean or registry.example.com:5000/quantconnect/lean:123, can be used with --image and with lean config set engine-image / research-image.
Actual Behavior
DockerImage.parse splits on every colon, so the registry port is read as the tag:
localhost:5000/lean -> DockerImage(name='localhost', tag='5000/lean')
localhost:5000/lean:latest -> ValueError: too many values to unpack (expected 2)
Potential Solution
Only treat what follows the last colon as the tag, and only when it does not contain a /. I can open a PR with the fix and tests.
Reproducing the Problem
from lean.models.docker import DockerImage
DockerImage.parse("localhost:5000/lean")
DockerImage.parse("localhost:5000/lean:latest")
System Information
lean-cli master at 20e58aa, Python 3.13, Linux.
Checklist
- I have completely filled out this template
- I have confirmed that this issue exists on the current
masterbranch - I have confirmed that this is not a duplicate issue by searching issues
- Lingua principale
- Python
- Stelle
- 327
- Fork
- 170
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di QuantConnect/lean-cli
-
project-create generates an invalid class name when the project name starts with a digitForse già presa @Amadeus-22 l’ha presa 5 giorni fa. Aperta
Difficoltà 2/5 Meno di un'ora Idoneità per principianti 40/100
QuantConnect/lean-cli#673 ·
-
lean cloud live deploy --notify-webhooks fails when the address is a URLForse già presa @Amadeus-22 l’ha presa 5 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 48/100
QuantConnect/lean-cli#671 ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 55/100
QuantConnect/lean-cli#661 · 1 commento ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
QuantConnect/lean-cli#619 ·
-
Implements AutocompleteForse di nuovo libera @shreejaykurhade l’ha presa 198 giorni fa e non c’è nessuna pull request aperta. Apertafeature
QuantConnect/lean-cli#602 · 6 commenti · 3 reazioni · 1 assegnatario ·
Tutte le issue di QuantConnect/lean-cli
Issue simili
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 83/100
PedestrianDynamics/pyFDS-Evac#766 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 1-3 ore Idoneità per principianti 91/100
alchaincyf/nuwa-skill#86 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
I maintainer di solito rispondono entro 2 giorni
-
Docs Needs Triage
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
pandas-dev/pandas#71055 ·
I maintainer di solito rispondono entro 1 giorno
-
[Bug]: graphify reads files that git's global ignore file hidesForse già presa @smngvlkz l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
Graphify-Labs/graphify#4335 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno