2018-02-20 00:24:10 +02:00
|
|
|
// Copyright 2018 Drone.IO Inc.
|
2018-03-21 15:02:17 +02:00
|
|
|
//
|
2018-02-20 00:24:10 +02:00
|
|
|
// Licensed under the Apache License, Version 2.0 (the "License");
|
|
|
|
// you may not use this file except in compliance with the License.
|
|
|
|
// You may obtain a copy of the License at
|
2018-03-21 15:02:17 +02:00
|
|
|
//
|
2018-02-20 00:24:10 +02:00
|
|
|
// http://www.apache.org/licenses/LICENSE-2.0
|
2018-03-21 15:02:17 +02:00
|
|
|
//
|
2018-02-20 00:24:10 +02:00
|
|
|
// Unless required by applicable law or agreed to in writing, software
|
|
|
|
// distributed under the License is distributed on an "AS IS" BASIS,
|
|
|
|
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
|
|
// See the License for the specific language governing permissions and
|
|
|
|
// limitations under the License.
|
|
|
|
|
2017-09-12 20:25:55 +02:00
|
|
|
package main
|
|
|
|
|
|
|
|
import (
|
|
|
|
"encoding/json"
|
|
|
|
"fmt"
|
|
|
|
"net/http"
|
Change healtcheck port into address format, redo #1197 (#1423)
As discussed in the comments in PR #1197. Also add documenation
accordingly.
One thing I'm not sure about is the simple check in health.go if the
address is usable in the GET request or not. From reading
https://pkg.go.dev/net#Dial it seems that the only non-standard address
format that would work in the `net` package but not in a GET url would
likely only be `:port`, as the others listed here are actually also
valid urls:
`For TCP, UDP and IP networks, if the host is empty or a literal
unspecified IP address, as in ":80", "0.0.0.0:80" or "[::]:80" for TCP
and UDP, "", "0.0.0.0" or "::" for IP, the local system is assumed.`
One additional thing I noticed is that while `WOODPECKER_SERVER_ADDR`
and `WOODPECKER_SERVER_ADDR` use the default value format of `:PORT`,
`WOODPECKER_SERVER` actually uses `localhost:9000`. I guess it makes a
bit of sense, considering the server might not be local to the agent,
but it looks a bit inconsistent this way. I don't think it would hurt to
make the `WOODPECKER_HEALTHCHECK_ADDR` in this format too, but then it's
different from the server flags again... :-)
2022-11-19 13:06:51 +02:00
|
|
|
"strings"
|
2017-09-12 20:25:55 +02:00
|
|
|
|
2021-11-23 16:36:52 +02:00
|
|
|
"github.com/rs/zerolog/log"
|
2021-10-27 21:03:14 +02:00
|
|
|
"github.com/urfave/cli/v2"
|
2021-10-12 09:25:13 +02:00
|
|
|
|
2021-09-23 16:58:12 +02:00
|
|
|
"github.com/woodpecker-ci/woodpecker/agent"
|
2021-06-22 12:34:35 +02:00
|
|
|
"github.com/woodpecker-ci/woodpecker/version"
|
2017-09-12 20:25:55 +02:00
|
|
|
)
|
|
|
|
|
|
|
|
// the file implements some basic healthcheck logic based on the
|
|
|
|
// following specification:
|
|
|
|
// https://github.com/mozilla-services/Dockerflow
|
|
|
|
|
|
|
|
func init() {
|
2017-09-12 22:40:24 +02:00
|
|
|
http.HandleFunc("/varz", handleStats)
|
|
|
|
http.HandleFunc("/healthz", handleHeartbeat)
|
|
|
|
http.HandleFunc("/version", handleVersion)
|
2017-09-12 20:25:55 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
func handleHeartbeat(w http.ResponseWriter, r *http.Request) {
|
2017-09-12 22:40:24 +02:00
|
|
|
if counter.Healthy() {
|
|
|
|
w.WriteHeader(200)
|
|
|
|
} else {
|
|
|
|
w.WriteHeader(500)
|
|
|
|
}
|
2017-09-12 20:25:55 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
func handleVersion(w http.ResponseWriter, r *http.Request) {
|
|
|
|
w.WriteHeader(200)
|
|
|
|
w.Header().Add("Content-Type", "text/json")
|
2021-11-23 16:36:52 +02:00
|
|
|
_ = json.NewEncoder(w).Encode(versionResp{
|
2021-05-25 14:08:27 +02:00
|
|
|
Source: "https://github.com/woodpecker-ci/woodpecker",
|
2019-11-12 22:49:38 +02:00
|
|
|
Version: version.String(),
|
2017-09-12 20:25:55 +02:00
|
|
|
})
|
|
|
|
}
|
|
|
|
|
2017-09-12 22:40:24 +02:00
|
|
|
func handleStats(w http.ResponseWriter, r *http.Request) {
|
|
|
|
if counter.Healthy() {
|
|
|
|
w.WriteHeader(200)
|
|
|
|
} else {
|
|
|
|
w.WriteHeader(500)
|
|
|
|
}
|
|
|
|
w.Header().Add("Content-Type", "text/json")
|
2021-11-23 16:36:52 +02:00
|
|
|
if _, err := counter.WriteTo(w); err != nil {
|
|
|
|
log.Error().Err(err).Msg("handleStats")
|
|
|
|
}
|
2017-09-12 22:40:24 +02:00
|
|
|
}
|
|
|
|
|
2017-09-12 20:25:55 +02:00
|
|
|
type versionResp struct {
|
|
|
|
Version string `json:"version"`
|
|
|
|
Source string `json:"source"`
|
|
|
|
}
|
|
|
|
|
2017-09-12 22:40:24 +02:00
|
|
|
// default statistics counter
|
2021-09-23 16:58:12 +02:00
|
|
|
var counter = &agent.State{
|
|
|
|
Metadata: map[string]agent.Info{},
|
2017-09-12 22:40:24 +02:00
|
|
|
}
|
|
|
|
|
2017-09-12 20:25:55 +02:00
|
|
|
// handles pinging the endpoint and returns an error if the
|
|
|
|
// agent is in an unhealthy state.
|
|
|
|
func pinger(c *cli.Context) error {
|
Change healtcheck port into address format, redo #1197 (#1423)
As discussed in the comments in PR #1197. Also add documenation
accordingly.
One thing I'm not sure about is the simple check in health.go if the
address is usable in the GET request or not. From reading
https://pkg.go.dev/net#Dial it seems that the only non-standard address
format that would work in the `net` package but not in a GET url would
likely only be `:port`, as the others listed here are actually also
valid urls:
`For TCP, UDP and IP networks, if the host is empty or a literal
unspecified IP address, as in ":80", "0.0.0.0:80" or "[::]:80" for TCP
and UDP, "", "0.0.0.0" or "::" for IP, the local system is assumed.`
One additional thing I noticed is that while `WOODPECKER_SERVER_ADDR`
and `WOODPECKER_SERVER_ADDR` use the default value format of `:PORT`,
`WOODPECKER_SERVER` actually uses `localhost:9000`. I guess it makes a
bit of sense, considering the server might not be local to the agent,
but it looks a bit inconsistent this way. I don't think it would hurt to
make the `WOODPECKER_HEALTHCHECK_ADDR` in this format too, but then it's
different from the server flags again... :-)
2022-11-19 13:06:51 +02:00
|
|
|
healthcheckAddress := c.String("healthcheck-addr")
|
|
|
|
if strings.HasPrefix(healthcheckAddress, ":") {
|
|
|
|
// this seems sufficient according to https://pkg.go.dev/net#Dial
|
|
|
|
healthcheckAddress = "localhost" + healthcheckAddress
|
|
|
|
}
|
|
|
|
resp, err := http.Get("http://" + healthcheckAddress + "/healthz")
|
2017-09-12 20:25:55 +02:00
|
|
|
if err != nil {
|
|
|
|
return err
|
|
|
|
}
|
|
|
|
defer resp.Body.Close()
|
|
|
|
if resp.StatusCode != 200 {
|
|
|
|
return fmt.Errorf("agent returned non-200 status code")
|
|
|
|
}
|
|
|
|
return nil
|
|
|
|
}
|