Hacker News Reader

Comment by api_or_ipa | Hacker News Reader

the dockerfile could use some work, no CMD, calling apt-get update multiple times (slow), wget'ing a non-https url leaves you exposed to MIMs and finally: why are you downloading aptitude when (from what i can see) apt-get would work just fine?

View on HN

Hi! The second apt-get update is there so microsoft-mlserver-all-9.2.1 would be found. The wget was used as per the installation instructions by Microsoft. I used aptitude because I encountered problems with the package dependencies. I'll trying a CMD in the near future. Thanks! :D

You should have installed apt-utils, which would have resolved the dependencies. Also, basing your image off ubuntu:16.04 would work a lot better, because that's the actual target.

I'm not sure what the purpose of the second-to-last line[1] is for, either - it just lists a directory.

[1] https://github.com/johnpaulada/microsoftmlserver-docker/blob...

It seems like there isn't much Linux experience in the company that once referred to it as "Cancer"

This isn't a Microsoft product.

Yes at least some of the people. Their instructions for getting a new SQL 2017 Docker image (somewhere not sure if it was the official documentation or Microsoft hosted blogs that should know better) included apt get update on the Linux container after download.

I was bit surprised so much is out of date for the official container.

But I mean... I’m new to docker and this was my first introduction... but I’m pretty sure from what I’ve read that it’s a huge no no.

The author of this Dockerfile didn't publish it with any affiliation with Microsoft.

Reply on news.ycombinator.com