Skip to content

Commit

Permalink
use healthcehck in liveness probe instead of init
Browse files Browse the repository at this point in the history
Signed-off-by: Roee Hershko <roee.hershko@riskified.com>
  • Loading branch information
roee-hersh committed Dec 22, 2024
1 parent 26b6bd7 commit 82c98a2
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions pkg/controller/vault/vault_controller.go
Original file line number Diff line number Diff line change
Expand Up @@ -22,14 +22,14 @@ import (
"crypto/x509"
"errors"
"fmt"
"github.com/blang/semver/v4"
"net/http"
"path/filepath"
"reflect"
"sort"
"strings"
"time"

"github.com/Masterminds/semver/v3"
vaultv1alpha1 "github.com/bank-vaults/vault-operator/pkg/apis/vault/v1alpha1"
bvtls "github.com/bank-vaults/vault-sdk/tls"
"github.com/bank-vaults/vault-sdk/vault"
Expand Down Expand Up @@ -1214,13 +1214,13 @@ func statefulSetForVault(v *vaultv1alpha1.Vault, externalSecretsToWatchItems []c
FailureThreshold: 18,
},
// This probe makes sure Vault is responsive in a HTTPS manner
// See: https://www.vaultproject.io/api/system/init.html
// See: https://www.vaultproject.io/api/system/health.html
LivenessProbe: &corev1.Probe{
ProbeHandler: corev1.ProbeHandler{
HTTPGet: &corev1.HTTPGetAction{
Scheme: getVaultURIScheme(v),
Port: intstr.FromString(v.Spec.GetAPIPortName()),
Path: "/v1/sys/init",
Path: "/v1/sys/health?standbyok=true",
},
},
},
Expand Down

0 comments on commit 82c98a2

Please # to comment.