New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Refactor all storage Operation Generator functions to remove top level errors reducing event spam. Functionality moved to inner retryable/backoffable funcs #88537
Conversation
032508e
to
59cb375
Compare
/assign @jsafrane |
/priority backlog |
59cb375
to
48532f0
Compare
48532f0
to
f8667a1
Compare
/retest |
2 similar comments
/retest |
/retest |
#74988 mentions errors in GenerateAttachVolumeFunc, but the function does not return errors any longer. Still, similar change should be done in all other operations. /approve I'd like Hemant or Jing to approve that there are no important errors accidentally hidden now. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: davidz627, jsafrane 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:
Approvers can indicate their approval by writing |
…l errors reducing event spam. Functionality moved to inner retryable/backoffable funcs
/retest |
@davidz627: PR needs rebase. 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 kubernetes/test-infra repository. |
/close |
@davidz627: Closed this PR. 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 kubernetes/test-infra repository. |
/kind cleanup
/assign @misterikkit @msau42
/cc @saad-ali @jingxu97 @mucahitkurt
Fixes #74988