Skip to content

Ensure that Dockerfile_multistage builds and is runnable#38

Merged
d-Rickyy-b merged 1 commit intod-Rickyy-b:masterfrom
alberts-s:fix-docker
Jun 16, 2024
Merged

Ensure that Dockerfile_multistage builds and is runnable#38
d-Rickyy-b merged 1 commit intod-Rickyy-b:masterfrom
alberts-s:fix-docker

Conversation

@alberts-s
Copy link
Copy Markdown
Contributor

It appears that the go build command was pointing to the wrong directory - the build was failing as follows:

...
 => ERROR [builder 7/8] RUN go build -ldflags="-w -s" -o /go/bin/certstream-server-go /go/src/certstream-server-go/cmd
------
 > [builder 7/8] RUN go build -ldflags="-w -s" -o /go/bin/certstream-server-go /go/src/certstream-server-go/cmd:
no Go files in /go/src/certstream-server-go/cmd

After it was fixed, the configfile had 0750 permissions - it was owned by root:root, thus certstream-server-go was failing to start.

config.go:49: Error while parsing yaml file: open /app/config.yaml: permission denied

This commit resolves this issue too.

It appears that the `go build` command was pointing to the wrong directory - the build was failing as follows:
```
...
 => ERROR [builder 7/8] RUN go build -ldflags="-w -s" -o /go/bin/certstream-server-go /go/src/certstream-server-go/cmd
------
 > [builder 7/8] RUN go build -ldflags="-w -s" -o /go/bin/certstream-server-go /go/src/certstream-server-go/cmd:
no Go files in /go/src/certstream-server-go/cmd
```

After it was fixed, the configfile had 0750 permissions - it was owned by root:root, thus certstream-server-go was failing to start.
```
config.go:49: Error while parsing yaml file: open /app/config.yaml: permission denied
```
This commit resolves this issue too.
@d-Rickyy-b d-Rickyy-b merged commit 10238cd into d-Rickyy-b:master Jun 16, 2024
@d-Rickyy-b
Copy link
Copy Markdown
Owner

Thank you very much for spotting and fixing the issue :)

messede-degod pushed a commit to messede-degod/certstream-server-go that referenced this pull request Mar 30, 2025
Ensure that Dockerfile_multistage builds and is runnable
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.

2 participants