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

KROKI_LISTEN option to start listening on an address #595

Merged
merged 5 commits into from
Feb 14, 2021

Conversation

ggrossetie
Copy link
Member

  • KROKI_LISTEN has higher precedence than KROKI_PORT
  • Support tcp://ip:port value on KROKI_PORT (this value can be automatically set by the container runtime: k8s, Docker...)
  • Set KROKI_CONTAINER_SUPPORT environment variable on the Docker image to indicate that we are running in a container

resolves #576

- KROKI_LISTEN has higher precedence than KROKI_PORT
- Support `tcp://ip:port` value on KROKI_PORT (this value can be automatically set by the container runtime: k8s, Docker...)
- Set KROKI_CONTAINER_SUPPORT environment variable on the Docker image to indicate that we are running in a container
Instead of relying on the default platform encoding (was UTF-8 on Linux).
@ggrossetie ggrossetie force-pushed the issue-576-container-support branch from 3ef5892 to 889f230 Compare February 6, 2021 23:12
@ggrossetie
Copy link
Member Author

@jkroepke I would appreciate very much if you could review this pull request. If you are not familiar with Java, you can review the documentation. I want to make sure that the documentation is comprehensible! Thanks 😉

Copy link
Contributor

@jkroepke jkroepke left a comment

Choose a reason for hiding this comment

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

Just found a typo, lgtm.

could not review the java lines. I just take care about the docs.

docs/modules/setup/pages/configuration.adoc Outdated Show resolved Hide resolved
@ggrossetie ggrossetie merged commit 31b5c02 into yuzutech:master Feb 14, 2021
@ggrossetie ggrossetie deleted the issue-576-container-support branch February 14, 2021 17:52
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.

Kubernetes automatically sets KROKI_PORT to tcp://ip:port causing a ClassCastException
2 participants