Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(maint) Switch PDB container off of Alpine #3040

Merged
merged 1 commit into from
Aug 14, 2019

Conversation

Iristyle
Copy link
Contributor

@Iristyle Iristyle requested a review from a team as a code owner August 13, 2019 22:22
Copy link
Contributor

@nwolfe nwolfe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep, Alpine is falling out of favor nowadays.

 - Alpine seems to still be having issues with DNS resolutions inside
   an LCOW environment. In an effort to reduce these transient
   problems, switch the base container to a non-Alpine platform.

   A ticket has been filed with a repro for Alpine DNS issues under LCOW
   moby/libnetwork#2371

 - While this may increase the image size by about 100MB, the goal here
   is reliability and robustness

   for the builder container:
   clojure:lein-alpine was about 142MB
   clojure:openjdk-8-lein is about 507MB

	 for the target container:
   openjdk:8-jre-alpine was about 85MB
   openjdk:8-buster-slim is about 184MB

 - Ubuntu 18.04 shares a lineage with debian buster, which should
   be a well supported platform for PDB

   All OpenJDK container variants are listed at:
   https://github.com/docker-library/docs/blob/master/openjdk/README.md#supported-tags-and-respective-dockerfile-links
@Iristyle Iristyle force-pushed the change-puppetdb-base-container branch from 9c202f1 to e2e6004 Compare August 13, 2019 22:48
@Iristyle
Copy link
Contributor Author

Iristyle commented Aug 13, 2019

I'm not sure if we can get away with an alpine builder still or not? I would expect not for a C++ application unless it was explicitly cross-compiling, but I'm not sure on our Java code.

Decided it's not worth investigating that.

@Iristyle Iristyle merged commit 3c876e5 into puppetlabs:master Aug 14, 2019
@Iristyle Iristyle deleted the change-puppetdb-base-container branch August 14, 2019 00:30
@puppetcla
Copy link

CLA signed by all contributors.

Iristyle added a commit to Iristyle/puppetdb that referenced this pull request Aug 15, 2019
 - When the base container image was changed from Alpine Linux to
   Debian Slim Buster in puppetlabs#3040, the /bin/sh implementation changed.

   3c876e5 merged this code

   On Alpine, /bin/sh is the standard bare-bones cross-platform
   Bourne Shell.

   On Debian, /bin/sh is a symlink to dash.

 - Update shellscript syntax to be compatible with Dash.
Iristyle added a commit that referenced this pull request Aug 15, 2019
 - When the base container image was changed from Alpine Linux to
   Debian Slim Buster in #3040, the /bin/sh implementation changed.

   3c876e5 merged this code

   On Alpine, /bin/sh is the standard bare-bones cross-platform
   Bourne Shell.

   On Debian, /bin/sh is a symlink to dash.

 - Update shellscript syntax to be compatible with Dash.
Iristyle added a commit to Iristyle/puppetdb that referenced this pull request Oct 28, 2019
 - When the base container image was changed from Alpine Linux to
   Debian Slim Buster in puppetlabs#3040, the /bin/sh implementation changed.

   3c876e5 merged this code

   On Alpine, /bin/sh is the standard bare-bones cross-platform
   Bourne Shell.

   On Debian, /bin/sh is a symlink to dash.

 - Update shellscript syntax to be compatible with Dash.
binford2k referenced this pull request in voxpupuli/container-puppetdb Nov 1, 2022
 - When the base container image was changed from Alpine Linux to
   Debian Slim Buster in #3040, the /bin/sh implementation changed.

   1777702 merged this code

   On Alpine, /bin/sh is the standard bare-bones cross-platform
   Bourne Shell.

   On Debian, /bin/sh is a symlink to dash.

 - Update shellscript syntax to be compatible with Dash.
binford2k referenced this pull request in voxpupuli/container-puppetdb Nov 1, 2022
 - When the base container image was changed from Alpine Linux to
   Debian Slim Buster in #3040, the /bin/sh implementation changed.

   1777702 merged this code

   On Alpine, /bin/sh is the standard bare-bones cross-platform
   Bourne Shell.

   On Debian, /bin/sh is a symlink to dash.

 - Update shellscript syntax to be compatible with Dash.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants