nginx icon indicating copy to clipboard operation
nginx copied to clipboard

Add support for the tiny stack

Open voor opened this issue 4 years ago • 3 comments

What happened?

  • What were you attempting to do?

I want to use the tiny stack with the nginx buildpack, because I want to have a very minimalist image.

  • What did you expect to happen?

nginx would happily run on tiny.

  • What was the actual behavior? Please provide log output, if possible.

'validating buildpack paketo-buildpacks/[email protected]: stack io.paketo.stacks.tiny is not supported'

Build Configuration

apiVersion: kpack.io/v1alpha1
kind: ClusterBuilder
metadata:
  name: node-script-builder
spec:
  order:
  - group:
    - id: paketo-buildpacks/nodejs
    - id: paketo-buildpacks/nginx
    - id: source-removal
  serviceAccountRef:
    name: default
    namespace: default
  stack:
    kind: ClusterStack
    name: tiny #! Make sure tiny is properly loaded from the dependency syncer.
  store:
    kind: ClusterStore
    name: node-cluster-store
  tag: #@ "{}/node-script-builder:0.0.1".format(data.values.docker_repository)
  • What platform (pack, kpack, tekton buildpacks plugin, etc.) are you using? Please include a version.

kpack released with Tanzu Build Service 1.2.2

  • What buildpacks are you using? Please include versions.

gcr.io/paketo-buildpacks/nginx:0.3.1

  • What builder are you using? If custom, can you provide the output from pack inspect-builder <builder>?

  • Can you provide a sample app or relevant configuration (buildpack.yml, nginx.conf, etc.)?

Checklist

  • [x] I have included log output.
  • [x] The log output includes an error message.
  • [x] I have included steps for reproduction.

voor avatar Aug 20 '21 19:08 voor

Hey @paketo-buildpacks/web-servers-maintainers do we have a sense of whether the Tiny build and run images contain the necessary packages for NGINX to run? If not, which are missing?

fg-j avatar May 23 '22 14:05 fg-j

@fg-j It seems the only missing package is libpcre3. It is the only one listed here that we don't currently include in tiny and I was able to build and run an nginx app after adding it to the list of packages.

thitch97 avatar May 24 '22 15:05 thitch97

I'm willing to submit a PR to enable this support, but with the perceived low priority of this work, the stacks work already in flight and this ongoing investigation into the nginx dependency itself, it might make sense to hold off until things are more stable.

thitch97 avatar May 26 '22 19:05 thitch97