Re: xlator option setting in gfapi

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

This is what happens when glfs_set_xlator_option is called:
- It updates ctx->cmd_args->xlator_options
- gf_add_cmdline_options (during glusterfs_graph_prepare.. pub_glfs_init) adds it to xl->options
- glusterfs_graph_unknown_options (during glusterfs_graph_activate.. pub_glfs_init) checks if the options in xl->options are present or not. If the option is not present it just logs a warning message. These options are treated as command line args.

To report the option as a failure, at the least we can fail glfs_init()

Regards,
Poornima

----- Original Message -----
> From: "Raghavendra Gowdappa" <rgowdapp@xxxxxxxxxx>
> To: "Poornima Gurusiddaiah" <pgurusid@xxxxxxxxxx>, "Raghavendra Talur" <rtalur@xxxxxxxxxx>
> Cc: "Gluster Devel" <gluster-devel@xxxxxxxxxxx>, "Jeff Cody" <jcody@xxxxxxxxxx>, "Rajesh Joseph" <rjoseph@xxxxxxxxxx>
> Sent: Monday, May 9, 2016 8:45:56 AM
> Subject: xlator option setting in gfapi
> 
> Hi Poornima/Raghavendra,
> 
> This mail is an initiation of a thread to discuss how to make xlator options
> setting in gfapi synchronous (so, that caller will know the result) or
> providing a cbk to let the application know about the status.
> 
> My very naive attempt of code-reading showed me that
> pub_glfs_set_xlator_option just adds the option to cmd_args.xlator_options.
> Can you please explain how/when these values are communicated to glusterd to
> change volfile? From there we can carry forward the conversation.
> 
> regards,
> Raghavendra
> 
_______________________________________________
Gluster-devel mailing list
Gluster-devel@xxxxxxxxxxx
http://www.gluster.org/mailman/listinfo/gluster-devel



[Index of Archives]     [Gluster Users]     [Ceph Users]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux