Re: [Libosinfo] [osinfo-db PATCH 02/15] install-script, fedora, desktop: Add avatar-disk template

2019-06-18 Thread Fabiano Fidêncio
On Tue, Jun 18, 2019 at 9:24 PM Cole Robinson wrote: > > On 6/18/19 1:10 PM, Fabiano Fidêncio wrote: > > On Mon, Jun 17, 2019 at 11:32 PM Cole Robinson wrote: > >> > >> On 6/13/19 7:48 AM, Fabiano Fidêncio wrote: > >>> Let's add the avatar disk template, which considers that the avatar file >

Re: [Libosinfo] [osinfo-db PATCH 00/16] RE-POST: install-scripts improvements

2019-06-18 Thread Cole Robinson
On 6/18/19 3:06 PM, Fabiano Fidêncio wrote: > This patch series, together with some small bug fixes/clean-ups, tries > to improve target, script, and avatar disks in a way that apps do not > actually have to set them up. > > The changes do not break the current state as, if set, those disks are >

Re: [Libosinfo] [osinfo-db PATCH 02/15] install-script, fedora, desktop: Add avatar-disk template

2019-06-18 Thread Cole Robinson
On 6/18/19 1:10 PM, Fabiano Fidêncio wrote: > On Mon, Jun 17, 2019 at 11:32 PM Cole Robinson wrote: >> >> On 6/13/19 7:48 AM, Fabiano Fidêncio wrote: >>> Let's add the avatar disk template, which considers that the avatar file >>> is stored together with the install scripts files, so we don't

[Libosinfo] [osinfo-db PATCH 00/16] RE-POST: install-scripts improvements

2019-06-18 Thread Fabiano Fidêncio
This patch series, together with some small bug fixes/clean-ups, tries to improve target, script, and avatar disks in a way that apps do not actually have to set them up. The changes do not break the current state as, if set, those disks are going to be used. With those changes, a lot of logic

[Libosinfo] [osinfo-db PATCH 16/16] install-script, rhel, centos: Merge centos & rhel scripts

2019-06-18 Thread Fabiano Fidêncio
Splitting centos & rhel script, done in 538ca137c, was not exactly the best idea as it increased the maintenance for pretty much no gain. With that in mind, let's merge back the scripts keeping the centos specific bits conditionally added as part of the templates. Signed-off-by: Fabiano Fidêncio

[Libosinfo] [osinfo-db PATCH 08/16] install-script, opensuse: Add target-disk template

2019-06-18 Thread Fabiano Fidêncio
Let's add a target-disk template so we actually can use what apps set as target-disk, instead of using a hardcoded path to /dev/vda. With this change we keep not requiring the apps to set this value. Signed-off-by: Fabiano Fidêncio --- .../opensuse-autoyast-desktop.xml.in | 21

[Libosinfo] [osinfo-db PATCH 07/16] install-script, opensuse: Remove extra empty space

2019-06-18 Thread Fabiano Fidêncio
repo-oss template has been generating an empy extra space before the URL it returns, which may break unattended installations. Signed-off-by: Fabiano Fidêncio --- .../install-script/opensuse.org/opensuse-autoyast-desktop.xml.in | 1 -

[Libosinfo] [osinfo-db PATCH 10/16] install-script, rhel: Only use virtio for 5.6+

2019-06-18 Thread Fabiano Fidêncio
Although we only support unattended installations for Red Hat Enterprise Linux 6 (or newer), let's fix the condition for using virtio as the target disk. Unlike suggested in the previous condition, virtio-block has only been added to Red Hat Enterprise Linux 5.6 (and not 5.0). Signed-off-by:

[Libosinfo] [osinfo-db PATCH 13/16] install-script, centos: Only use virtio for 5.6+

2019-06-18 Thread Fabiano Fidêncio
Although we only support unattended installations for CentOS 6 (or newer), let's fix the condition for using virtio as the target disk. Unlike suggested in the previous condition, virtio-block has only been added to CentOS 5.6 (and not 5.0). Signed-off-by: Fabiano Fidêncio ---

[Libosinfo] [osinfo-db PATCH 14/16] install-script, centos: Improve script-disk template

2019-06-18 Thread Fabiano Fidêncio
Let's improve the script disk template so we don't actually require the apps to set it (but, if they do, the value set will be used). Signed-off-by: Fabiano Fidêncio --- .../centos.org/centos-kickstart-desktop.xml.in | 14 -- .../centos.org/centos-kickstart-jeos.xml.in|

[Libosinfo] [osinfo-db PATCH 04/16] install-script, silverblue: Add script-disk template

2019-06-18 Thread Fabiano Fidêncio
Let's add the script disk template so we don't actually require the apps to set it (but, if they do, the value set will be used). Signed-off-by: Fabiano Fidêncio --- .../silverblue-kickstart-desktop.xml.in | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff

[Libosinfo] [osinfo-db PATCH 05/16] install-script, silverblue: Add target-disk template

2019-06-18 Thread Fabiano Fidêncio
Let's add a target-disk template so we actually can use what apps set as target-disk, instead of using a hardcoded path to /dev/vda. With this change we keep not requiring the apps to set this value. Signed-off-by: Fabiano Fidêncio --- .../silverblue-kickstart-desktop.xml.in | 15

[Libosinfo] [osinfo-db PATCH 11/16] install-script, rhel: Improve script-disk template

2019-06-18 Thread Fabiano Fidêncio
Let's improve the script disk template so we don't actually require the apps to set it (but, if they do, the value set will be used). Signed-off-by: Fabiano Fidêncio --- .../redhat.com/rhel-kickstart-desktop.xml.in | 14 -- .../redhat.com/rhel-kickstart-jeos.xml.in |

[Libosinfo] [osinfo-db PATCH 03/16] install-script, fedora, desktop: Add avatar-disk template

2019-06-18 Thread Fabiano Fidêncio
Let's add the avatar disk template, which considers that the avatar file is stored together with the install scripts files, so we don't actually require the apps to set it (but, if they do, the value set will be used). Signed-off-by: Fabiano Fidêncio --- .../fedora-kickstart-desktop.xml.in

[Libosinfo] [osinfo-db PATCH 12/16] install-script, rhel, desktop: Add avatar-disk template

2019-06-18 Thread Fabiano Fidêncio
Let's add the avatar disk template, which considers that the avatar file is stored together with the install scripts files, so we don't actually require the apps to set it (but, if they do, the value set will be used). Signed-off-by: Fabiano Fidêncio ---

[Libosinfo] [osinfo-db PATCH 01/16] install-script, fedora, desktop: Drop unused bits

2019-06-18 Thread Fabiano Fidêncio
autopart usage has been introduced in commit 81beb9970 and, since then, there's no need keep the rootfs and bootfs templates. Signed-off-by: Fabiano Fidêncio --- .../fedora-kickstart-desktop.xml.in | 22 --- 1 file changed, 22 deletions(-) diff --git

[Libosinfo] [osinfo-db PATCH 15/16] install-script, centos, desktop: Add avatar-disk template

2019-06-18 Thread Fabiano Fidêncio
Let's add the avatar disk template, which considers that the avatar file is stored together with the install scripts files, so we don't actually require the apps to set it (but, if they do, the value set will be used). Signed-off-by: Fabiano Fidêncio ---

[Libosinfo] [osinfo-db PATCH 02/16] install-script, fedora: Improve script-disk template

2019-06-18 Thread Fabiano Fidêncio
Let's improve the script disk template so we don't actually require the apps to set it (but, if they do, the value set will be used). Signed-off-by: Fabiano Fidêncio --- .../fedora-kickstart-desktop.xml.in | 17 +++-- .../fedora-kickstart-jeos.xml.in| 17

[Libosinfo] [osinfo-db PATCH 06/16] install-script, ubuntu: Improve target-disk template

2019-06-18 Thread Fabiano Fidêncio
Let's use virtio as target-disk in case a target-disk is not set by the apps. We can safely do this as we support unattended installation for Ubuntu since its 12.04 release and virtio support has been added on 9.04. Signed-off-by: Fabiano Fidêncio ---

[Libosinfo] [osinfo-db PATCH 09/16] install-script, opensuse, desktop: Add avatar-disk template

2019-06-18 Thread Fabiano Fidêncio
Let's add the avatar disk template, which considers that the avatar file is stored together with the install scripts files, so we don't actually require the apps to set it (but, if they do, the value set will be used). Signed-off-by: Fabiano Fidêncio ---