test(integration): enable tests for RAM cache and zonal buckets - #4994
test(integration): enable tests for RAM cache and zonal buckets#4994vipnydav wants to merge 1 commit into
Conversation
vipnydav
added
the
execute-integration-tests-on-zb
github-actions
Bot
added
the
remind-reviewers
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the integration test configurations in tools/integration_tests/test_config.yaml by enabling RAM cache tests (TestCacheFileForRangeReadTrueWithRamCache) with specific protocol and kernel reader flags, adding a new Pirlo test configuration, and enabling zonal bucket compatibility for streaming writes. Feedback was provided to add the missing --enable-kernel-reader=false flag to one of the Pirlo test configurations to prevent bypassing the file cache.
106652c to
5c1422e
Compare
|
Hi @meet2mky, @abhishek10004, your feedback is needed to move this pull request forward. This automated reminder was triggered because there has been no activity for over 24 hours. Please provide your input when you have a moment. Thank you! |
5 similar comments
|
Hi @meet2mky, @abhishek10004, your feedback is needed to move this pull request forward. This automated reminder was triggered because there has been no activity for over 24 hours. Please provide your input when you have a moment. Thank you! |
|
Hi @meet2mky, @abhishek10004, your feedback is needed to move this pull request forward. This automated reminder was triggered because there has been no activity for over 24 hours. Please provide your input when you have a moment. Thank you! |
|
Hi @meet2mky, @abhishek10004, your feedback is needed to move this pull request forward. This automated reminder was triggered because there has been no activity for over 24 hours. Please provide your input when you have a moment. Thank you! |
|
Hi @meet2mky, @abhishek10004, your feedback is needed to move this pull request forward. This automated reminder was triggered because there has been no activity for over 24 hours. Please provide your input when you have a moment. Thank you! |
|
Hi @meet2mky, @abhishek10004, your feedback is needed to move this pull request forward. This automated reminder was triggered because there has been no activity for over 24 hours. Please provide your input when you have a moment. Thank you! |
1 participant
Description
This PR enables integration tests that were previously blocked or disabled, and cleans up old configurations:
TestCacheFileForRangeReadTrueWithRamCache) which were disabled due to bug b/383682524. We are re-enabling this per the discussion in the bug.interrupttest for Zonal buckets (zonal: true) now that read support for unfinalized objects has been fixed, as requested in bug b/417136852.Link to the issue in case of a bug fix.
Testing details
Any backward incompatible change? If so, please explain.
No.