Skip to content

Pass explicit JUnit outfile to critest.exe in Windows workflow.#6912

Merged
dmcgowan merged 1 commit intocontainerd:mainfrom
aznashwan:windows-ci-junit-outfile
May 9, 2022
Merged

Pass explicit JUnit outfile to critest.exe in Windows workflow.#6912
dmcgowan merged 1 commit intocontainerd:mainfrom
aznashwan:windows-ci-junit-outfile

Conversation

@aznashwan
Copy link
Copy Markdown
Contributor

Following kubernetes-sigs/cri-tools#910 the Ginkgo reporters are left
configured with default settings and thus do not generate a JUnit report
file unless we explicitly pass a path for the outfile in the Windows
workflow when calling critest.exe.

Signed-off-by: Nashwan Azhari [email protected]

Following kubernetes-sigs/cri-tools#910 the Ginkgo reporters are left
configured with default settings and thus do not generate a JUnit report
file unless we explicitly pass a path for the outfile in the Windows
workflow when calling critest.exe.

Signed-off-by: Nashwan Azhari <[email protected]>
@k8s-ci-robot
Copy link
Copy Markdown

Hi @aznashwan. Thanks for your PR.

I'm waiting for a containerd member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@aznashwan
Copy link
Copy Markdown
Contributor Author

aznashwan commented May 9, 2022

This PR should fix the missing CRI test results on testgrid, which shows the tests which went unreported over the weekend because the junit_.xml file which was usually generated/uploaded by critest.exe is no longer being created at all in later runs.

@dmcgowan @dcantah @kzys could one of you please take a look at this so we don't potentially lose more test results because of it until we can get a proper fix in kubernetes-sigs/cri-tools itself?

Copy link
Copy Markdown
Member

@estesp estesp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Member

@dcantah dcantah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks @aznashwan

@dmcgowan dmcgowan merged commit 1e749e5 into containerd:main May 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants