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

Are the integration tests run anywhere? #936

Closed
TBBle opened this issue Jan 31, 2021 · 3 comments
Closed

Are the integration tests run anywhere? #936

TBBle opened this issue Jan 31, 2021 · 3 comments

Comments

@TBBle
Copy link
Contributor

TBBle commented Jan 31, 2021

I just discovered, while poking at #901, that the AppVeyor CI script appears to just compile the integration tests, but does not run them.

Are they being run somewhere I can't see (or have overlooked)? Would it be useful to get them running on AppVeyor?

I was looking into using the AppVeyor Job Workflows to have the compile done once (as it is now), then then run the integration tests on both the VS2017 (Windows Server 2016) and VS2019 (Windows Server 2019) images, but I stopped because I couldn't see an easy way to pass the artifacts generated by the first job through to the fan-out jobs to run.

I suspect it's possible with sufficient script hackery, but I didn't want to implement that if it was going to end up being uninteresting, e.g., if I was replicating something that already existed somewhere.

@TBBle TBBle changed the title Are the integration tests run anywhere Are the integration tests run anywhere? Jan 31, 2021
@katiewasnothere
Copy link
Contributor

katiewasnothere commented Feb 1, 2021

As part of our internal package process, we do run the cri-containerd test suite, but that's it. Unfortunately, the tests in test/functional are pretty out of date and still rely on docker (for images). I've been working on updating the test/functional tests, (see related containerd/containerd#4848), but haven't finished up there. As for the test/runhcs tests, since we no longer use runhcs, we don't run these. Lastly, I imagine the tests under test/containerd-shim-runhcs-v1 are also out of date, but haven't looked recently.

The main reason we don't run them via appveyor is that LCOW tests require an LCOW kernel/initrd to run.

@TBBle
Copy link
Contributor Author

TBBle commented Feb 2, 2021

Thank you, that makes sense. I got what I needed for my own functional test (worked on RS5), and will wait for your efforts on test/functional to bear fruit before worrying about it further. I think the platform where I've seen the problem (20H2) isn't available on AppVeyor anyway.

I am curious, if runhcs is being left untested and bit-rotting, have you considered deleting it and its tests?

@katiewasnothere
Copy link
Contributor

@TBBle

I am curious, if runhcs is being left untested and bit-rotting, have you considered deleting it and its tests?

Yep! It's on our backlog, but we haven't looked at it in a while.

TBBle added a commit to TBBle/hcsshim that referenced this issue Mar 14, 2021
This gives a comparison-point for RS5, as used by AppVeyor, compared to
my 20H2 desktop machine.

TODO: Actually integrate a proper test-suite run, possibly using GitHub
Actions, per microsoft#936 and microsoft#970.

Signed-off-by: Paul "TBBle" Hampson <Paul.Hampson@Pobox.com>
TBBle added a commit to TBBle/hcsshim that referenced this issue Mar 14, 2021
This gives a comparison-point for RS5, as used by AppVeyor, compared to
my 20H2 desktop machine.

TODO: Actually integrate a proper test-suite run, possibly using GitHub
Actions, per microsoft#936 and microsoft#970.

Signed-off-by: Paul "TBBle" Hampson <Paul.Hampson@Pobox.com>
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

No branches or pull requests

2 participants