Bug 1491241 - RFE: Migrate Fedora server install class into the Anaconda source repository
Summary: RFE: Migrate Fedora server install class into the Anaconda source repository
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: fedora-productimg-server
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Stephen Gallagher
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 1466967
Blocks:
TreeView+ depends on / blocked
 
Reported: 2017-09-13 11:39 UTC by Radek Vykydal
Modified: 2018-09-20 14:53 UTC (History)
9 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of: 1466967
Environment:
Last Closed: 2018-09-20 14:53:46 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Bugzilla 1513024 0 unspecified CLOSED Changing blivet imports in the install class 2021-02-22 00:41:40 UTC

Internal Links: 1513024

Description Radek Vykydal 2017-09-13 11:39:08 UTC
+++ This bug was initially created as a clone of Bug #1466967 +++

The Feodra server install class is currently hosted separately[0] from the Anaconda source code and is added via an install class during the installation.

As the Anaconda now contains even non Fedora/RHEL install classes (such as for CentOS[1] or Scientific Linux[2]) I think it no longer makes sense to host the Fedora Server install class separately.

Having install class together with the other install classes should make maintenance, debugging and testing easier. This might also help prevent or at least make easier to spot Install Class related issues such as for example bug 1466964.

As for making sure the Fedora Server is used during Fedora server compsoe generation (which has been raised during the original discussion in bug 1466964) there is now an Anaconda pull request[3] under review that adds support for selecting install class via kickstart which seems like an ideal fit for this.


[0] http://pkgs.fedoraproject.org/cgit/rpms/fedora-productimg-server.git/tree/fedora-server.py
[1] https://github.com/rhinstaller/anaconda/blob/master/pyanaconda/installclasses/centos.py
[2] https://github.com/rhinstaller/anaconda/blob/master/pyanaconda/installclasses/scientific.py
[3] https://github.com/rhinstaller/anaconda/pull/1120

--- Additional comment from Martin Kolman on 2017-06-30 19:49:19 EDT ---

A slight correction - the bug number in description is unfortunately wrong - bug 1463297 is the correct one.

--- Additional comment from Vendula Poncova on 2017-07-03 05:06:36 EDT ---

The Fedora Workstation install class could be moved as well.

--- Additional comment from Martin Kolman on 2017-09-12 07:12:24 EDT ---

(In reply to Vendula Poncova from comment #2)
> The Fedora Workstation install class could be moved as well.
Good point!

Seems to be located here:
https://src.fedoraproject.org/rpms/fedora-productimg-workstation/blob/master/f/fedora-workstation.py

--- Additional comment from Stephen Gallagher on 2017-09-12 07:52:25 EDT ---

Martin, is this something the Anaconda team is going to take care of, or do you need something from me?

--- Additional comment from Stephen Gallagher on 2017-09-12 07:52:26 EDT ---

Martin, is this something the Anaconda team is going to take care of, or do you need something from me?

--- Additional comment from Martin Kolman on 2017-09-12 08:23:03 EDT ---

(In reply to Stephen Gallagher from comment #4)
> Martin, is this something the Anaconda team is going to take care of, or do
> you need something from me?

We can handle the merging of the install class to Anaconda codebase ourselves, but coordination with you will certainly be needed for Server to switch to using it.

BTW, this is the final form of the install class selection API I've mentioned in the description:
http://anaconda-installer.readthedocs.io/en/latest/kickstart.html#installclass

--- Additional comment from Radek Vykydal on 2017-09-12 08:47:30 EDT ---

Stephen, could you point me to kickstarts for composing the Server installer (or logs)? I am not able to find using of fedora-productimg-server among fedora-kickstarts.

I'd even think the current building of server installer composes would keep working after moving the installclass to anaconda tree, without any need for specifying install class in kickstart (which could be a good thing to do nonetheless). I'd think choosing the right class is independent of the fedora-productimg-server package, rather it is set by lorax --version or --product options.

--- Additional comment from Stephen Gallagher on 2017-09-12 08:56:24 EDT ---

(In reply to Radek Vykydal from comment #7)
> Stephen, could you point me to kickstarts for composing the Server installer
> (or logs)? I am not able to find using of fedora-productimg-server among
> fedora-kickstarts.
> 
> I'd even think the current building of server installer composes would keep
> working after moving the installclass to anaconda tree, without any need for
> specifying install class in kickstart (which could be a good thing to do
> nonetheless). I'd think choosing the right class is independent of the
> fedora-productimg-server package, rather it is set by lorax --version or
> --product options.

https://pagure.io/pungi-fedora/blob/master/f/variants-fedora.xml#_92

--- Additional comment from Radek Vykydal on 2017-09-12 09:29:12 EDT ---

Thank you, BTW I guess the place where the install class is set (in compose process) is here: https://pagure.io/pungi-fedora/blob/master/f/fedora.conf#_432.

--- Additional comment from Radek Vykydal on 2017-09-13 07:38:04 EDT ---

https://github.com/rhinstaller/anaconda/pull/1185
When this change is applied in Anaconda we can remove the installclass from fedora-productimg-server. I am going to clone a BZ to track that.

Comment 1 Radek Vykydal 2017-09-13 11:40:24 UTC
(In reply to Radek Vykydal from comment #0)
 
> --- Additional comment from Radek Vykydal on 2017-09-13 07:38:04 EDT ---
> 
> https://github.com/rhinstaller/anaconda/pull/1185
> When this change is applied in Anaconda we can remove the installclass from
> fedora-productimg-server. I am going to clone a BZ to track that.

Comment 2 Radek Vykydal 2017-11-15 13:17:51 UTC
The installclass will be added to anaconda package in anaconda-28.11. When the package is built it can be removed from fedora-productimg-server. I'll let you know then.

Comment 3 Stephen Gallagher 2017-11-28 19:03:48 UTC
Working on it now.


Note You need to log in before you can comment on or make changes to this bug.