Skip to content

Conversation

@tomzawadzki
Copy link
Contributor

No description provided.

@tomzawadzki tomzawadzki changed the title All jobs Enable all jobs Feb 21, 2025
This patch indiscriminately adds configuration and
execution of jobs that match runs from last year.

Please note that names match the old ones,
but effects of each SPDK_TEST_* flags do not have
to match just the name.

Often times some tests were lumped together
for simplicity, optimizing test time or just to
place it anywhere if nothing else matched.
Neither does 'vm' or 'phy' mean all (or even most)
of the job _requires_ a specific machine.

Meanwhile it can be good starting point,
please note that the best indicator of not
missing anything crucial is "run_test" summary.

Signed-off-by: Tomasz Zawadzki <[email protected]>
At some point runnning the whole autorun.sh for tests that
never ran any of the SPDK code became too much. Those instead
were relagated to running within containers on much larger scale,
as a set of pre-tests before actual autorun.sh jobs started.

For the most part tests here cover building, packaging,
code formatting, etc.

Signed-off-by: Tomasz Zawadzki <[email protected]>
For purose of figuring out to what extent the current
autorun.sh and jobs are ready for execution on GitHub -
disabled cancelation of the jobs and extended timeout.

Signed-off-by: Tomasz Zawadzki <[email protected]>
run: |
mkdir -p /opt/output/nvme
output_dir=/opt/output/nvme /opt/spdk/autorun.sh /opt/configs/nvme.conf
Copy link
Contributor

Choose a reason for hiding this comment

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

this duplication here is pretty terrible

- name: blobfs-phy-autotest
run: |
mkdir -p /opt/output/blobfs-phy-autotest
output_dir=/opt/output/blobfs-phy-autotest /opt/spdk/autorun.sh /opt/configs/blobfs-phy-autotest.conf
Copy link
Contributor

Choose a reason for hiding this comment

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

blobfs was deprecated

- name: freebsd-vm-autotest
run: |
mkdir -p /opt/output/freebsd-vm-autotest
output_dir=/opt/output/freebsd-vm-autotest /opt/spdk/autorun.sh /opt/configs/freebsd-vm-autotest.conf
Copy link
Contributor

Choose a reason for hiding this comment

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

freebsd was merged

Comment on lines +229 to +235
- name: check-so-deps-docker-autotest
run: |
touch empty.conf
mkdir -p /opt/output/check-so-deps-docker-autotest
output_dir=/opt/output/check-so-deps-docker-autotest rootdir=/opt/spdk source "/opt/spdk/test/common/autobuild_common.sh" "./empty.conf"
ocf_precompile
SPDK_ABI_DIR=/opt/spdk-abi check_so_deps # TODO: pull spdk-abi repo
Copy link
Contributor

Choose a reason for hiding this comment

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

merged

Comment on lines +237 to +242
- name: doc-docker-autotest
run: |
touch empty.conf
mkdir -p /opt/output/doc-docker-autotest
output_dir=/opt/output/doc-docker-autotest rootdir=/opt/spdk source "/opt/spdk/test/common/autobuild_common.sh" "./empty.conf"
build_doc
Copy link
Contributor

Choose a reason for hiding this comment

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

merged

Comment on lines +222 to +227
- name: check-format-docker-autotest
run: |
touch empty.conf
mkdir -p /opt/output/check-format-docker-autotest
output_dir=/opt/output/check-format-docker-autotest rootdir=/opt/spdk source "/opt/spdk/test/common/autobuild_common.sh" "./empty.conf"
CHECK_FORMAT_ONLY_DIFF=yes check_format
Copy link
Contributor

Choose a reason for hiding this comment

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

merged

Comment on lines +214 to +220
- name: build-files-docker-autotest
run: |
touch empty.conf
mkdir -p /opt/output/build-files-docker-autotest
output_dir=/opt/output/build-files-docker-autotest rootdir=/opt/spdk source "/opt/spdk/test/common/autobuild_common.sh" "./empty.conf"
build_files
dpdk_pci_api
Copy link
Contributor

Choose a reason for hiding this comment

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

merged

Comment on lines +206 to +212
- name: unittest-main-docker-autotest
run: |
mkdir -p /opt/output/unittest-main-docker-autotest
output_dir=/opt/output/unittest-main-docker-autotest rootdir=/opt/spdk source "/opt/spdk/test/common/autobuild_common.sh" "/opt/configs/unittest-main-docker-autotest.conf"
ocf_precompile
unittest_build
/opt/spdk/test/unit/unittest.sh
Copy link
Contributor

Choose a reason for hiding this comment

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

merged

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.

2 participants