nmstate / kubernetes-nmstate

Declarative node network configuration driven through Kubernetes API.
GNU General Public License v2.0
172 stars 86 forks source link

handler: Retry enactment update #1215

Closed qinqon closed 8 months ago

qinqon commented 9 months ago

Is this a BUG FIX or a FEATURE ?:

/kind bug

What this PR does / why we need it: At some scenarios it takes some retries to update the enactment status if the network is not totally settle. This change add some retries to enactment status update.

Retry enactment status update on failure.
mkowalski commented 8 months ago

/approve

kubevirt-bot commented 8 months ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mkowalski

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/nmstate/kubernetes-nmstate/blob/main/OWNERS)~~ [mkowalski] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
mkowalski commented 8 months ago

/lgtm