Skip to content

Conversation

@benluddy
Copy link

No description provided.

// are purely passthroughs to the delegate. During a rebase, consider whether or not it is
// safe and appropriate for a new method added to the method set of storage.Interface to
// perform retries.
delegate storage.Interface

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah, good call. thanks.

err error
)
return rv, OnError(ctx, DefaultRetry, IsRetriableErrorOnRead, func() error {
rv, err = c.delegate.GetCurrentResourceVersion(ctx)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes, this is a read operation.

err error
)
return rv, OnError(ctx, DefaultRetry, IsRetriableErrorOnRead, func() error {
rv, err = c.delegate.GetCurrentResourceVersion(ctx)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the err will always be propagated.

@p0lyn0mial
Copy link

/lgtm

@p0lyn0mial p0lyn0mial merged commit 182485d into openshift:openshift-apiserver-4.20-kubernetes-1.33 Sep 17, 2025
@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Sep 17, 2025
@benluddy benluddy changed the title UPSTREAM: <carry> Add retries for GetCurrentResourceVersion. OCPBUGS-61754: UPSTREAM: <carry> Add retries for GetCurrentResourceVersion. Sep 17, 2025
@openshift-ci-robot
Copy link

@benluddy: Jira Issue OCPBUGS-61754: Some pull requests linked via external trackers have merged:

The following pull request, linked via external tracker, has not merged:

All associated pull requests must be merged or unlinked from the Jira bug in order for it to move to the next state. Once unlinked, request a bug refresh with /jira refresh.

Jira Issue OCPBUGS-61754 has not been moved to the MODIFIED state.

In response to this:

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants