Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

integration with OneData #75

Open
2 of 6 tasks
TomasKulhanek opened this issue Aug 8, 2018 · 8 comments
Open
2 of 6 tasks

integration with OneData #75

TomasKulhanek opened this issue Aug 8, 2018 · 8 comments
Assignees

Comments

@TomasKulhanek
Copy link
Member

TomasKulhanek commented Aug 8, 2018

  • backend support, DONE
  • frontend support, DONE
  • test, optionally unit test, asked for storage on https://onezone.cloud.cnaf.infn.it
  • add documentation, user, developer
  • submit to production - portal.west-life.eu
  • VF_ALLOW_ONEDATA env. variable will enable OnedataProvider - By default disabled as onedata have not released stable version yet
@pandreetto
Copy link
Contributor

The oneclient is a requirement for the integration.
Since there are several compatibility problems between oneclient and oneprovider, it is strongly reccomended to install the same release of the service:

yum -y localinstall http://packages.onedata.org/yum/centos/7x/x86_64/oneclient-18.02.0.rc8-1.el7.centos.x86_64.rpm

@TomasKulhanek TomasKulhanek self-assigned this Sep 12, 2018
@TomasKulhanek
Copy link
Member Author

  • I commented AdjProvInfo, alias defined by user interface, or by intermittent API
  • oneclient process seems to timeout, nothing in mounted directory, no logs produced

@pandreetto
Copy link
Contributor

I'm preparing a new PR for handling the alias parameter.
Concerning the timeout of the oneclient I've got to investigate. Please remember that you've got to install the same version of the onedata suite both on server and client side.
The problem that I have now is the TLS support and the connections to the Oneprovider.
The callback that should verify (or in our case to ignore) the Oneprovider certificate is not called anymore and I don't know why. Any query to the Oneprovider API service fails due to TLS verification error.

@TomasKulhanek
Copy link
Member Author

TomasKulhanek commented Sep 13, 2018

I force alias to be defined by user or to be generated before it's delegated to OnedataProvider. I recommend not to use AdjProvInfo as it doesn't match concept of other provider implementation. Or do you have good reason to keep it and override the alias value?

As this becomes more serious than I expected. I'd recommend

  1. either to give up the integration effort for the following reasons stated already above
  • dependency on third party libraries which are not stable enough
  • client and server seems to be tightly coupled by exact version number, it's onerous effort to maintain several 'oneclient' version on VF instance to match possible different version of 'oneserver' by different providers.
  1. or limit the supported onedata to exact version installed on the server and limit the usage on INFN instance - thus givin choice of hostname to only 'https://onezone.cloud.cnaf.infn.it'

@pandreetto
Copy link
Contributor

No problem with AdjProvInfo, I'm going to remove it.
Onedata is not yet stable enough, we know it, that's the reason why we consider the integration with VF as a prototype, at least for the moment. The integration will be "production ready" only when a final release of Onedata is available, and we will support only the stable release.
Can we devise a way to disable the plugin and re-activate it only if an env. variable, or something like that, is defined?

@TomasKulhanek
Copy link
Member Author

Thanks,

  1. AdjProvInfo - I already commited to master branch commented version - no action needed.

  2. I have VF_ALLOW_FILESYSTEM, I will introduce VF_ALLOW_ONEDATA and put it into ProviderFactory. No changes need to be done in other classes.

TomasKulhanek pushed a commit that referenced this issue Sep 13, 2018
@marcoverl
Copy link

Hi there,
could we anyway go ahead with the option 2 ( limit the supported onedata to exact version installed on the server and limit the usage on INFN instance - thus giving choice of hostname to only 'https://onezone.cloud.cnaf.infn.it') for the implementation in portal.west-life.eu, even with a disclaimer that is a experimental feature?

@pandreetto
Copy link
Contributor

The problem I have is described here
In the thread there's an interesting link to stackoverflow.
If I'm not able to fix the problem I will call curl from the plugin, so we can handle the trusted certificates via system truststores.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants