mirror of
https://github.com/vmware-tanzu/velero.git
synced 2026-09-18 05:54:29 +00:00
Run the E2E test on kind / setup-test-matrix (push) Failing after 3s
Scorecard supply-chain security / Scorecard analysis (push) Skipped
e2e-test-kind.yaml / extract (push) Failing after 6s
Run the E2E test on kind / get-go-version (push) Failing after 7s
Run the E2E test on kind / build (push) Skipped
Run the E2E test on kind / run-e2e-test (push) Skipped
push.yml / extract (push) Failing after 6s
Main CI / get-go-version (push) Failing after 7s
Main CI / Build (push) Skipped
* Remove toleration whitelist for PodVolumeBackup and data mover pods Instead of filtering tolerations through a hardcoded allowlist (ThirdPartyTolerations), inherit all tolerations from the node-agent daemonset for PodVolumeBackup/Restore and DataUpload/Download pods, and from the Velero deployment for maintenance jobs. This enables backups and restores on nodes with custom NoExecute taints, which was previously impossible since only two specific toleration keys were whitelisted. Fixes #9476 Signed-off-by: Tiger Kaovilai <tkaovila@redhat.com> Generated with [Claude Code](https://claude.ai/code) via [Happy](https://happy.engineering) Co-Authored-By: Claude <noreply@anthropic.com> Co-Authored-By: Happy <yesreply@happy.engineering> * Fix codespell: replace 'whitelist' with 'allowlist' in changelog Signed-off-by: Tiger Kaovilai <tkaovila@redhat.com> Generated with [Claude Code](https://claude.ai/code) via [Happy](https://happy.engineering) Co-Authored-By: Claude <noreply@anthropic.com> Co-Authored-By: Happy <yesreply@happy.engineering> * Implement deduplication of tolerations and add unit tests for the new function Signed-off-by: Tiger Kaovilai <tkaovila@redhat.com> * Merge node-agent-configmap tolerations with third-party allowlist Add a `tolerations` field to the node-agent-configmap so operators can declare hosting-pod tolerations explicitly, per blackpiglet's review feedback that tolerations shouldn't be read from the DaemonSet alone. These are merged with (and deduplicated against) DaemonSet tolerations matching the existing third-party allowlist (kubernetes.azure.com/scalesetpriority, CriticalAddonsOnly), restoring that allowlist per the follow-up suggestion to keep inheriting it alongside the new config option. The toleration dedup helper is moved from pkg/exposer to pkg/util/kube (exported as DeduplicateTolerations) so it can be shared with pkg/nodeagent without an import cycle. Signed-off-by: Tiger Kaovilai <tkaovila@redhat.com> * Fix testifylint finding in TestGetTolerations golangci-lint v2.12.0 (pinned in pr-linter-check.yml) flagged the shared assert.Equal after the if/else as require-error: use require for the error assertion so each branch is self-contained, matching the pattern used elsewhere in this file. Signed-off-by: Tiger Kaovilai <tkaovila@redhat.com> * Document toleration merge priority in GetTolerations Per blackpiglet's review feedback: clarify that configured tolerations take priority over allowlisted daemonset tolerations because they're appended first and DeduplicateTolerations keeps only the first occurrence of each exact (Key, Operator, Value, Effect) combination. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Signed-off-by: Tiger Kaovilai <tkaovila@redhat.com> --------- Signed-off-by: Tiger Kaovilai <tkaovila@redhat.com> Co-authored-by: Claude <noreply@anthropic.com> Co-authored-by: Happy <yesreply@happy.engineering>
332 lines
11 KiB
Go
332 lines
11 KiB
Go
/*
|
|
Copyright The Velero Contributors.
|
|
|
|
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
|
|
|
|
http://www.apache.org/licenses/LICENSE-2.0
|
|
|
|
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.
|
|
*/
|
|
|
|
package nodeagent
|
|
|
|
import (
|
|
"context"
|
|
"encoding/json"
|
|
"fmt"
|
|
|
|
"github.com/cockroachdb/errors"
|
|
appsv1api "k8s.io/api/apps/v1"
|
|
corev1api "k8s.io/api/core/v1"
|
|
apierrors "k8s.io/apimachinery/pkg/api/errors"
|
|
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
|
|
"k8s.io/apimachinery/pkg/labels"
|
|
"k8s.io/client-go/kubernetes"
|
|
ctrlclient "sigs.k8s.io/controller-runtime/pkg/client"
|
|
|
|
velerotypes "github.com/vmware-tanzu/velero/pkg/types"
|
|
"github.com/vmware-tanzu/velero/pkg/util"
|
|
"github.com/vmware-tanzu/velero/pkg/util/kube"
|
|
)
|
|
|
|
const (
|
|
// daemonSet is the name of the Velero node agent daemonset on linux nodes.
|
|
daemonSet = "node-agent"
|
|
|
|
// daemonsetWindows is the name of the Velero node agent daemonset on Windows nodes.
|
|
daemonsetWindows = "node-agent-windows"
|
|
|
|
// nodeAgentRole marks pods with node-agent role on all nodes.
|
|
nodeAgentRole = "node-agent"
|
|
|
|
// HostPodVolumeMount is the name of the volume in node-agent for host-pod mount
|
|
HostPodVolumeMount = "host-pods"
|
|
|
|
// HostPodVolumeMountPoint is the mount point of the volume in node-agent for host-pod mount
|
|
HostPodVolumeMountPoint = "host_pods"
|
|
)
|
|
|
|
var (
|
|
ErrDaemonSetNotFound = errors.New("daemonset not found")
|
|
ErrNodeAgentLabelNotFound = errors.New("node-agent label not found")
|
|
ErrNodeAgentAnnotationNotFound = errors.New("node-agent annotation not found")
|
|
)
|
|
|
|
func IsRunningOnLinux(ctx context.Context, kubeClient kubernetes.Interface, namespace string) error {
|
|
return isRunning(ctx, kubeClient, namespace, daemonSet)
|
|
}
|
|
|
|
func IsRunningOnWindows(ctx context.Context, kubeClient kubernetes.Interface, namespace string) error {
|
|
return isRunning(ctx, kubeClient, namespace, daemonsetWindows)
|
|
}
|
|
|
|
func isRunning(ctx context.Context, kubeClient kubernetes.Interface, namespace string, daemonset string) error {
|
|
if _, err := kubeClient.AppsV1().DaemonSets(namespace).Get(ctx, daemonset, metav1.GetOptions{}); apierrors.IsNotFound(err) {
|
|
return ErrDaemonSetNotFound
|
|
} else if err != nil {
|
|
return err
|
|
} else {
|
|
return nil
|
|
}
|
|
}
|
|
|
|
// KbClientIsRunningInNode checks if the node agent pod is running properly in a specified node through kube client. If not, return the error found
|
|
func KbClientIsRunningInNode(ctx context.Context, namespace string, nodeName string, kubeClient kubernetes.Interface) error {
|
|
return isRunningInNode(ctx, namespace, nodeName, nil, kubeClient)
|
|
}
|
|
|
|
// IsReady checks whether the node-agent daemonset has at least one ready pod
|
|
// by inspecting the DaemonSet status. Both the linux and windows daemonsets
|
|
// are checked before returning any non-NotFound lookup error, so that a
|
|
// transient error fetching one daemonset does not mask the other daemonset
|
|
// being ready.
|
|
func IsReady(ctx context.Context, namespace string, crClient ctrlclient.Client) error {
|
|
dsLinux := new(appsv1api.DaemonSet)
|
|
var lookupErr error
|
|
if err := crClient.Get(ctx, ctrlclient.ObjectKey{Namespace: namespace, Name: daemonSet}, dsLinux); err != nil {
|
|
dsLinux = nil
|
|
if !apierrors.IsNotFound(err) {
|
|
lookupErr = errors.Wrap(err, "failed to get linux node-agent daemonset")
|
|
}
|
|
}
|
|
|
|
dsWindows := new(appsv1api.DaemonSet)
|
|
if err := crClient.Get(ctx, ctrlclient.ObjectKey{Namespace: namespace, Name: daemonsetWindows}, dsWindows); err != nil {
|
|
dsWindows = nil
|
|
if !apierrors.IsNotFound(err) {
|
|
lookupErr = errors.CombineErrors(lookupErr, errors.Wrap(err, "failed to get windows node-agent daemonset"))
|
|
}
|
|
}
|
|
|
|
if dsLinux != nil && dsLinux.Status.NumberReady > 0 {
|
|
return nil
|
|
}
|
|
|
|
if dsWindows != nil && dsWindows.Status.NumberReady > 0 {
|
|
return nil
|
|
}
|
|
|
|
if lookupErr != nil {
|
|
return lookupErr
|
|
}
|
|
|
|
return errors.New("node-agent is not ready: no ready pods found")
|
|
}
|
|
|
|
// IsRunningInNode checks if the node agent pod is running properly in a specified node through controller client. If not, return the error found
|
|
func IsRunningInNode(ctx context.Context, namespace string, nodeName string, crClient ctrlclient.Client) error {
|
|
return isRunningInNode(ctx, namespace, nodeName, crClient, nil)
|
|
}
|
|
|
|
func isRunningInNode(ctx context.Context, namespace string, nodeName string, crClient ctrlclient.Client, kubeClient kubernetes.Interface) error {
|
|
if nodeName == "" {
|
|
return errors.New("node name is empty")
|
|
}
|
|
|
|
pods := new(corev1api.PodList)
|
|
parsedSelector, err := labels.Parse(fmt.Sprintf("role=%s", nodeAgentRole))
|
|
if err != nil {
|
|
return errors.Wrap(err, "fail to parse selector")
|
|
}
|
|
|
|
if crClient != nil {
|
|
err = crClient.List(ctx, pods, &ctrlclient.ListOptions{
|
|
LabelSelector: parsedSelector,
|
|
Namespace: namespace,
|
|
})
|
|
} else {
|
|
pods, err = kubeClient.CoreV1().Pods(namespace).List(ctx, metav1.ListOptions{LabelSelector: parsedSelector.String()})
|
|
}
|
|
|
|
if err != nil {
|
|
return errors.Wrap(err, "failed to list node-agent pods")
|
|
}
|
|
|
|
for i := range pods.Items {
|
|
if kube.IsPodRunning(&pods.Items[i]) != nil {
|
|
continue
|
|
}
|
|
|
|
if pods.Items[i].Spec.NodeName == nodeName {
|
|
return nil
|
|
}
|
|
}
|
|
|
|
return errors.Errorf("daemonset pod not found in running state in node %s", nodeName)
|
|
}
|
|
|
|
func GetPodSpec(ctx context.Context, kubeClient kubernetes.Interface, namespace string, osType string) (*corev1api.PodSpec, error) {
|
|
dsName := daemonSet
|
|
if osType == kube.NodeOSWindows {
|
|
dsName = daemonsetWindows
|
|
}
|
|
|
|
ds, err := kubeClient.AppsV1().DaemonSets(namespace).Get(ctx, dsName, metav1.GetOptions{})
|
|
if err != nil {
|
|
return nil, errors.Wrapf(err, "error to get %s daemonset", dsName)
|
|
}
|
|
|
|
return &ds.Spec.Template.Spec, nil
|
|
}
|
|
|
|
func GetConfigs(ctx context.Context, namespace string, kubeClient kubernetes.Interface, configName string) (*velerotypes.NodeAgentConfigs, error) {
|
|
cm, err := kubeClient.CoreV1().ConfigMaps(namespace).Get(ctx, configName, metav1.GetOptions{})
|
|
if err != nil {
|
|
return nil, errors.Wrapf(err, "error to get node agent configs %s", configName)
|
|
}
|
|
|
|
if cm.Data == nil {
|
|
return nil, errors.Errorf("data is not available in config map %s", configName)
|
|
}
|
|
|
|
if len(cm.Data) > 1 {
|
|
return nil, errors.Errorf("more than one keys are found in ConfigMap %s's data. only expect one", configName)
|
|
}
|
|
|
|
jsonString := ""
|
|
for _, v := range cm.Data {
|
|
jsonString = v
|
|
}
|
|
|
|
configs := &velerotypes.NodeAgentConfigs{}
|
|
err = json.Unmarshal([]byte(jsonString), configs)
|
|
if err != nil {
|
|
return nil, errors.Wrapf(err, "error to unmarshall configs from %s", configName)
|
|
}
|
|
|
|
return configs, nil
|
|
}
|
|
|
|
func GetLabelValue(ctx context.Context, kubeClient kubernetes.Interface, namespace string, key string, osType string) (string, error) {
|
|
dsName := daemonSet
|
|
if osType == kube.NodeOSWindows {
|
|
dsName = daemonsetWindows
|
|
}
|
|
|
|
ds, err := kubeClient.AppsV1().DaemonSets(namespace).Get(ctx, dsName, metav1.GetOptions{})
|
|
if err != nil {
|
|
return "", errors.Wrapf(err, "error getting %s daemonset", dsName)
|
|
}
|
|
|
|
if ds.Spec.Template.Labels == nil {
|
|
return "", ErrNodeAgentLabelNotFound
|
|
}
|
|
|
|
val, found := ds.Spec.Template.Labels[key]
|
|
if !found {
|
|
return "", ErrNodeAgentLabelNotFound
|
|
}
|
|
|
|
return val, nil
|
|
}
|
|
|
|
func GetAnnotationValue(ctx context.Context, kubeClient kubernetes.Interface, namespace string, key string, osType string) (string, error) {
|
|
dsName := daemonSet
|
|
if osType == kube.NodeOSWindows {
|
|
dsName = daemonsetWindows
|
|
}
|
|
|
|
ds, err := kubeClient.AppsV1().DaemonSets(namespace).Get(ctx, dsName, metav1.GetOptions{})
|
|
if err != nil {
|
|
return "", errors.Wrapf(err, "error getting %s daemonset", dsName)
|
|
}
|
|
|
|
if ds.Spec.Template.Annotations == nil {
|
|
return "", ErrNodeAgentAnnotationNotFound
|
|
}
|
|
|
|
val, found := ds.Spec.Template.Annotations[key]
|
|
if !found {
|
|
return "", ErrNodeAgentAnnotationNotFound
|
|
}
|
|
|
|
return val, nil
|
|
}
|
|
|
|
// GetTolerations returns the tolerations that should be applied to a node-agent-driven
|
|
// hosting pod: the explicitly configured tolerations (typically sourced from the
|
|
// node-agent-configmap), plus any toleration on the node-agent daemonset (linux or
|
|
// windows, based on osType) whose key is in util.ThirdPartyTolerations. The combined
|
|
// list is deduplicated by kube.DeduplicateTolerations. On a daemonset lookup error,
|
|
// configuredTolerations is still returned alongside the error so callers don't lose
|
|
// explicitly configured tolerations to a transient lookup failure.
|
|
func GetTolerations(ctx context.Context, kubeClient kubernetes.Interface, namespace string, osType string, configuredTolerations []corev1api.Toleration) ([]corev1api.Toleration, error) {
|
|
dsName := daemonSet
|
|
if osType == kube.NodeOSWindows {
|
|
dsName = daemonsetWindows
|
|
}
|
|
|
|
ds, err := kubeClient.AppsV1().DaemonSets(namespace).Get(ctx, dsName, metav1.GetOptions{})
|
|
if err != nil {
|
|
return configuredTolerations, errors.Wrapf(err, "error getting %s daemonset", dsName)
|
|
}
|
|
|
|
// configuredTolerations is appended first so it wins: DeduplicateTolerations
|
|
// keeps only the first occurrence of each exact (Key, Operator, Value,
|
|
// Effect) combination, so an allowlisted daemonset toleration identical to
|
|
// one already set in the configmap is dropped as a duplicate rather than
|
|
// overriding it. A daemonset toleration that only shares a Key (but
|
|
// differs in Operator/Value/Effect) isn't a duplicate and is kept
|
|
// alongside the configured one, not replaced by it.
|
|
merged := make([]corev1api.Toleration, 0, len(configuredTolerations)+len(ds.Spec.Template.Spec.Tolerations))
|
|
merged = append(merged, configuredTolerations...)
|
|
for _, t := range ds.Spec.Template.Spec.Tolerations {
|
|
for _, allowed := range util.ThirdPartyTolerations {
|
|
if t.Key == allowed {
|
|
merged = append(merged, t)
|
|
break
|
|
}
|
|
}
|
|
}
|
|
|
|
return kube.DeduplicateTolerations(merged), nil
|
|
}
|
|
|
|
func GetHostPodPath(ctx context.Context, kubeClient kubernetes.Interface, namespace string, osType string) (string, error) {
|
|
dsName := daemonSet
|
|
if osType == kube.NodeOSWindows {
|
|
dsName = daemonsetWindows
|
|
}
|
|
|
|
ds, err := kubeClient.AppsV1().DaemonSets(namespace).Get(ctx, dsName, metav1.GetOptions{})
|
|
if err != nil {
|
|
return "", errors.Wrapf(err, "error getting daemonset %s", dsName)
|
|
}
|
|
|
|
var volume *corev1api.Volume
|
|
for _, v := range ds.Spec.Template.Spec.Volumes {
|
|
if v.Name == HostPodVolumeMount {
|
|
volume = &v
|
|
break
|
|
}
|
|
}
|
|
|
|
if volume == nil {
|
|
return "", errors.New("host pod volume is not found")
|
|
}
|
|
|
|
if volume.HostPath == nil {
|
|
return "", errors.New("host pod volume is not a host path volume")
|
|
}
|
|
|
|
if volume.HostPath.Path == "" {
|
|
return "", errors.New("host pod volume path is empty")
|
|
}
|
|
|
|
return volume.HostPath.Path, nil
|
|
}
|
|
|
|
func HostPodVolumeMountPath() string {
|
|
return "/" + HostPodVolumeMountPoint
|
|
}
|
|
|
|
func HostPodVolumeMountPathWin() string {
|
|
return "\\" + HostPodVolumeMountPoint
|
|
}
|