serhepopovych / simple-cdd-meta

Debian and it's derivatives Simple-CDD preseed configuration with profiles
MIT License
6 stars 3 forks source link

Network installations using PXE are broken since #16 that reworks profiles support #24

Closed serhepopovych closed 4 years ago

serhepopovych commented 4 years ago

It is caused by simple-cdd-profiles postinst script that fails to find optional profiles that not listed in simple-cdd/profiles because they not fetched from HTTP during install.

See serhepopovych/simple-cdd-ext#1 for more info. This issue exists here only to refer to change that updates simple-cdd-ext submodules for all Debian releases.

serhepopovych commented 4 years ago

When installing from local media this causes to apply profiles that not listed in simple-cdd/profiles which is broken too.

In this context fixing in simple-cdd-ext by fetching all packages listed in simple-cdd.templates isn't correct. Instead we should introduce new mechanism to construct dependencies from simple-cdd/profiles variable.

serhepopovych commented 4 years ago

Preliminary fix for this issue available with commit 6c1ffea50348 ("simple-cdd-meta/debian: Fix install if user modifies simple-cdd/profiles").

However there is still room for improvements that should be added as part of this issue:

  1. Rename distro/debian.conf to debian/distro.conf and add distro,release among with default profiles to simple-cdd/profiles explicitly. These profiles are special and should always be present in profiles list.
  2. Respect profiles ordering. For example if we add virtual,minimal,text,net then they should appear in simple-cdd/profiles value as they included (sourced) within minimal profile include_deps() callback.
serhepopovych commented 4 years ago

Mentioned parts added to master branch and available since commit 305f81930ad9 ("simple-cdd-meta: Update simple-cdd-ext submodule pointer to 00a7e2c3ba20").