airavata icon indicating copy to clipboard operation
airavata copied to clipboard
verified publisher icon apache

Implementation of WebDAV Adapter for Nextcloud Integration

Open karankotz opened this issue 7 years ago • 4 comments

This pull request is for the addition of the WebDAV adapter in the apache airavata to support the integration of the nextcloud. The first part changes are made to support the data movement to the nextcloud via WebDAV adapter in the airavata api-orchestrator.

The following changes are made -

  • Addition of the new data movement protocol in data_movement_model of the resource_catalog_models.

  • Implemented new data movement model WebDAVDataMovement in the airavata-data-models

  • Added the following methods to the airavata-api and registry-api (specifically in airavataserverhandler and registryserverhandler) --addWebDAVDataMovementDetails(..) --updateWebDAVDataMovementDetails(..) --getWebDAVDataMovement(..)

  • Implemented the following methods for the ComputeResource interface. --addWebDAVDataMovementDetails(..) --getWebDAVDataMovement(..)

  • Made changes in the AppCatalogJPAUtils, AppCatalogResourceType, AppCatalogThriftConversion to support the WebDAV adapter in the airavata-registry-core.

  • Added the WebDavDataMovementResource in the app catalog resources of airavata-registry-core.

  • Added the WebDavDataMovement model in the app catalog model of the airavata-registry-core.

  • Created the sample test case addWebDavStorageResourceTest() in the StorageResourceAppCatalogTest class of the airavata-registry-core.

  • The further changes are yet to be updated in the helix module with the WebDAV adapter (Added in the commit f15dfc2f288079c37c32e238a2a9d23635078378).

  • The helix module has been added with the WebDAV adaptor. The WebDAV adaptor is used to connect to the nextcloud instance and upload the output files generated by the compute resource to the nextcloud server.

karankotz avatar Aug 04 '18 21:08 karankotz

Looks good but the registry changes should be in the registry-refactoring module instead of registry-core.

machristie avatar Aug 06 '18 14:08 machristie

@machristie There is no registry-refactoring module, this branch was built from the staging branch, do you have any suggestions to achieve this?

karankotz avatar Aug 06 '18 19:08 karankotz

If this branch were based on develop then it would have the registry-refactoring module. Is there a reason it is based on staging?

machristie avatar Aug 07 '18 13:08 machristie

@machristie As discussed these changes will currently stay on this branch and will need be merged later with the registry-refactoring module of the develop branch.

karankotz avatar Aug 11 '18 18:08 karankotz