ndarshan has posted comments on this change.

Change subject: cli: Correct ordering of fields for "list bricks" with 
all-content as true(#990009)
......................................................................


Patch Set 2:

This is rest-api output and it is in required order:


        <memory_pools>
            <memory_pool>
                <name>v2-server:fd_t</name>
                <hot_count>0</hot_count>
                <cold_count>1024</cold_count>
                <padded_size>108</padded_size>
                <alloc_count>0</alloc_count>
                <max_alloc>0</max_alloc>
                <pool_misses>0</pool_misses>
                <max_stdalloc>0</max_stdalloc>
            </memory_pool>
            <memory_pool>
                <name>v2-server:dentry_t</name>
                <hot_count>0</hot_count>
                <cold_count>16384</cold_count>
                <padded_size>84</padded_size>
                <alloc_count>0</alloc_count>
                <max_alloc>0</max_alloc>
                <pool_misses>0</pool_misses>
                <max_stdalloc>0</max_stdalloc>
            </memory_pool>


This is cli output:


memory_pools-memory_pool-alloc_count        : 0
memory_pools-memory_pool-name               : v2-server:fd_t
memory_pools-memory_pool-cold_count         : 1024
memory_pools-memory_pool-hot_count          : 0
memory_pools-memory_pool-max_alloc          : 0
memory_pools-memory_pool-max_stdalloc       : 0
memory_pools-memory_pool-padded_size        : 108
memory_pools-memory_pool-pool_misses        : 0
memory_pools-memory_pool-alloc_count        : 0
memory_pools-memory_pool-name               : v2-server:dentry_t
memory_pools-memory_pool-cold_count         : 16384
memory_pools-memory_pool-hot_count          : 0
memory_pools-memory_pool-max_alloc          : 0
memory_pools-memory_pool-max_stdalloc       : 0
memory_pools-memory_pool-padded_size        : 84
memory_pools-memory_pool-pool_misses        : 0

-- 
To view, visit http://gerrit.ovirt.org/19203
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I974bde6937992b391f01accd7977b9dc0e5a5ef7
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine-cli
Gerrit-Branch: master
Gerrit-Owner: ndarshan <dnara...@redhat.com>
Gerrit-Reviewer: Aravinda VK <avish...@redhat.com>
Gerrit-Reviewer: Michael Pasternak <mpast...@redhat.com>
Gerrit-Reviewer: Ravi Nori <rn...@redhat.com>
Gerrit-Reviewer: ndarshan <dnara...@redhat.com>
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to