[Engine-devel] [vdsm] Adding vdsm_api support for gluster vdsm verbs

Aravinda avishwan at redhat.com
Wed Apr 17 12:49:13 UTC 2013


[Adding Saggi, Adam, Dan, Sahina]

On 04/16/2013 02:13 PM, Aravinda wrote:
> [Adding engine-devel]
>
> On 04/16/2013 02:10 PM, Aravinda wrote:
>>
>> vdsm/gluster is vdsm plugin for gluster related functionality. These 
>> functionalities are available only when vdsm-gluster package is 
>> installed. So the schema JSON of vdsm-gluster cannot be added to the 
>> same file(vdsm_api/vdsmapi-schema.json)
>>
>> Looks like vdsm_api is not providing plugin support. This patch adds 
>> functionality to vdsm_api to read vdsmapi-gluster-schema.json if 
>> available. But with this approach we need to edit the core vdsmapi.py 
>> file.
>>
>> http://gerrit.ovirt.org/#/c/13921/
>>
>> Alternate approach:
>> We can have "vdsm_api/plugins" or "vdsm_api/schema" directory inside 
>> vdsm_api, so that we can modify vdsmapi.py to read all schema files 
>> from that dir. When vdsm-gluster package installed, it copies 
>> vdsmapi-gluster-schema.json into schema directory.
>>
>>
>> -- 
>> regards
>> Aravinda
>>
>>
>> On 04/15/2013 04:14 PM, Aravinda wrote:
>>> Hi,
>>>
>>> We are trying to add json rpc support for vdsm gluster verbs. I 
>>> submitted a patch to read gluster verbs schema from vdsm/gluster 
>>> directory.
>>> http://gerrit.ovirt.org/#/c/13921/
>>>
>>> Let me know if the approach is fine.
>>>
>>> -- 
>>> regards
>>> Aravinda
>>> _______________________________________________
>>> vdsm-devel mailing list
>>> vdsm-devel at lists.fedorahosted.org
>>> https://lists.fedorahosted.org/mailman/listinfo/vdsm-devel
>>
>> _______________________________________________
>> vdsm-devel mailing list
>> vdsm-devel at lists.fedorahosted.org
>> https://lists.fedorahosted.org/mailman/listinfo/vdsm-devel
>
> _______________________________________________
> Engine-devel mailing list
> Engine-devel at ovirt.org
> http://lists.ovirt.org/mailman/listinfo/engine-devel




More information about the Engine-devel mailing list