Skip to content

Add a utility function to exec in shimdiag for cri-containerd tests#992

Merged
katiewasnothere merged 1 commit intomicrosoft:masterfrom
katiewasnothere:test_execinhost_utility
Apr 8, 2021
Merged

Add a utility function to exec in shimdiag for cri-containerd tests#992
katiewasnothere merged 1 commit intomicrosoft:masterfrom
katiewasnothere:test_execinhost_utility

Conversation

@katiewasnothere
Copy link

This PR creates a new utility function in the cri-containerd test suite for running shimdiag exec commands. This PR will eventually be used by #931 to verify resources are updated as expected.

This PR additionally refactors some code related to pipe creation to prevent code duplication for running shimdiag exec requests from the test suite.

Signed-off-by: Kathryn Baldauf kabaldau@microsoft.com

@katiewasnothere katiewasnothere requested a review from a team as a code owner April 8, 2021 03:28
@katiewasnothere katiewasnothere force-pushed the test_execinhost_utility branch 2 times, most recently from 1576885 to 554c3b9 Compare April 8, 2021 03:43
@katiewasnothere katiewasnothere force-pushed the test_execinhost_utility branch from 554c3b9 to 623a520 Compare April 8, 2021 20:30
Signed-off-by: Kathryn Baldauf <kabaldau@microsoft.com>
@katiewasnothere katiewasnothere force-pushed the test_execinhost_utility branch from 623a520 to 8a22e26 Compare April 8, 2021 20:47
@katiewasnothere katiewasnothere merged commit da33ecd into microsoft:master Apr 8, 2021
@katiewasnothere katiewasnothere deleted the test_execinhost_utility branch April 8, 2021 20:54
princepereira pushed a commit to princepereira/hcsshim that referenced this pull request Aug 29, 2024
…_utility

Add a utility function to exec in shimdiag for cri-containerd tests
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