Skip to content

Commit

Permalink
Fix Docker Build
Browse files Browse the repository at this point in the history
* Replace docker file with template from https://github.com/vercel/next.js/blob/canary/examples/with-docker/Dockerfile
* Uses node 18 instead of Node 10 which is no longer compatible
* Add the option via ENV to build to config standalone and use this for the docker image
  • Loading branch information
amenk committed Nov 18, 2023
1 parent ab1969c commit c264442
Show file tree
Hide file tree
Showing 2 changed files with 59 additions and 8 deletions.
66 changes: 58 additions & 8 deletions Dockerfile
Original file line number Diff line number Diff line change
@@ -1,14 +1,64 @@
FROM mhart/alpine-node:10 as build-stage
FROM node:18-alpine AS base

# Install dependencies only when needed
FROM base AS deps
# Check https://github.com/nodejs/docker-node/tree/b4117f9333da4138b03a546ec926ef50a31506c3#nodealpine to understand why libc6-compat might be needed.
RUN apk add --no-cache libc6-compat
WORKDIR /app

# Install dependencies based on the preferred package manager
COPY package.json yarn.lock* package-lock.json* pnpm-lock.yaml* ./
RUN \
if [ -f yarn.lock ]; then yarn --frozen-lockfile; \
elif [ -f package-lock.json ]; then npm ci; \
elif [ -f pnpm-lock.yaml ]; then yarn global add pnpm && pnpm i --frozen-lockfile; \
else echo "Lockfile not found." && exit 1; \
fi


# Rebuild the source code only when needed
FROM base AS builder
WORKDIR /app
COPY --from=deps /app/node_modules ./node_modules
COPY . .
RUN yarn install --pure-lockfile
RUN yarn build
RUN yarn --production

FROM mhart/alpine-node:base-10
# Next.js collects completely anonymous telemetry data about general usage.
# Learn more here: https://nextjs.org/telemetry
# Uncomment the following line in case you want to disable telemetry during the build.
# ENV NEXT_TELEMETRY_DISABLED 1

RUN NEXTJS_OUTPUT=standalone yarn build

# Production image, copy all the files and run next
FROM base AS runner
WORKDIR /app
ENV NODE_ENV="production"
COPY --from=build-stage /app .

ENV NODE_ENV production
# Uncomment the following line in case you want to disable telemetry during runtime.
# ENV NEXT_TELEMETRY_DISABLED 1

RUN addgroup --system --gid 1001 nodejs
RUN adduser --system --uid 1001 nextjs

COPY --from=builder /app/public ./public

# Set the correct permission for prerender cache
RUN mkdir .next
RUN chown nextjs:nodejs .next

# Automatically leverage output traces to reduce image size
# https://nextjs.org/docs/advanced-features/output-file-tracing
COPY --from=builder --chown=nextjs:nodejs /app/.next/standalone ./
COPY --from=builder --chown=nextjs:nodejs /app/.next/static ./.next/static

USER nextjs

EXPOSE 3000
CMD ["node", "./node_modules/.bin/next", "start"]

ENV PORT 3000
# set hostname to localhost
ENV HOSTNAME "0.0.0.0"

# server.js is created by next build from the standalone output
# https://nextjs.org/docs/pages/api-reference/next-config-js/output
CMD ["node", "server.js"]
1 change: 1 addition & 0 deletions next.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@ const languages = {
};

module.exports = withPWA({
output: process.env.NEXTJS_OUTPUT || undefined,
//TODO fails with current webpack config. Probably needs to get rid of sentry? (@sentry/nextjs was not cool)
// future: {
// webpack5: true,
Expand Down

0 comments on commit c264442

Please sign in to comment.