Docker image with a registry port cannot be parsed
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 45/100
Línea de trabajo
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.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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
- Lenguaje dominante
- Python
- Estrellas
- 327
- Forks
- 170
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de QuantConnect/lean-cli
-
project-create generates an invalid class name when the project name starts with a digitPosiblemente ocupada @Amadeus-22 la tomó hace 2 días. Abierto
Dificultad 2/5 Menos de una hora Aptitud para principiantes 40/100
QuantConnect/lean-cli#673 ·
-
lean cloud live deploy --notify-webhooks fails when the address is a URLPosiblemente ocupada @Amadeus-22 la tomó hace 2 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 48/100
QuantConnect/lean-cli#671 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 55/100
QuantConnect/lean-cli#661 · 1 comentario ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
QuantConnect/lean-cli#619 ·
-
Implements AutocompleteQuizá libre de nuevo @shreejaykurhade la tomó hace 195 días y no hay ningún pull request abierto. Abiertofeature
QuantConnect/lean-cli#602 · 6 comentarios · 3 reacciones · 1 asignado ·
Todos los issues de QuantConnect/lean-cli
Issues similares
-
Broken link in index.rstAbiertodocumentation
Dificultad 1/5 Menos de una hora Aptitud para principiantes 65/100
ansys/pydpf-core#3547 ·
Los mantenedores suelen responder en 1 día
-
core
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
vectorize-io/hindsight#5457 ·
Los mantenedores suelen responder en 1 día
-
[Bug]: LangChain drops OpenAI Responses text blocks from session recordingPosiblemente ocupada @ktz03 la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
volcengine/OpenViking#5806 ·
Los mantenedores suelen responder en 1 día
-
HTML: <template> content is extracted as document textPosiblemente ocupada @ryanmeowy la tomó hoy. Abiertobug html
Dificultad 1/5 Menos de una hora Aptitud para principiantes 82/100
docling-project/docling#4714 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
APIv2 event data accepts a non-string reply and a NaN upper_boundPosiblemente ocupada @awss1i la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
freedomofpress/securedrop#7946 · 1 comentario ·
Los mantenedores suelen responder en 2 días