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

[WIP] Add a SharesStorageProvider and an oc10 sql share manager #2232

Closed
wants to merge 8 commits into from

Conversation

aduffeck
Copy link
Contributor

@aduffeck aduffeck commented Jun 30, 2021

This is WIP to access oc10 shares from ocis.

Requires cs3org/reva#2023

\cc @butonic

@update-docs
Copy link

update-docs bot commented Jun 30, 2021

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.

@aduffeck
Copy link
Contributor Author

aduffeck commented Jun 30, 2021

@butonic please note (as discussed) that the database information is currently hardcoded in storage/pkg/command/storageshares.go

@aduffeck aduffeck force-pushed the sharestorageprovider branch from 6c127fa to 401563f Compare July 8, 2021 07:09
@aduffeck
Copy link
Contributor Author

Just for reference, the according reva draft pr can be found at cs3org/reva#1846

@aduffeck aduffeck force-pushed the sharestorageprovider branch from 401563f to 33beee7 Compare July 23, 2021 09:08
@ownclouders
Copy link
Contributor

💥 Acceptance tests Core-API-Tests-owncloud-storage-8 failed. The build is cancelled...

@ownclouders
Copy link
Contributor

💥 Acceptance tests Core-API-Tests-ocis-storage-1 failed. The build is cancelled...

@ownclouders
Copy link
Contributor

💥 Acceptance tests Core-API-Tests-owncloud-storage-3 failed. The build is cancelled...

@ownclouders
Copy link
Contributor

💥 Acceptance tests localApiTests-apiBugDemonstration-ocis failed. The build is cancelled...

@wkloucek wkloucek mentioned this pull request Aug 11, 2021
9 tasks
@wkloucek
Copy link
Contributor

@aduffeck can you make this PR ready? :-)

@ownclouders
Copy link
Contributor

💥 Acceptance tests Web-Tests-ocis-ocis-storage-2 failed. The build is cancelled...

@ownclouders
Copy link
Contributor

💥 Acceptance tests localApiTests-apiAccountsHashDifficulty-ocis failed. The build is cancelled...

@ownclouders
Copy link
Contributor

💥 Acceptance tests settingsUITests failed. The build is cancelled...

@ownclouders
Copy link
Contributor

💥 Acceptance tests Core-API-Tests-ocis-storage-7 failed. The build is cancelled...

@ownclouders
Copy link
Contributor

💥 Acceptance tests Web-Tests-ocis-ocis-storage-1 failed. The build is cancelled...

@ownclouders
Copy link
Contributor

💥 Acceptance tests Core-API-Tests-ocis-storage-3 failed. The build is cancelled...

@ownclouders
Copy link
Contributor

💥 Acceptance tests Core-API-Tests-ocis-storage-1 failed. The build is cancelled...

@ownclouders
Copy link
Contributor

💥 Acceptance tests Core-API-Tests-ocis-storage-10 failed. The build is cancelled...

@ownclouders
Copy link
Contributor

💥 Acceptance tests Core-API-Tests-ocis-storage-4 failed. The build is cancelled...

@butonic butonic self-assigned this Aug 31, 2021
@butonic butonic force-pushed the sharestorageprovider branch from e9dbf3e to 14d080f Compare September 3, 2021 18:56
@sonarqubecloud
Copy link

sonarqubecloud bot commented Sep 3, 2021

SonarCloud Quality Gate failed.    Quality Gate failed

Bug C 1 Bug
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

17.0% 17.0% Coverage
6.4% 6.4% Duplication

@ownclouders
Copy link
Contributor

💥 Acceptance tests Core-API-Tests-ocis-storage-10 failed. The build is cancelled...

@ownclouders
Copy link
Contributor

💥 Acceptance tests Core-API-Tests-ocis-storage-1 failed. The build is cancelled...

@ownclouders
Copy link
Contributor

💥 Acceptance tests Core-API-Tests-ocis-storage-9 failed. The build is cancelled...

@ownclouders
Copy link
Contributor

💥 Acceptance tests Core-API-Tests-ocis-storage-6 failed. The build is cancelled...

@aduffeck aduffeck force-pushed the sharestorageprovider branch from 14d080f to d67a9ea Compare September 14, 2021 12:15
@micbar
Copy link
Contributor

micbar commented Dec 22, 2021

please update or close.

@aduffeck aduffeck closed this Dec 22, 2021
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.

5 participants