Skip to content

test: reorganize integration tests and add CRI-O test coverage#3785

Merged
dims merged 1 commit intogoogle:masterfrom
dims:reorganize-integration-tests
Dec 13, 2025
Merged

test: reorganize integration tests and add CRI-O test coverage#3785
dims merged 1 commit intogoogle:masterfrom
dims:reorganize-integration-tests

Conversation

@dims
Copy link
Collaborator

@dims dims commented Dec 13, 2025

Reorganize integration tests by creating a new common package for
runtime-independent tests that work with any container runtime
(Docker, CRI-O, containerd, etc.):

  • TestAttributeInformationIsReturned: tests v2 attributes API
  • TestHealthzOk: tests health endpoint
  • TestMachineInformationIsReturned: tests machine info API
  • TestMachineStatsIsReturned: tests machine stats API

Deprecate the old healthz package in favor of the new common package.

Add new CRI-O-specific integration tests:

  • TestCrioContainerNetworkStats: tests network stats (skipped - CI issue)
  • TestCrioContainerSpec: tests container spec properties
  • TestCrioContainerDeletionExitCode: tests exit code events (skipped)
  • TestCrioFilesystemStats: tests filesystem statistics

Update build infrastructure to run common tests for both Docker and
CRI-O integration test suites.

@google-cla
Copy link

google-cla bot commented Dec 13, 2025

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@dims dims changed the title Reorganize integration tests [WIP] Reorganize integration tests Dec 13, 2025
@dims dims force-pushed the reorganize-integration-tests branch 2 times, most recently from 500649e to 849a59b Compare December 13, 2025 13:47
Add new CRI-O integration tests to match Docker test coverage:
- TestCrioContainerNetworkStats: tests network statistics collection
- TestCrioContainerSpec: tests container spec properties
- TestCrioContainerDeletionExitCode: tests exit code in deletion events
- TestCrioFilesystemStats: tests filesystem statistics collection

Signed-off-by: Davanum Srinivas <davanum@gmail.com>
@dims dims force-pushed the reorganize-integration-tests branch from 75bea57 to 2ec7792 Compare December 13, 2025 14:06
@dims dims merged commit 1e4adf8 into google:master Dec 13, 2025
8 checks passed
@dims dims changed the title [WIP] Reorganize integration tests test: reorganize integration tests and add CRI-O test coverage Dec 13, 2025
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.

1 participant