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.

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

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

Merged

Conversation

katiewasnothere
Copy link
Contributor

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