Skip to content

Commit

Permalink
feat(build): support env var of skipDockerTests
Browse files Browse the repository at this point in the history
  • Loading branch information
featherchen committed Jul 22, 2024
1 parent 2a2a190 commit 4290ee6
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 4 deletions.
8 changes: 6 additions & 2 deletions build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -690,8 +690,12 @@ fun printDockerCheckInfo() {
val dockerRunning = project.extra["dockerRunning"] as? Boolean ?: false
val macDockerConnector = project.extra["macDockerConnector"] as? Boolean ?: false
val isOrbStack = project.extra["isOrbStack"] as? Boolean ?: false

if (extra["skipDockerTests"].toString().toBoolean()) {
val skipDockerTest = if (extra["skipDockerTests"].toString().toBoolean()) {
System.getenv("SKIP_DOCKER_TESTS")?.toBoolean() ?: true
} else {
false
}
if (skipDockerTest) {
project.extra["dockerTest"] = false
} else if (OperatingSystem.current().isMacOsX() &&
dockerRunning &&
Expand Down
4 changes: 2 additions & 2 deletions docs/how-to-build.md
Original file line number Diff line number Diff line change
Expand Up @@ -26,8 +26,8 @@ license: "This software is licensed under the Apache License version 2."
+ Gravitino uses the Gradle Java Toolchain to detect and manage JDK versions, it checks the
installed JDK by running the `./gradlew javaToolchains` command. See [Gradle Java Toolchain](https://docs.gradle.org/current/userguide/toolchains.html#sec:java_toolchain).
+ Gravitino excludes all Docker-related tests by default. To run Docker related tests, make sure you have installed
Docker in your environment and set `skipDockerTests=false` in the `gradle.properties` file (or
use `-PskipDockerTests=false` in the command). Otherwise, all Docker required tests will skip.
Docker in your environment and either (1) set `skipDockerTests=false` in the `gradle.properties` file (or
use `-PskipDockerTests=false` in the command) or (2) `export SKIP_DOCKER_TESTS=false` in shell. Otherwise, all Docker required tests will skip.
+ macOS uses `docker-connector` to make the Gravitino Trino connector work with Docker
for macOS. See [docker-connector](https://github.com/wenjunxiao/mac-docker-connector), `$GRAVITINO_HOME/dev/docker/tools/mac-docker-connector.sh`, and
`$GRAVITINO_HOME/dev/docker/tools/README.md` for more details.
Expand Down

0 comments on commit 4290ee6

Please sign in to comment.