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: (1 comment) .................................................... File src/ovirtcli/format/text.py Line 152: # Ordering the fields for listing brick memory detail Line 153: if isinstance(resource,params.GlusterMemoryPool): Line 154: sort_strategy = ['id','name', 'hot_count','cold_count', Line 155: 'padded_size','alloc_count','max_alloc', Line 156: 'pool_misses','max_stdalloc','description'] Can I get some suggestions on how this issue can be resolved. Line 157: fields = self.sort_fields(self._get_fields(resource), sort_strategy) Line 158: width0 = width Line 159: Line 160: if width0 == -1: -- 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: Yes _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
