Skip to content

Commit de9368d

Browse files
Merge pull request openshift#1100 from SchSeba/merge-bot-master
Merge https://github.com/k8snetworkplumbingwg/sriov-network-operator:master into main
2 parents 6e91e95 + 065a8d8 commit de9368d

File tree

3 files changed

+8
-4
lines changed

3 files changed

+8
-4
lines changed

bundle/manifests/sriov-network-operator.clusterserviceversion.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,7 @@ metadata:
100100
categories: Networking
101101
certified: "false"
102102
containerImage: quay.io/openshift/origin-sriov-network-operator:4.19
103-
createdAt: "2025-06-02T23:48:40Z"
103+
createdAt: "2025-06-04T23:48:47Z"
104104
description: An operator for configuring SR-IOV components and initializing SRIOV
105105
network devices in Openshift cluster.
106106
features.operators.openshift.io/cnf: "false"

manifests/stable/sriov-network-operator.clusterserviceversion.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,7 @@ metadata:
100100
categories: Networking
101101
certified: "false"
102102
containerImage: quay.io/openshift/origin-sriov-network-operator:4.19
103-
createdAt: "2025-06-02T23:48:40Z"
103+
createdAt: "2025-06-04T23:48:47Z"
104104
description: An operator for configuring SR-IOV components and initializing SRIOV
105105
network devices in Openshift cluster.
106106
features.operators.openshift.io/cnf: "false"

pkg/daemon/daemon.go

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -260,9 +260,13 @@ func (dn *NodeReconciler) Reconcile(ctx context.Context, req ctrl.Request) (ctrl
260260
reqLogger.Error(err, "failed to handle drain")
261261
return ctrl.Result{}, err
262262
}
263-
// drain is still in progress we don't need to re-queue the request as the operator will update the annotation
263+
264+
// TODO: remove this after we stop using the node annotation
265+
// drain is still in progress we will still requeue the request in case there is an un-expect state in the draining
266+
// this will allow the daemon to try again.
264267
if drainInProcess {
265-
return ctrl.Result{}, nil
268+
reqLogger.Info("node drain still in progress, requeue")
269+
return ctrl.Result{RequeueAfter: consts.DaemonRequeueTime}, nil
266270
}
267271
}
268272

0 commit comments

Comments
 (0)