Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dockerize #97

Merged
merged 1 commit into from
Aug 13, 2023
Merged

Dockerize #97

merged 1 commit into from
Aug 13, 2023

Conversation

Goorzhel
Copy link
Contributor

I'm leaving it in draft mode because I'd like to fix the logger (it should output to stdout and only stdout), but I don't yet have the Go skill for doing that.

Closes #53.

@Goorzhel
Copy link
Contributor Author

Goorzhel commented Aug 6, 2023

Well, in the interest of shipping more quickly, one could mount a volume at the container's /logs.

@Goorzhel Goorzhel marked this pull request as ready for review August 6, 2023 01:58
yunginnanet added a commit that referenced this pull request Aug 13, 2023
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Went ahead and set this up for you by adding a configuration directive to [logger] called docker_logging

I also implemented something else you might want to use for this before we merge. All configuration directives should now abide/be overridden by matching environment variables, example:

# [logger]
# docker_logging = true
HELLPOT_LOGGER_DOCKER_LOGGING=1 # "true" is also acceptable

# [http]
# bind_addr = "0.0.0.0"
HELLPOT_HTTP_BIND_ADDR="0.0.0.0"

yunginnanet
yunginnanet previously approved these changes Aug 13, 2023
@yunginnanet
Copy link
Owner

you'll need to sign your commit for sure either way

thanks for the pr

@Goorzhel
Copy link
Contributor Author

Works for me! I'll wipe the TODO and push a signed commit.

@yunginnanet yunginnanet merged commit 6ea9d47 into yunginnanet:main Aug 13, 2023
@yunginnanet
Copy link
Owner

appreciate you, cuzzin

@yunginnanet
Copy link
Owner

/release tag=v0.4.0

yunginnanet added a commit that referenced this pull request Jun 26, 2024
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.

[Featurerequest] - Docker
2 participants