From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.1 (2015-04-28) on sa.local.altlinux.org X-Spam-Level: X-Spam-Status: No, score=-1.7 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS autolearn=no autolearn_force=no version=3.4.1 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=sender:date:from:to:subject:message-id:references:mime-version :content-disposition:content-transfer-encoding:in-reply-to; bh=ngKLr2aWh3ySwS6uTkUV5jPRRX6vUr9yIsQRq65sB3M=; b=TuHqPORi8Sdp46GNsMS0QOdDNsnZLr0byDmZkUBMtxEDUs3R44p3I8V0tbJuAdfIQx +wX4ABXu6XlcGjTsVJBMtEfWzqMk4tW0+DrGdm9iYckFl91u2H1roIpwbtTWVWveDan5 hztVOjhrfBZDqbhC5AAyp+iO2rzSK1aYH+xXG6RJlog4PEMQGUORZNGvRNPHmW6aUqL3 XicLhXQwafEdaluUDu9rB5LJpM5XViLXlXsAyZJ4EByiOLHARJMYHBk66QBnVE9oUMK+ 3bSnNaMwDNj7USl+B8SQ3djSYc2OBNTO7RClgcOABvk/VdEvWQ52pPv0l7Hg1Ncvm5xU mOrg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:sender:date:from:to:subject:message-id :references:mime-version:content-disposition :content-transfer-encoding:in-reply-to; bh=ngKLr2aWh3ySwS6uTkUV5jPRRX6vUr9yIsQRq65sB3M=; b=Ht7elJgE+H2qBZmkH7XJjqH/eatlYqiSEplcXs8DZAzIxEQm4htAGrBP92ZUaVSL45 eIdVbMJ2f4QwufRlV6H1R+LRtAMe2zSBPxnglY0ijEfVfBtJRgar6Ww5bZ1F4/jrBFDb UJHHAZp/UsOeAYVkGzzcy3RRKKeVLBSzfChSAO5U3JovKX8FUzKpayrz7MDF6rvYAJk0 gNEQ3mxEodPmlEu2X3sZiiGeNWDJm3KpyrJJdHnyzjm1jCoBGAXJAe7r4/15698pFmQf 1jCUcgxgp/f/iDztG7zbciKoQKzUWQKZDqOCOliISMsnzG1Eutt4xV5zJ3UZvvXiCHNu BJqw== X-Gm-Message-State: APjAAAWtpQGHX42g2eIJoBuUJu1PIj4pgZ/RRgPEaiypLIWsnY6socMh r1xzGDJk9+FNjlY+v48z5kmXJ4aA X-Google-Smtp-Source: APXvYqzzsoE5BpPV7xIkHEI9wBUZ0p0zLNxbhB8ihi48J4d0AtlYLpw7vqoaaC7gkhJx9OkJKuTHCg== X-Received: by 2002:a2e:8416:: with SMTP id z22mr18612124ljg.162.1575961612529; Mon, 09 Dec 2019 23:06:52 -0800 (PST) Sender: "Ivan A. Melnikov" Date: Tue, 10 Dec 2019 11:06:49 +0400 From: "Ivan A. Melnikov" To: Distributions development Message-ID: <20191210070649.gmjw5ueiykuuq363@titan.localdomain> References: <20191206131223.tzqyuxmdc6m36z4q@titan.localdomain> <20191206172146.783cb0cd@sem.office.basealt.ru> <5e619d0b-3842-e24a-44a2-b65f7cd95b0c@ya.ru> <20191209171954.1ea5bc5e@sem.office.basealt.ru> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20191209171954.1ea5bc5e@sem.office.basealt.ru> Subject: Re: [devel-distro] =?utf-8?q?installer-feature-*_=D0=B8_rootfs?= X-BeenThere: devel-distro@lists.altlinux.org X-Mailman-Version: 2.1.12 Precedence: list Reply-To: Distributions development List-Id: Distributions development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 10 Dec 2019 07:06:54 -0000 Archived-At: List-Archive: On Mon, Dec 09, 2019 at 05:19:54PM +0300, Mikhail Efremov wrote: > On Fri, 6 Dec 2019 23:21:40 +0700 Антон Мидюков wrote: > > 06.12.2019 21:48, Антон Мидюков пишет: > > > 06.12.2019 21:21, Mikhail Efremov пишет: > > >> On Fri, 6 Dec 2019 17:12:23 +0400 Ivan A. Melnikov wrote: > > >> [...] > > >>> Как думаете, не стоит ли включить такой хук непосредственно > > >>> в alterator-setup, а не в отдельный пакет? > > >> Лучше отдельный пакет, см. выше. Хуки в installer-feature-* писались > > >> для использования в инсталляторе и могут работать неправильно при таком > > >> нештатном использовании. Т.е. installer-features и > > >> rootfs-installer-features - это грязные хаки, нельзя бездумно > > >> использовать с ними произвольную installer-feature. Всегда нужно > > >> внимательно эту installer-feature прочитать и убедиться, что ее можно > > >> так использовать. > > > > > > Я думаю, что не надо включать в собираемый rootfs installer-features, > > > которые не предназначены для этой rootfs. > > И также отдельно включать {rootfs,livecd}-installer-features явно и > только если хорошо знаешь, что делаешь. Пусть живет отдельно, иначе > может создаться впечатление, что это штатная вещь, а не грязный хак, > которым это на самом деле является. > > > > Ограничить типом собираемой цели в mkimage-profiles не проблема. > > > > > Мысль о бездумном использовании правильная. Давайте введём новую > > категорию features-common, которые подходят и для installer, и для > > livecd-install, и для alterator-setup. Поместим их скрипты, например, в > > /usr/share/install-common/. Научим выполнять оттуда скрипты всех троих. > > А затем останется постепенно выделить общие фичи, проверив, что они > > подходят всем троим. Тогда точно никому ничего не поломаем. А в названии > > feature укажем -common. Как идея? > > Вот только проблемы хотя бы с тем, какие стадии вообще есть и когда > какие хуки должны запускаться. В alterator-setup вообще нет установки > пакетов, например. Поэтому не очевидно когда должны запускаться > preinstall-хуки. И это далеко не единственное различие. > Нужно продумать какие нужны хуки, в какой момент они должны > запускаться, какие аргументы/переменные им должны передаваться и как > они будут называться. И все это должно быть более-менее согласованно для > installer, livecd-install и alterator-setup. Я просмотрел то, что используется в simply; там что-то применимое для rootfs происходит только в postinstall. Так что я оставил запуск только postinstall хуков и в таком виде отправлю это в Сизиф, p9 и применю. > > Не такая уж и большая работа, как мне кажется. Но результат будет не > > здесь и не сейчас, конечно. > > Я бы предпочел сначала хорошенько все продумать прежде чем что-то > делать. Иначе нагородим еще один жуткий набор костылей. Мне кажется, что по большому счёту существующих костылей вполне достатночно, нужно просто ими аккуратнee воспользоваться. Из того, что я увидел, большая часть всего очень специфична для installer'а и не требуется или должна делаться совсем по-другому для livecd и rootfs. Включение сервисов и настройки control логичнее мне кажется превести на фичи m-p, там для этого есть более логичные и переносимые механизмы (например, installer-feature-sudo-enable-by-default или installer-feature-nfs*). Оставшиеся немногие вещи (вроде подстановки первого пользователя в lightdm) мне кажется можно перетащить хоть в /etc/firsttime.d (если на она выполняется до alterator-setup, это надо в любом случае поправить). -- wbr, iv m.