Skip to content

Separate volume tests#1198

Closed
ambarve wants to merge 2 commits intomicrosoft:mainfrom
ambarve:separate_volume_tests
Closed

Separate volume tests#1198
ambarve wants to merge 2 commits intomicrosoft:mainfrom
ambarve:separate_volume_tests

Conversation

@ambarve
Copy link
Copy Markdown
Contributor

@ambarve ambarve commented Oct 18, 2021

Tests for changes added in kevpar/containerd#27

@ambarve ambarve requested a review from a team as a code owner October 18, 2021 03:46
@ambarve ambarve marked this pull request as draft October 18, 2021 03:47
@ambarve
Copy link
Copy Markdown
Contributor Author

ambarve commented Oct 18, 2021

Marked as draft to make sure this is not merged before kevpar/containerd#27, kevpar/cri#6 and #1188 is merged.

@ambarve ambarve marked this pull request as ready for review January 27, 2022 20:12
@ambarve ambarve marked this pull request as draft February 10, 2022 23:10
@ambarve
Copy link
Copy Markdown
Contributor Author

ambarve commented Feb 10, 2022

Marked as draft temporarily as I am planning to rebase this on master and add one more test.

Signed-off-by: Amit Barve <ambarve@microsoft.com>
Signed-off-by: Amit Barve <ambarve@microsoft.com>
@ambarve ambarve force-pushed the separate_volume_tests branch from ead4ae7 to 99785ed Compare February 11, 2022 02:04
@ambarve
Copy link
Copy Markdown
Contributor Author

ambarve commented Oct 23, 2023

We might remove the support for overriding the scratch directory location. In that case the tests that this PR adds won't be applicable.

@ambarve ambarve closed this Oct 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants