jozzsi
jozzsi
Perhaps I misunderstood, but the current default for dracut config (that needs to be preserved) is `/etc/dracut.conf.d` - see e.g. `man/dracut.8.adoc`
(Unfortunately) the new test needs to be added explicitly to `.github/workflows/integration.yml` (and .github/workflows/integration-extra.yml), otherwise it would not run.
> Missed that, added (and hope I did it correctly)! Thanks! Thanks @vittyvk - I hope we can also add this test under 'omitsystemd' in integration-extra.yml
A lot of the CI failures caused by how the rootfs is constructed in the test (which is not the core part of this PR, so I recommend sticking with...
Thank you @vittyvk for working together. Dropping `dsync` would ensure that the test works with busybox implementation of dsync as well (and would fix the remaining alpine CI failure). This...
> There is absolutely no reason whatsoever to have multiple of these. I can think of at least two reasons: - distribution wants to fork just a dracut module, and...
> Is there any test coverage for this? Well, it is documented here - https://dracut-ng.github.io/dracut-ng/developer/modules.html > Modules with lower numbers are installed first. This is important because the dracut install...
@walid-omari can you please retest with dracut v109, ideally on Ubuntu. CC @dracut-ng/ubuntu-maint
~~There are few more places in the codebase. Marking these on a case-by-case basis might be useful, instead of disabling it project wide.~~ I misread the code, this PR actually...
This proposal looks sensible to me if the implementation does not add a lot of complexity and if it is modular enough - which we can discuss during PR review....