We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
hi there I found a bug in the arangodb starter project in the runner_docker.go file
func (r *dockerRunner) CreateStartArangodbCommand(index int, masterIP string, masterPort string) string { addr := masterIP hostPort := 4000 + (portOffsetIncrement * (index - 1)) if masterPort != "" { addr = addr + ":" + masterPort masterPortI, _ := strconv.Atoi(masterPort) hostPort = masterPortI + (portOffsetIncrement * (index - 1)) } lines := []string{ fmt.Sprintf("docker volume create arangodb%d &&", index), fmt.Sprintf("docker run -it --name=adb%d --rm -p %d:4000 -v arangodb%d:/data -v /var/run/docker.sock:/var/run/docker.sock arangodb/arangodb-starter", index, hostPort, index), fmt.Sprintf("--dockerContainer=adb%d --ownAddress=%s --join=%s", index, masterIP, addr), } return strings.Join(lines, " \\\n ") }
the line
fmt.Sprintf("docker run -it --name=adb%d --rm -p %d:4000 -v arangodb%d:/data -v /var/run/docker.sock:/var/run/docker.sock arangodb/arangodb-starter", index, hostPort, index)
should not be static for the docker Image Name arangodb/arangodb-starter
because we are using our own build with our own docker registry for arangodb-starter for arm32 and arm64
This image name should be assignable and dynamic
The text was updated successfully, but these errors were encountered:
No branches or pull requests
hi there I found a bug in the arangodb starter project
in the runner_docker.go file
the line
should not be static for the docker Image Name arangodb/arangodb-starter
because we are using our own build with our own docker registry for arangodb-starter for arm32 and arm64
This image name should be assignable and dynamic
The text was updated successfully, but these errors were encountered: