Merge pull request #7295 from dokku/fix-proxy-logs-again

Respect the exit code and pass flags correctly when tailing nginx logs on k3s
This commit is contained in:
Jose Diaz-Gonzalez
2024-10-21 21:51:58 -04:00
committed by GitHub
2 changed files with 3 additions and 1 deletions

View File

@@ -85,6 +85,7 @@ nginx_logs() {
local DOKKU_SCHEDULER=$(get_app_scheduler "$APP")
if [[ "$DOKKU_SCHEDULER" != "docker-local" ]]; then
plugn trigger scheduler-proxy-logs "$DOKKU_SCHEDULER" "$APP" "nginx" "$NGINX_LOGS_TYPE" "$tail" "$num"
return $?
fi
if [[ "$tail" == "true" ]]; then

View File

@@ -866,7 +866,8 @@ func TriggerSchedulerProxyLogs(scheduler string, appName string, proxyType strin
command = append(command, "-F")
}
if numLines > 0 {
command = append(command, fmt.Sprintf("-n %d", numLines))
command = append(command, "-n")
command = append(command, strconv.FormatInt(numLines, 10))
}
command = append(command, filename)