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

Support QLEVER_IS_RUNNING_IN_CONTAINER #84

Merged
merged 2 commits into from
Nov 10, 2024

Conversation

hannahbast
Copy link
Member

@hannahbast hannahbast commented Nov 6, 2024

With ad-freiburg/qlever#1439, the recommended way to run QLever inside a container is to use the https://github.com/ad-freiburg/qlever-control script. Inside of the container, the environment variable QLEVER_IS_RUNNING_IN_CONTAINER is set, with the following two effects:

  1. After qlever setup-config <config name>, the Qleverfile will contain the setting SYSTEM = native and not SYSTEM = docker and a warning message will be displayed that that is the case
  2. When running qlever ui, an error message will be displayed that this command is disabled (reason: qlever ui runs another container, and we don't want containers inside of containers).

These two environment variables are useful when running the qlever CLI
within a Docker container, introduced by ad-freiburg/qlever#1439

The effect of QLEVER_OVERRIDE_SYSTEM_NATIVE is that after `qlever
setup-config <config name>`, the Qleverfile will contain the setting
`SYSTEM = native` and not `SYSTEM = docker`.

The effect of QLEVER_OVERRIDE_DISABLE_UI is that `qlever ui` is
disabled, which makes sense when running inside a container.

In both cases, meaningful log message are provided for the user.
@hannahbast hannahbast changed the title Add QLEVER_OVERRIDE_SYSTEM_NATIVE and QLEVER_OVERRIDE_DISABLE_UI Add QLEVER_OVERRIDE_SYSTEM_NATIVE and QLEVER_OVERRIDE_DISABLE_UI Nov 6, 2024
@hannahbast hannahbast changed the title Add QLEVER_OVERRIDE_SYSTEM_NATIVE and QLEVER_OVERRIDE_DISABLE_UI Support QLEVER_IS_RUNNING_IN_CONTAINER Nov 10, 2024
@hannahbast hannahbast merged commit 6b16696 into main Nov 10, 2024
4 checks passed
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.

1 participant