mistercrunch opened a new pull request, #31606:
URL: https://github.com/apache/superset/pull/31606

   fix: docker-compose-image-tag fails to start
   
   Recently I did some work to use `uv pip install` in place of plain `pip` in 
our docker images. This made builds significantly faster, but broke 
compatibility between some docker compose setups.
   
   The issue is around the fact that `docker-compose-image-tag` mounts the 
local, newer, docker bootstrap scripts into the image, and those have evolve to 
use `uv`, which isn't installed in the image.
   
   One option would be to not mount the docker folder and use what's in the 
image, but I believe some of the script may have changed names or location, and 
`docker-compose-image-tag` specifies their location as it passes arguments to 
them.
   
   In any case, and at least for now, I decided to simply conditionally use uv 
or plain pip in the docker scripts based on what's installed in the image.
   
   Also added a CI step to make sure `docker compose -f 
docker-compose-image-tag.yml up` works.
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to