Second round of output cleaning

[scaronni]

Second out of output refining. Example with autoinstall and multiple modules, before:

dkms: running auto installation service for kernel 6.10.7-200.fc40.x86_64
Sign command: /lib/modules/6.10.7-200.fc40.x86_64/build/scripts/sign-file
Signing key: /var/lib/dkms/mok.key
Public certificate (MOK): /var/lib/dkms/mok.pub

Cleaning build area... done.
Building module(s)........................ done.
Signing module /var/lib/dkms/nvidia/560.35.03/build/kernel-open/nvidia.ko
Signing module /var/lib/dkms/nvidia/560.35.03/build/kernel-open/nvidia-modeset.ko
Signing module /var/lib/dkms/nvidia/560.35.03/build/kernel-open/nvidia-drm.ko
Signing module /var/lib/dkms/nvidia/560.35.03/build/kernel-open/nvidia-uvm.ko
Signing module /var/lib/dkms/nvidia/560.35.03/build/kernel-open/nvidia-peermem.ko
Cleaning build area... done.

Installing /lib/modules/6.10.7-200.fc40.x86_64/extra/nvidia.ko.xz
Installing /lib/modules/6.10.7-200.fc40.x86_64/extra/nvidia-modeset.ko.xz
Installing /lib/modules/6.10.7-200.fc40.x86_64/extra/nvidia-drm.ko.xz
Installing /lib/modules/6.10.7-200.fc40.x86_64/extra/nvidia-uvm.ko.xz
Installing /lib/modules/6.10.7-200.fc40.x86_64/extra/nvidia-peermem.ko.xz

Running depmod.... done.
Sign command: /lib/modules/6.10.7-200.fc40.x86_64/build/scripts/sign-file
Signing key: /var/lib/dkms/mok.key
Public certificate (MOK): /var/lib/dkms/mok.pub

Cleaning build area... done.
Building module(s)... done.
Signing module /var/lib/dkms/xpadneo/0.9.6/build/hid-xpadneo.ko
Cleaning build area... done.

Installing /lib/modules/6.10.7-200.fc40.x86_64/extra/hid-xpadneo.ko.xz

Running depmod.... done.
dkms autoinstall on 6.10.7-200.fc40.x86_64/x86_64 succeeded for nvidia xpadneo
dkms: autoinstall for kernel 6.10.7-200.fc40.x86_64 Done. 
dkms: running auto installation service for kernel 6.10.7-200.fc40.x86_64
dkms: autoinstall for kernel 6.10.7-200.fc40.x86_64 Done. 

Most of the autoinstall output is useless and not well placed. Also the depmod line should be attached to the "Installng [..]" output and separated from the "Sign [..]" output.

After changing the output, it's structured like this, with each block separated by context:

Sign command: /lib/modules/6.10.7-200.fc40.x86_64/build/scripts/sign-file
Signing key: /var/lib/dkms/mok.key
Public certificate (MOK): /var/lib/dkms/mok.pub

Cleaning build area... done.
Building module(s)........................ done.
Signing module /var/lib/dkms/nvidia/560.35.03/build/kernel-open/nvidia.ko
Signing module /var/lib/dkms/nvidia/560.35.03/build/kernel-open/nvidia-modeset.ko
Signing module /var/lib/dkms/nvidia/560.35.03/build/kernel-open/nvidia-drm.ko
Signing module /var/lib/dkms/nvidia/560.35.03/build/kernel-open/nvidia-uvm.ko
Signing module /var/lib/dkms/nvidia/560.35.03/build/kernel-open/nvidia-peermem.ko
Cleaning build area... done.

Installing /lib/modules/6.10.7-200.fc40.x86_64/extra/nvidia.ko.xz
Installing /lib/modules/6.10.7-200.fc40.x86_64/extra/nvidia-modeset.ko.xz
Installing /lib/modules/6.10.7-200.fc40.x86_64/extra/nvidia-drm.ko.xz
Installing /lib/modules/6.10.7-200.fc40.x86_64/extra/nvidia-uvm.ko.xz
Installing /lib/modules/6.10.7-200.fc40.x86_64/extra/nvidia-peermem.ko.xz
Running depmod.... done.

Sign command: /lib/modules/6.10.7-200.fc40.x86_64/build/scripts/sign-file
Signing key: /var/lib/dkms/mok.key
Public certificate (MOK): /var/lib/dkms/mok.pub

Cleaning build area... done.
Building module(s)... done.
Signing module /var/lib/dkms/xpadneo/0.9.6/build/hid-xpadneo.ko
Cleaning build area... done.

Installing /lib/modules/6.10.7-200.fc40.x86_64/extra/hid-xpadneo.ko.xz
Running depmod.... done.

Autoinstall on 6.10.8-200.fc40.x86_64 succeeded for modules nvidia xpadneo.

After spending lot of time trying to figure out why the output trapped by the tests is slightly different on old Alpine containers and the old Ubuntu VM that was used for the test, I narrowed it down to the invoke_command combined with stdbuf. Funny thing is that if you run the commands directly on the containers the output is consistent, and does not exhibit the extra blank line before "Cleaning build area".

To avoid this issue entirely, in generalize_expected_output remove all blank lines, so the output is trapped 100% the same on all containers.

[scaronni]

Thanks for the review!

[evelikov]

Greetings, I have emerged from the woodwork - sorry for dropping off the radar.

Can we split off /etc/sysconfig in separate commit please - I could swear I've saw some distro use it.

[scaronni]

Greetings, I have emerged from the woodwork - sorry for dropping off the radar.

Can we split off /etc/sysconfig in separate commit please - I could swear I've saw some distro use it.

Sure, ok.

[scaronni]

/etc/sysconfig restored.

[evelikov]

Note: removing sysconfig thing is a good idea IMHO. We just need to a) ensure we have dkms.conf equivalent + is parsed and b) add a bit scary note in the release notes.

But that's for another day. Thanks, updated PR good.