dotnet-docs-samples icon indicating copy to clipboard operation
dotnet-docs-samples copied to clipboard

samples(storage): add samples and test cases for bucket soft delete policy

Open mahendra-google opened this issue 7 months ago • 12 comments

Add samples and test cases for bucket soft delete policy as follows:-

  • [ ] storage_disable_soft_delete
  • [ ] storage_get_soft_delete_policy
  • [ ] storage_set_soft_delete_policy

mahendra-google avatar May 19 '25 10:05 mahendra-google

Here is the summary of changes.

You are about to add 3 region tags.

This comment is generated by snippet-bot. If you find problems with this result, please file an issue at: https://github.com/googleapis/repo-automation-bots/issues. To update this comment, add snippet-bot:force-run label or use the checkbox below:

  • [ ] Refresh this comment

snippet-bot[bot] avatar May 19 '25 10:05 snippet-bot[bot]

This PR is ready for review

mahendra-google avatar May 23 '25 03:05 mahendra-google

Gentle Reminder :- This PR is ready for review.

mahendra-google avatar Jun 03 '25 08:06 mahendra-google

This PR is ready for re-review

mahendra-google avatar Jun 12 '25 07:06 mahendra-google

@mahendra-google please fix test failure

cy-yun avatar Jun 13 '25 23:06 cy-yun

@cy-yun BucketDisableSoftDeletePolicyTest.TestBucketDisableSoftDeletePolicy test is passing locally without any issues . Also on [Kokoro CI Windows 3] server it is passing in the first place and next time when it is running its failing. I am not sure what is the issue.

mahendra-google avatar Jun 16 '25 09:06 mahendra-google

@cy-yun BucketDisableSoftDeletePolicyTest.TestBucketDisableSoftDeletePolicy test is passing locally without any issues . Also on [Kokoro CI Windows 3] server it is passing in the first place and next time when it is running its failing. I am not sure what is the issue.

Then there's a race condition or a flake that needs to be fixed anyway.

amanda-tarafa avatar Jun 16 '25 17:06 amanda-tarafa

@cy-yun BucketDisableSoftDeletePolicyTest.TestBucketDisableSoftDeletePolicy test is passing locally without any issues . Also on [Kokoro CI Windows 3] server it is passing in the first place and next time when it is running its failing. I am not sure what is the issue.

Then there's a race condition or a flake that needs to be fixed anyway.

@amanda-tarafa if I do any code changes to resolve flake or race condition how can I test if error is fixed or not because I dont have previledge to add kokoro label to run the tests?

mahendra-google avatar Jun 19 '25 07:06 mahendra-google

@cy-yun BucketDisableSoftDeletePolicyTest.TestBucketDisableSoftDeletePolicy test is passing locally without any issues . Also on [Kokoro CI Windows 3] server it is passing in the first place and next time when it is running its failing. I am not sure what is the issue.

Then there's a race condition or a flake that needs to be fixed anyway.

@amanda-tarafa if I do any code changes to resolve flake or race condition how can I test if error is fixed or not because I dont have previledge to add kokoro label to run the tests?

cy-yun , @amanda-tarafa Team (xWF: GCS Clients Development) confirmed that they are working on to add kokoro force run label on PR. Till the time I get permission to add kokoro force run label, I will not be able to resolve the error (race condition or test flakiness ).

mahendra-google avatar Jun 23 '25 13:06 mahendra-google

@cy-yun BucketDisableSoftDeletePolicyTest.TestBucketDisableSoftDeletePolicy test is passing locally without any issues . Also on [Kokoro CI Windows 3] server it is passing in the first place and next time when it is running its failing. I am not sure what is the issue.

Then there's a race condition or a flake that needs to be fixed anyway.

@amanda-tarafa if I do any code changes to resolve flake or race condition how can I test if error is fixed or not because I dont have previledge to add kokoro label to run the tests?

@cy-yun , @amanda-tarafa Team (xWF: GCS Clients Development) confirmed that they are working on to add kokoro force run label on PR. Till that time error (race condition or test flakiness ) cannot be resolved.

mahendra-google avatar Jun 23 '25 13:06 mahendra-google

@cy-yun Can you add kokoro label to run tests again?

mahendra-google avatar Jun 24 '25 14:06 mahendra-google

@cy-yun Gentle Reminder , PR is ready for re-review

mahendra-google avatar Jun 27 '25 01:06 mahendra-google