Skip to content

Commit

Permalink
[hotfix] Fixed container build
Browse files Browse the repository at this point in the history
Removed unnecessary certs;
Removed unnecessary fake AWS Creds.
  • Loading branch information
massenz committed Feb 2, 2023
1 parent 98e7776 commit 1f90522
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 12 deletions.
8 changes: 1 addition & 7 deletions .github/workflows/release.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -20,14 +20,8 @@ jobs:
steps:
- uses: actions/checkout@v3

# TODO: run this in a build container instead
- uses: arduino/setup-protoc@v1
with:
version: '3.19'
- name: Install protoc-gen & build
- name: build
run: |
go install google.golang.org/protobuf/cmd/[email protected]
go install google.golang.org/grpc/cmd/[email protected]
make build
- name: Create release tag
Expand Down
5 changes: 0 additions & 5 deletions docker/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -9,9 +9,6 @@ FROM ubuntu:22.04
RUN apt-get update && apt-get install ca-certificates -y
RUN groupadd -r sm-bot && useradd -r -g sm-bot sm-bot

# Fake AWS configuration to connect to LocalStack docker.
ENV AWS_REGION=us-west-2 AWS_PROFILE=sm-bot

# Sensible defaults for the server, for reference
# we list all the environment variables used by the
# entrypoint script.
Expand All @@ -25,8 +22,6 @@ WORKDIR /app
RUN chown sm-bot:sm-bot /app

USER sm-bot
ADD docker/aws-credentials /home/sm-bot/.aws/credentials
ADD certs/* /etc/statemachine/certs/
ADD build/bin/sm-server docker/entrypoint.sh ./

EXPOSE ${SERVER_PORT}
Expand Down

0 comments on commit 1f90522

Please sign in to comment.