updated data image #69
Conversation
image_definitions/data/Dockerfile
Outdated
| RUN pip install databricks-cli pre-commit tomli |
There was a problem hiding this comment.
ideally add "azure-cli" here rather than separate line otherwise we create unnecessary docker layers, happy to approve now.
There was a problem hiding this comment.
Azure cli is already in the base image?
There was a problem hiding this comment.
Hi Jack, I can see it in base image but I think its getting removed from data image ,please refer this
RUN apk del py-pip \
&& apk del python3-dev py3-pip
&& rm -rf ~/.cache/pip \
&& apk del python3
There was a problem hiding this comment.
@russellseymour shouldn't az cli be available?
There was a problem hiding this comment.
Sorry, juust catching up with PRs
Yes I was just looking at this.
If we are upgrading Python herte then I think we should be consistent and upgrage Python in the base image.
I do not want to have to install az twice.
|
Hey @russellseymour, has this PR become part of our new images? |
📲 What
Added jq and azure-cli in data image
🤔 Why
Why have you implemented it?
Required for Data platform