[Kimchi-devel] [PATCH V2] [Ginger-Base 0/1] Issue 728 - Processor Info in s390 architecture

[Chandra Shekhar Reddy Potula]

Aline,

You could expect patch to accommodate the UI changes from me.

Since I will be sending a patch for feature split into tabs for 
gingerbase to avoid patch conflicts between me and suresh in these files.

Regards
Chnadra

On 26/11/15 5:14 AM, Aline Manera wrote:
>
> Hi all,
>
> The plan was to separate the gingerbase code into a new git repository 
> today.
> But I want to merge this patch prior to do that, otherwise this patch 
> will not apply in the new git repository without new changes.
> So I will wait until tomorrow to get it in. I will send an 
> appropriated note when the new git repository is created.
>
> Regards,
> Aline Manera
>
> On 24/11/2015 09:59, sureshab at linux.vnet.ibm.com wrote:
>> From: Suresh Babu Angadi <sureshab at in.ibm.com>
>>
>> V1 to V2 changes:
>> Changed cpus and memory object to be same across all platforms(aline)
>> Changed API.md and lookup as per review comments(aline & abhiram)
>>
>> As per RFC mail thread -
>>       [Kimchi-devel] RFC - #728 - Processor Info in s390 architecture
>> this patch set adds functionality in back-end to add
>> architecture, and host name(for all architecture),
>> split CPUs to online and offline(x86, s390x),
>> additional virtualization details(for s390x)
>>
>> Note: Subsequent patch set for UI changes will follow
>>        to accommodate these changes
>>
>> Test Cases Executed:
>> ====================
>> 1) On x86 machine(ppc o/p will be similar to x86):
>> curl -k -u suresh -H "Content-Type: application/json" -H "Accept: 
>> application/json" -X GET 
>> 'https://127.0.0.1:8001/plugins/gingerbase/host'
>> {
>>    "os_distro":"Fedora",
>>    "cpus":{
>>      "offline":0,
>>      "online":4
>>    },
>>    "cpu_model":"Intel(R) Core(TM) i5-3320M CPU @ 2.60GHz",
>>    "os_version":"21",
>>    "host":"localhost.localdomain",
>>    "os_codename":"Twenty One",
>>    "architecture":"x86_64",
>>    "memory":{
>>      "offline":0,
>>      "online":7933902848
>>    }
>> }
>>
>> 2) On s390x machine:
>> curl -k -u root -H "Content-Type: application/json" -H "Accept: 
>> application/json" -X GET 
>> 'https://127.0.0.1:8001/plugins/gingerbase/host'
>> {
>>    "os_distro":"KVM for IBM z Systems",
>>    "cpus":{
>>      "shared":2,
>>      "offline":2,
>>      "dedicated":0,
>>      "online":2
>>    },
>>    "cpu_model":"IBM/2827/743 H43",
>>    "os_version":"1.1.1",
>>    "host":"zfwcec103",
>>    "os_codename":"Z",
>>    "architecture":"s390x",
>>    "memory":{
>>      "offline":2147483648,
>>      "online":2147483648
>>    },
>>    "virtualization":{
>>      "lpar_name":"CSTLIN1",
>>      "hypervisor":"PR/SM",
>>      "lpar_number":55,
>>      "hypervisor_vendor":"IBM"
>>    }
>> }
>>
>>
>> Suresh Babu Angadi (1):
>>    Fix for issue 728: processor info displays blank for system z
>>    this patch set also adds additional capability:
>>    retrieving architecture and host name (for all architecture)
>>    split CPUs to show online and offline cpus also
>>      includes shared and dedicated cpus for s390x
>>    split memory to show online and offline
>>    additional virtualization details(for s390x):
>>    virtualization will have hypervisor details and lpar details
>>
>>   src/wok/plugins/gingerbase/docs/API.md        |  18 +-
>>   src/wok/plugins/gingerbase/i18n.py            |   1 +
>>   src/wok/plugins/gingerbase/lscpu.py           |  60 ++++++
>>   src/wok/plugins/gingerbase/model/host.py      | 259 
>> ++++++++++++++++++++++----
>>   src/wok/plugins/gingerbase/tests/test_host.py |  12 +-
>>   5 files changed, 307 insertions(+), 43 deletions(-)
>>
>