search

OpenLiberty / ci.docker

Eclipse Public License 1.0
42 stars 59 forks source link

Revert ubuntu builds back to single stage #429

Closed mbroz2 closed 11 months ago

mbroz2 commented 11 months ago

Revert ubuntu builds back to single stage as requested by docker team for the official builds

mbroz2 commented 11 months ago

@leochr I think technically we don't need to include openssl since it's already included in the base image, but we should certainly be consistent and since listing it will cause it to update the version if it's not already on the latest one, we might as well have it so I'll get it added.

Regarding && rm -rf /var/lib/apt/lists/* \ I don't see it missing anywhere; it's included in all the RUN apt-get update sections after the apt-get remove commands. Can you point it out in the review exactly where it's missing?

leochr commented 11 months ago

@mbroz2 Thanks for the update. I was looking for && rm -rf /var/lib/apt/lists/* \ n a different line and missed it. Sorry. Looks good. I'll go ahead and merge.