On Tue, 2019-06-04 at 10:05 +0000, Vendula Poncova wrote:
> On Tue, 2019-05-28 at 07:34 +0200, Tomas Popela wrote:
>
> Well...a couple of things.
>
> 1. Any maintenance on this will have to go through anaconda team. The
> name of the directory ('product.d') rather suggests this is a drop-in
> style thing; if so, is it possible for this config file to be shipped
> in a separate package maintained directly by Workstation folks, and
> just ensure that package is included in all scenarios where we want
> this tweak?
It wouldn't work with the current pull request, because Silverblue uses a
configuration file for Workstation Live. However, we already discuss other solutions for
Silverblue, so it shouldn't be an issue.
Anyway, we would like to maintain all product configuration files until we are done with
the Anaconda modularization. It might be necessary to do some changes across all these
files and it is easier this way. That is the reason why install classes were moved to the
Anaconda repository in the first place.
Another solution could be to just modify fedora-live-workstation.ks and write a new file
to /etc/anaconda/conf.d that will override the hidden_spokes option. Then we don't
need a special configuration file for Workstation Live. The content of the file would look
like this:
[User Interface]
hidden_spokes =
NetworkSpoke
PasswordSpoke
UserSpoke
> 2. Have we thought through all cases where the variant now being
> 'Workstation Live' rather than 'Workstation' might cause problems?
I have checked the code and the variant name is used only for loading the right product
configuration files. It was introduced quite recently in in a commit at
https://github.com/rhinstaller/anaconda/commit/6e4bfc4 for this specific purpose.
The spins don't seem to use the configuration file for Fedora Workstation, so it
shouldn't affect them.
Thanks for the explanations, that all seems reasonable to me.
--
Adam Williamson
Fedora QA Community Monkey
IRC: adamw | Twitter: AdamW_Fedora | XMPP: adamw AT happyassassin . net
http://www.happyassassin.net