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

EOS grpc fixes #3420

Merged
merged 29 commits into from
Nov 29, 2022
Merged

EOS grpc fixes #3420

merged 29 commits into from
Nov 29, 2022

Conversation

gmgigi96
Copy link
Member

@gmgigi96 gmgigi96 commented Oct 31, 2022

The shares and the applications were not working with EOS grpc storage driver.
This PR introduces the fixes to make them working.

@update-docs
Copy link

update-docs bot commented Oct 31, 2022

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@gmgigi96 gmgigi96 marked this pull request as ready for review November 29, 2022 11:42
@gmgigi96 gmgigi96 requested review from labkode, glpatcern and a team as code owners November 29, 2022 11:42
@gmgigi96 gmgigi96 changed the title eos grpc fixes EOS grpc fixes Nov 29, 2022
@glpatcern glpatcern merged commit 48a2870 into cs3org:master Nov 29, 2022
kobergj pushed a commit to kobergj/reva that referenced this pull request Nov 30, 2022
* return inode in fileinfo

* fix log

* fix eos grpc parsing

* stat as a root

* workaround for inode (revert me)

* set parent id in eosfs

* Revert "stat as a root"

This reverts commit 81ce7b3.

* set storage id to parent id in storage provider

* fix file path

* clean path returned by eos

* set xattrs for lock as root

* fix key when setting attribute

* stat as root

* Revert "stat as root"

This reverts commit e865e6e.

* trim prefix for user attrs

* do not filter sys attrs

* fix add acl

* fix initiate file upload reference resolution

* fix initiate file upload

* Revert "fix initiate file upload"

This reverts commit 265d091.

* set sys acl as root

* workaround for adding acl

* fix unset attr

* unset lock as root

* Avoid crash when getting checksum in a stat request

* fix

* add changelog

* fix linter

* fix license

Co-authored-by: Fabrizio Furano <furano@cern.ch>
@kobergj kobergj mentioned this pull request Nov 30, 2022
@kobergj kobergj mentioned this pull request Dec 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants