Welcome to Kimchi, Dinar! :-) It is always a pleasure to see new faces around here. My comments below: On 06/30/2014 01:40 PM, Dinar valeev wrote: > From: Dinar Valeev <dvaleev(a)suse.com> Add SLES 12 information and set openSUSE's version to 13.1 > Signed-off-by: Dinar Valeev <dvaleev(a)suse.com> --- > src/kimchi/isoinfo.py | 1 + > src/kimchi/osinfo.py | 9 ++++++--- > 2 files changed, 7 insertions(+), 3 deletions(-) > diff --git a/src/kimchi/isoinfo.py b/src/kimchi/isoinfo.py > index b7315e0..c394a32 100644 > --- a/src/kimchi/isoinfo.py > +++ b/src/kimchi/isoinfo.py > @@ -95,6 +95,7 @@ iso_dir = [ > '|HRM_CENA_X64CHKV|HRM_CPRA_X64FREV|HRM_CPRNA_X64FREV')), > ('sles', '10', 'SLES10|SUSE-Linux-Enterprise-Server.001'), > ('sles', '11', 'SUSE_SLES-11-0-0'), > + ('sles', '12', 'SLE-12'), > ('sles', lambda m: "11sp%s" % m.group(1), 'SLES-11-SP(\d+)'), > ('opensuse', lambda m: m.group(1), 'openSUSE[ -](\d+\.\d+)'), > ('opensuse', '11.1', 'SU1110.001'), > diff --git a/src/kimchi/osinfo.py b/src/kimchi/osinfo.py > index 093feca..9e8b62e 100644 > --- a/src/kimchi/osinfo.py > +++ b/src/kimchi/osinfo.py > @@ -57,10 +57,13 @@ template_specs = {'x86': {'old': dict(common_spec, > disk_bus='ide', > modern_version_bases = {'x86': {'debian': '6.0', 'ubuntu': '7.10', > - 'opensuse': '10.3', 'centos': '5.3', > - 'rhel': '6.0', 'fedora': '16', 'gentoo': > '0'}, > + 'opensuse': '13.1', 'centos': '5.3', > + 'rhel': '6.0', 'fedora': '16', 'gentoo': > '0', > + 'sles': '12'}, > 'power': {'rhel': '7.0', 'fedora': '19', > - 'ubuntu': '14.04'}} > + 'ubuntu': '14.04', > + 'opensuse': '13.1', > + 'sles': '12'}} > icon_available_distros = [icon[5:-4] for icon in glob.glob1('%s/images/' > % paths.ui_dir, 'icon-*.png')] The modern_version_bases dict tells Kimchi from whichi OS version it can use virtio bus. You can see more details in src/kimchi/osinfo.py:116 if distro in modern_version_bases[arch]: if LooseVersion(version) >= LooseVersion( modern_version_bases[arch][distro]): params.update(template_specs[arch]['modern']) else: params.update(template_specs[arch]['old']) So your patch should not change the opensuse version for x86

And for power, you need to add the lowest version that supports virtio bus. I know Ubuntu 14.04 is the first Ubuntu version supported for Power so it is correct But is it true for opensuse 13.1 and sles 12?

On 07/02/2014 08:42 AM, Dinar Valeev wrote: On Wed, Jul 2, 2014 at 4:49 AM, Aline Manera <alinefm(a)linux.vnet.ibm.com&gt; wrote: Welcome to Kimchi, Dinar! :-) It is always a pleasure to see new faces around here. My comments below: On 06/30/2014 01:40 PM, Dinar valeev wrote: From: Dinar Valeev <dvaleev(a)suse.com&gt; Add SLES 12 information and set openSUSE's version to 13.1 Signed-off-by: Dinar Valeev <dvaleev(a)suse.com&gt; --- src/kimchi/isoinfo.py | 1 + src/kimchi/osinfo.py | 9 ++++++--- 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a/src/kimchi/isoinfo.py b/src/kimchi/isoinfo.py index b7315e0..c394a32 100644 --- a/src/kimchi/isoinfo.py +++ b/src/kimchi/isoinfo.py @@ -95,6 +95,7 @@ iso_dir = [ '|HRM_CENA_X64CHKV|HRM_CPRA_X64FREV|HRM_CPRNA_X64FREV')), ('sles', '10', 'SLES10|SUSE-Linux-Enterprise-Server.001'), ('sles', '11', 'SUSE_SLES-11-0-0'), + ('sles', '12', 'SLE-12'), ('sles', lambda m: "11sp%s" % m.group(1), 'SLES-11-SP(\d+)'), ('opensuse', lambda m: m.group(1), 'openSUSE[ -](\d+\.\d+)'), ('opensuse', '11.1', 'SU1110.001'), diff --git a/src/kimchi/osinfo.py b/src/kimchi/osinfo.py index 093feca..9e8b62e 100644 --- a/src/kimchi/osinfo.py +++ b/src/kimchi/osinfo.py @@ -57,10 +57,13 @@ template_specs = {'x86': {'old': dict(common_spec, disk_bus='ide', modern_version_bases = {'x86': {'debian': '6.0', 'ubuntu': '7.10', - 'opensuse': '10.3', 'centos': '5.3', - 'rhel': '6.0', 'fedora': '16', 'gentoo': '0'}, + 'opensuse': '13.1', 'centos': '5.3', + 'rhel': '6.0', 'fedora': '16', 'gentoo': '0', + 'sles': '12'}, 'power': {'rhel': '7.0', 'fedora': '19', - 'ubuntu': '14.04'}} + 'ubuntu': '14.04', + 'opensuse': '13.1', + 'sles': '12'}} icon_available_distros = [icon[5:-4] for icon in glob.glob1('%s/images/' % paths.ui_dir, 'icon-*.png')] The modern_version_bases dict tells Kimchi from whichi OS version it can use virtio bus. You can see more details in src/kimchi/osinfo.py:116 if distro in modern_version_bases[arch]: if LooseVersion(version) >= LooseVersion( modern_version_bases[arch][distro]): params.update(template_specs[arch]['modern']) else: params.update(template_specs[arch]['old']) So your patch should not change the opensuse version for x86 virtio modules is there, I don't see a limitation here This is your patch: - 'opensuse': '10.3', 'centos': '5.3', - 'rhel': '6.0', 'fedora': '16', 'gentoo': '0'}, + 'opensuse': '13.1', 'centos': '5.3', + 'rhel': '6.0', 'fedora': '16', 'gentoo': '0', + 'sles': '12'}, Here you are saying to Kimchi "only uses virtio on opensuse >= 13.1" instead of 10.3 But opensuse 10.3 supports virtio and you should keep as it is.

And for power, you need to add the lowest version that supports virtio bus. I know Ubuntu 14.04 is the first Ubuntu version supported for Power so it is correct But is it true for opensuse 13.1 and sles 12? 13.1 and SLE12 have virtio modules. And any other older version supports virtio? In this dict, you should add the older OS version that supports virtio