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

Minor updates #33

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

tim-day-387
Copy link
Contributor

I noticed that bcachefs-tools got dropped from Debian - it caused an install failure when setting up ktest on my laptop. I saw you had a patch to build it when running interactively. I imagine we'd need to unconditionally build it from git, even in non-interactive mode. Curious your thoughts.

Also, added a small Lustre convenience ktest. Still trying to find cycles for my OSD ktests.

After the removal [1] of bcachefs-tools from Debian,
the installation fails. It will need to be built
from source instead.

[1] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=%231079375

Signed-off-by: Timothy Day <[email protected]>
This just boots using the kernel required for
Lustre. I find myself doing this quite often
with llmount.ktest. An explicit boot.ktest
would be helpful.

Signed-off-by: Timothy Day <[email protected]>
@koverstreet
Copy link
Owner

Well, the reason I switched to the debian package is because the CI has multiple workers sharing the same ktest and bcachefs-tools repos, and different workers racing and trying to build bcachefs-tools at the same time doesn't work.

So test dependencies really need to be packaged for non interactive use :/

yet another reason we need to get this converted to nixos - that will make all this cleaner and we won't be relying on any centralized package repository

@tim-day-387
Copy link
Contributor Author

tim-day-387 commented Sep 7, 2024

Couple quick questions:

  1. Do you have some example config files for the CI? Or a quick run down? I was testing it out - but only able to get it partially working (i.e. Apache renders my user and branch, but I couldn't the CI to generate a list of jobs for the workers to consume). I wanted to try leveraging that for running ktests locally in parallel. And it'd help me understand the concurrency issue with the workers building bcachefs-tools.
  2. I see the nixOS stuff in the repository. Is that just a skeleton? Are you aiming to remove Debian long-term and move entirely to nixOS?

We could build bcachefs-tools into the root image rather than building it at test time. But then something (or someone) would have to periodically update it. Although the Debian bcachefs-tools lagged behind, so maybe that isn't a huge concern.

@koverstreet koverstreet force-pushed the master branch 3 times, most recently from c0037b0 to 9a71187 Compare October 14, 2024 21:48
@koverstreet koverstreet force-pushed the master branch 2 times, most recently from 95b4fc3 to caad18f Compare November 8, 2024 00:38
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