Bug 1257880 - cli/quota: enhance CLI and quotad communication
Summary: cli/quota: enhance CLI and quotad communication
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: GlusterFS
Classification: Community
Component: cli
Version: mainline
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Vijaikumar Mallikarjuna
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-08-28 10:05 UTC by Vijaikumar Mallikarjuna
Modified: 2016-05-11 22:48 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-02-15 06:48:36 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Description Vijaikumar Mallikarjuna 2015-08-28 10:05:25 UTC
In the current CLI framework CLI can hang when executing 'gluster volume quota volname list' if quotad is not running.
Bug# 1242819 has more information on this issue.

This is fixed with temporary solution: http://review.gluster.org/#/c/11990/


The temporary solution should be reverted once the CLI quotad communicated is enhanced in the framework

Below is the review comments from the patch# 11990

> Krishnan Parthasarathi		Aug 26 2:27 PM
>
> You could fix this by mimicking how CLI process ensures that it is connected to glusterd before sending RPCs. Note,
> this approach is not completely safe from a disconnect when an RPC is in flight.
> The time window where this could happen is small and we haven't seen it yet.
> The reason I suggest we do it this way is uniformity. Connections that CLI make should use the same pattern (read structure of code) to handle similar services.
> It gives us opportunity to fix issues that remain in the pattern once and reap benefits across problem areas.
>
> ## Implementation details
>
>  * See cli_cmd_needs_connection(), which returns a boolean depending on the cmd, e.g volume-start.
>
>  * See how cli_cmd_process uses the above function and state->await_connected to ensure that CLI is connected to glusterd before issuing any RPCs.

Comment 1 Vijaikumar Mallikarjuna 2016-02-15 06:48:36 UTC
AS of now with the current CLI framework we will not be able to fix the problem. For now we will go with the current fix of using a separate thread to broadcast result: http://review.gluster.org/#/c/11990/


Note You need to log in before you can comment on or make changes to this bug.