Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Robustness patch delete #18242

Draft
wants to merge 7 commits into
base: main
Choose a base branch
from
Draft

Robustness patch delete #18242

wants to merge 7 commits into from

Conversation

serathius
Copy link
Member

Need to still fix some tests.

@serathius
Don't validate watch patching time as we no longer use watch to decid…
0ee0da4 
…e to discard

Signed-off-by: Marek Siarkowicz <siarkowicz@google.com>
@serathius
Refactor patchOperations to handle each operation separetly, which is…
a47ad53 
… useful to implement patching for non-put

Signed-off-by: Marek Siarkowicz <siarkowicz@google.com>
@serathius
Unify patchOperation arguments to be map from keyValue to int64
1dc2165 
Signed-off-by: Marek Siarkowicz <siarkowicz@google.com>
@serathius
Unify calculating returnTime
eeeda9e 
Signed-off-by: Marek Siarkowicz <siarkowicz@google.com>
@serathius
Refactor watch response util functions for unit tests
8ffe921 
Signed-off-by: Marek Siarkowicz <siarkowicz@google.com>
@serathius
Use client request count for uniqness
a7c3843 
Signed-off-by: Marek Siarkowicz <siarkowicz@google.com>
@serathius
Patch delete request based on their uniqness in client requests
b2a21be 
Signed-off-by: Marek Siarkowicz <siarkowicz@google.com>
@k8s-ci-robot
Copy link

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
area/robustness-testing area/testing do-not-merge/work-in-progress size/XL
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@serathius @k8s-ci-robot