Skip to content

Enable all jobs #20

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

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Enable all jobs #20

wants to merge 3 commits into from

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 <tomasz.zawadzki@nutanix.com>
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 <tomasz.zawadzki@nutanix.com>
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 <tomasz.zawadzki@nutanix.com>
@@ -54,6 +54,193 @@ steps:
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

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