Merge pull request #448 from flanksource/improve-dockerfile

feat: remove apt install from dockerfile

.dockerignore

@@ -0,0 +1,9 @@
+Dockerfile
+.bin/
+.vscode/
+.idea/
+.env
+.DS_Store
+cover.out
+test.test
+build/

Dockerfile

@@ -6,23 +6,13 @@ COPY go.mod /app/go.mod
 COPY go.sum /app/go.sum
 RUN go mod download
 COPY ./ ./
-
-WORKDIR /app
-RUN go version
 RUN make build
 
 FROM ubuntu:jammy
 WORKDIR /app
 
-# install CA certificates
-RUN apt-get update && \
-  apt-get install -y ca-certificates && \
-  rm -Rf /var/lib/apt/lists/*  && \
-  rm -Rf /usr/share/doc && rm -Rf /usr/share/man  && \
-  apt-get clean
-
 COPY --from=builder /app/.bin/incident-commander /app
+COPY --from=builder /etc/ssl/certs/ca-certificates.crt /etc/ssl/certs/
 
 RUN /app/incident-commander go-offline
-
 ENTRYPOINT ["/app/incident-commander"]

cmd/server.go

@@ -192,7 +192,7 @@ var Serve = &cobra.Command{
 		listenAddr := fmt.Sprintf(":%d", httpPort)
 		logger.Infof("Listening on %s", listenAddr)
 		if err := e.Start(listenAddr); err != nil {
-			e.Logger.Fatal(err)
+			logger.Fatalf("Failed to start server: %v", err)
 		}
 	},
 }