On Wed, Jul 2, 2014 at 4:49 AM, Aline Manera <alinefm(a)linux.vnet.ibm.com> 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
> 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?
13.1 and SLE12 have virtio modules.