| Summary: | [FEAT] peer status should show all servers including current server | ||
|---|---|---|---|
| Product: | [Community] GlusterFS | Reporter: | Jacob Shucart <jacob> |
| Component: | cli | Assignee: | Niels de Vos <ndevos> |
| Status: | CLOSED CURRENTRELEASE | QA Contact: | |
| Severity: | low | Docs Contact: | |
| Priority: | medium | ||
| Version: | mainline | CC: | ahecox, amarts, ndevos, rfortier, vraman |
| Target Milestone: | --- | Keywords: | FutureFeature, Patch |
| Target Release: | --- | ||
| Hardware: | x86_64 | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | glusterfs-3.4.0 | Doc Type: | Enhancement |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2013-07-24 17:19:33 UTC | Type: | --- |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Bug Depends On: | |||
| Bug Blocks: | 870940 | ||
|
Description
Jacob Shucart
2011-05-16 17:28:16 UTC
peer status is intended to display information about peers. Since the local server is not a peer to itself, I don't see why it should be listed in peer status. Do you want an alternate command which describes the trusted storage pool? Yes, we need a command that describes that. Any priority on this? Also, what should be the new cli? I think of "bash# gluster cloud info" or "bash# gluster pool info" as the new cli and it just prints one peer (and also local) info per line.. bash# gluster pool info SERVER UUID STATUS server1 <uuid1> Online server2 <uuid2> Offline server3 <uuid3> Online server4 <uuid4> <Some-other-status-if-any> (In reply to comment #3) > Any priority on this? Also, what should be the new cli? > > I think of > > "bash# gluster cloud info" or "bash# gluster pool info" as the new cli and it > just prints one peer (and also local) info per line.. > > bash# gluster pool info > SERVER UUID STATUS > server1 <uuid1> Online > server2 <uuid2> Offline > server3 <uuid3> Online > server4 <uuid4> <Some-other-status-if-any> Dave, Can you please comment on how we would want this command to be structured? Thanks, Vijay since you can't actually create a volume using "localhost", I think even the localhost system should be considered a peer. My preference would be to just add the local address to peer status and not cretae a whole new command. Planing to keep 3.4.x branch as "internal enhancements" release without any features. So moving these bugs to 3.4.0 target milestone. *** Bug 768749 has been marked as a duplicate of this bug. *** We are thinking 'gluster cluster list' as the command which does script friendly, one line output with 'UUID' of glusterd process being the first column. *** Bug 870940 has been marked as a duplicate of this bug. *** REVIEW: http://review.gluster.org/4862 (cli: add a command 'gluster pool list-nodes [--xml]') posted (#1) for review on master by Niels de Vos (ndevos) REVIEW: http://review.gluster.org/4862 (cli: add a command 'gluster pool list [--xml]') posted (#2) for review on master by Niels de Vos (ndevos) REVIEW: http://review.gluster.org/4862 (cli: add a command 'gluster pool list [--xml]') posted (#3) for review on master by Niels de Vos (ndevos) COMMIT: http://review.gluster.org/4862 committed in master by Vijay Bellur (vbellur) ------ commit e6b6d1235daa483384d2b0701561b8f3a85be583 Author: Niels de Vos <ndevos> Date: Sat Apr 20 16:55:15 2013 +0200 cli: add a command 'gluster pool list [--xml]' * unlike 'gluster peer status', which lists only info about peers, this command lists localhost also in the list, so the sorted output from all the nodes should match. * made the output script friendly by keeping it one output per line. Change-Id: I853656753b35c617debbcceecbb71c8d6dd3c334 BUG: 764638 Original-review: http://review.gluster.org/4221 Original-author: Amar Tumballi <amarts> Signed-off-by: Niels de Vos <ndevos> Reviewed-on: http://review.gluster.org/4862 Tested-by: Gluster Build System <jenkins.com> Reviewed-by: Vijay Bellur <vbellur> |